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

Re: "Submodule registered for path" output with config aliases mixed in

From
Mateusz Loskot <mateusz@loskot.net>
Date
Feb 21, 2019, 10:53 UTC
Message-ID
<CABUeae81zRSZrAce86R7OQgGFoc3R1Lk2X8Ujg4ZPUAGVVSknw@mail.gmail.com>
In-Reply-To
<20190221041756.GA28389@sigill.intra.peff.net>
On Thu, 21 Feb 2019 at 05:17, Jeff King <peff@peff.net> wrote:
Show 25 quoted lines
> On Thu, Feb 21, 2019 at 10:47:45AM +0700, Duy Nguyen wrote:
>
> > > > I managed to identify where that garbage injections come from:
> > > > from git aliases I've got configured [1]
> > > >
> > > > Could anyone explain what is happening here?
> > > > Is there anything wrong with my ~/.gitconfig [1] ?
> > >
> > > I see the right behavior on my system (Debian amd64/sid) with
> > > 2.21.0.rc0.258.g878e2cd30e and with master. I've built with ASan and
> > > don't see any memory warnings on my system.
> >
> > I ran valgrind yesterday (gentoo amd64) and didn't find the problem
> > either even though it clearly looked like some dangling pointers. I
> > also audited this code and couldn't find anything obviously wrong. My
> > only suspicion is maybe some strange getenv() behavior on Windows
> > (_if_ the super prefix is used, but I can't see how...) and that's
> > just a dead end for me.
>
> Certainly we fixed a bunch of getenv() problems in the upcoming 2.21
> release, including 8aac69038f (get_super_prefix(): copy getenv() result,
> 2019-01-11).
>
> Mateusz, can you try with the one of the v2.21.0 release candidates (or
> the current tip of "master")?

Jeff, I have just tried git version 2.21.0.rc2.windows.1 and I no longer see the problem. It seems like a bug in earlier version indeed, that has been fixed now.

Brian, yes, the .gitmodules was perfectly correct.

Brian, Duy, Jeff, thank you all for responding; the problem has been solved for me now.

Best regards,
-- 
Mateusz Loskot, http://mateusz.loskot.net
Previous: Jeff KingNext: Jeff King
Message 6 of 8 in “"Submodule registered for path" output with config aliases mixed in”
  1. Mateusz LoskotFeb 19, 2019
  2. Mateusz LoskotFeb 20, 2019
  3. brian m. carlsonFeb 21, 2019
  4. Duy NguyenFeb 21, 2019
  5. Jeff KingFeb 21, 2019
  6. Mateusz LoskotFeb 21, 2019
  7. Jeff KingFeb 21, 2019
  8. Junio C HamanoFeb 21, 2019

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.