Re: [PATCH v2] remote rename: rename branch.<name>.pushRemote config values too
- From
Bert Wesarg <bert.wesarg@googlemail.com>
- Date
- Jan 20, 2020, 13:14 UTC
- Message-ID
- <CAKPyHN0v9eJWXBCmBS6kdPXx5F8CGE_P3hSfb8+vMaPTF9FWug@mail.gmail.com>
- In-Reply-To
- <nycvar.QRO.7.76.6.2001201221350.46@tvgsbejvaqbjf.bet>
Hi Dscho,
On Mon, Jan 20, 2020 at 12:25 PM Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:
Show 27 quoted lines
> > Hi Bert, > > On Fri, 17 Jan 2020, Bert Wesarg wrote: > > > On Fri, Jan 17, 2020 at 2:30 PM Johannes Schindelin > > <Johannes.Schindelin@gmx.de> wrote: > > > > > > On Fri, 17 Jan 2020, Bert Wesarg wrote: > > > > > > > On Fri, Jan 17, 2020 at 12:50 PM Johannes Schindelin > > > > <Johannes.Schindelin@gmx.de> wrote: > > > > AFAIU, the value of remote.pushDefault wont be renamed yet. So you > > > > suggest to issue a warning in case remote.pushDefault is X. But as X > > > > does not exists anymore after the rename, the value of > > > > remote.pushDefault is invalid. So why not rename it too? > > > > > > If this setting was usually a repository-specific one, I would suggest to > > > change its value, too. But it is my understanding that this might be set > > > in `~/.gitconfig` more often than not, so I recommend a warning instead. > > > > than why not rename it, if its a repository-specific setting and warn > > if it is a global one? If this is detectable at all. > > Sure, but you might need to re-parse the config to detect that (and you > have to use `git_config_from_file()` to make sure that you know that you > are looking at the repository config and not at anything else).
I found current_config_scope() which serves the purpose for me. Anything wrong with this approach?
Best, Bert
> > Ciao, > Dscho