[PR #2255] [MERGED] Temporarily patch go-ini/ini with fork #16309

New Issue

GiteaMirror · 2025-11-02T12:07:19-06:00

GiteaMirror commented

2025-11-02 12:07:19 -06:00

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/2255
Author: @ethantkoenig
Created: 8/4/2017
Status: ✅ Merged
Merged: 8/7/2017
Merged by: @lunny

Base: master ← Head: fix/docker_ini

📝 Commits (1)

84534a6 Temporarily patch go-ini/ini with fork

📊 Changes

8 files changed (+515 additions, -160 deletions)

View changed files

📝 vendor/gopkg.in/ini.v1/README.md (+50 -7)
📝 vendor/gopkg.in/ini.v1/README_ZH.md (+53 -10)
📝 vendor/gopkg.in/ini.v1/ini.go (+104 -49)
📝 vendor/gopkg.in/ini.v1/key.go (+107 -41)
📝 vendor/gopkg.in/ini.v1/parser.go (+54 -18)
📝 vendor/gopkg.in/ini.v1/section.go (+49 -7)
📝 vendor/gopkg.in/ini.v1/struct.go (+94 -25)
📝 vendor/vendor.json (+4 -3)

📄 Description

~~A short-term fix for #2197.~~

~~Fixing the underlying problem will involve changing the SaveTo(..) method in the go-ini/ini package, but this will at least~~

~~Let Docker users run gitea in the meantime~~
~~Mitigate the problem enough to not block the 1.2.0 release~~

Fixes #2197. Updates the SaveTo(..) method in the go-ini/ini package. For the sake of not blocking the 1.2.0 release, I've temporarily updated the dependency to a fork. Once the fix to the SaveTo(..) method is merged into the main repository (go-ini/ini#115), we can revert.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/gitea/pull/2255 **Author:** [@ethantkoenig](https://github.com/ethantkoenig) **Created:** 8/4/2017 **Status:** ✅ Merged **Merged:** 8/7/2017 **Merged by:** [@lunny](https://github.com/lunny) **Base:** `master` ← **Head:** `fix/docker_ini` --- ### 📝 Commits (1) - [`84534a6`](https://github.com/go-gitea/gitea/commit/84534a6536c67b5f145f15fc2bbd0fa7bd682139) Temporarily patch go-ini/ini with fork ### 📊 Changes **8 files changed** (+515 additions, -160 deletions) <details> <summary>View changed files</summary> 📝 `vendor/gopkg.in/ini.v1/README.md` (+50 -7) 📝 `vendor/gopkg.in/ini.v1/README_ZH.md` (+53 -10) 📝 `vendor/gopkg.in/ini.v1/ini.go` (+104 -49) 📝 `vendor/gopkg.in/ini.v1/key.go` (+107 -41) 📝 `vendor/gopkg.in/ini.v1/parser.go` (+54 -18) 📝 `vendor/gopkg.in/ini.v1/section.go` (+49 -7) 📝 `vendor/gopkg.in/ini.v1/struct.go` (+94 -25) 📝 `vendor/vendor.json` (+4 -3) </details> ### 📄 Description ~A short-term fix for #2197.~ ~Fixing the underlying problem will involve changing the `SaveTo(..)` method in the `go-ini/ini` package, but this will at least~ 1. ~Let Docker users run gitea in the meantime~ 2. ~Mitigate the problem enough to not block the 1.2.0 release~ Fixes #2197. Updates the `SaveTo(..)` method in the `go-ini/ini` package. For the sake of not blocking the 1.2.0 release, I've temporarily updated the dependency to a fork. Once the fix to the `SaveTo(..)` method is merged into the main repository (go-ini/ini#115), we can revert. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2025-11-02 12:07:19 -06:00

GiteaMirror closed this issue

2025-11-02 12:07:19 -06:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/gitea#16309