git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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
Previous: Johannes SchindelinNext: Johannes Schindelin
Message 9 of 15 in “remote: rename also remotes in the branch.<name>.pushRemote config”
  1. remote: rename also remotes in the branch.<name>.pushRemote configBert Wesarg, Jan 16, 2020
  2. Junio C HamanoJan 16, 2020
  3. remote rename: rename branch.<name>.pushRemote config values tooBert Wesarg, Jan 17, 2020
  4. Johannes SchindelinJan 17, 2020
  5. Bert WesargJan 17, 2020
  6. Johannes SchindelinJan 17, 2020
  7. Bert WesargJan 17, 2020
  8. Johannes SchindelinJan 20, 2020
  9. Bert WesargJan 20, 2020
  10. Johannes SchindelinJan 20, 2020
  11. Junio C HamanoJan 17, 2020
  12. Bert WesargJan 17, 2020
  13. Junio C HamanoJan 17, 2020
  14. Johannes SchindelinJan 17, 2020
  15. Johannes SchindelinJan 17, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.