{"thread":{"id":"50540","subject":"\"Submodule registered for path\" output with config aliases mixed in","startedAt":"2019-02-19T16:51:35Z","lastAt":"2019-02-21T20:01:51Z","messageCount":8,"participants":["Mateusz Loskot","brian m. carlson","Duy Nguyen","Jeff King","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"369652","messageId":"CABUeae_N3NFXn-E1+LHORL3RDf5iTCFn=zyuOo3c2Aot2QF7pg@mail.gmail.com","threadId":"50540","inReplyTo":null,"subject":"\"Submodule registered for path\" output with config aliases mixed in","fromName":"Mateusz Loskot","fromEmail":"mateusz@loskot.net","sentAt":"2019-02-19T16:51:05Z","receivedAt":"2019-02-19T16:51:35Z","isPatch":false,"sender":{"key":"mateusz@loskot.net","avatar":"https://gravatar.com/avatar/fe5a0ce8995d4986f5c46aad6353e21717808da58e4bc710da1a100e247be239?d=mp&s=160"},"body":"Hi,\n\n$ git version\ngit version 2.20.1.windows.1\n\nI'm running `git clone --recurse-submodules https://XXX`\nThe command seems to run well and completes with success.\n\nHowever, I noticed strange output from git:\n\n```\nSubmodule 'Binary/A' (https://XXX) registered for path '!f() {\n~/AppData/Local/Fork/Fork.exe $PWD; }; fBinary/A'\nSubmodule 'Binary/B' (https://XXX) registered for path 'À   ♂♦Binary/Intergraph'\nSubmodule 'Binary/C' (https://XXX) registered for path '!sh -c 'git\nlog $1@{1}..$1@{0} $@'Binary/C'\nSubmodule 'Binary/D' (https://XXX) registered for path 'ls-files -o -i\n--exclude-standardBinary/D'\nSubmodule 'Binary/E' (https://XXX) registered for path\n'mergetool.TortoiseGitMerge.trustexitcodeBinary/E'\n```\n\nI managed to identify where that garbage injections come from:\nfrom git aliases I've got configured [1]\n\nCould anyone explain what is happening here?\nIs there anything wrong with my ~/.gitconfig [1] ?\n\n\n[1] https://github.com/mloskot/wsl-config/tree/master/git\n\n-- \nBest regards,\nMateusz Łoskot, http://mateusz.loskot.net\n"},{"id":"369724","messageId":"CABUeae9vyjWszUkfUJ7-JNmrX-7dAQDcvn9488zM4Lcr3VGwqw@mail.gmail.com","threadId":"50540","inReplyTo":"CABUeae_N3NFXn-E1+LHORL3RDf5iTCFn=zyuOo3c2Aot2QF7pg@mail.gmail.com","subject":"Re: \"Submodule registered for path\" output with config aliases mixed in","fromName":"Mateusz Loskot","fromEmail":"mateusz@loskot.net","sentAt":"2019-02-20T16:30:12Z","receivedAt":"2019-02-20T16:30:42Z","isPatch":false,"sender":{"key":"mateusz@loskot.net","avatar":"https://gravatar.com/avatar/fe5a0ce8995d4986f5c46aad6353e21717808da58e4bc710da1a100e247be239?d=mp&s=160"},"body":"Hmm, sorry for pushing, but no ideas, really?\nDoesn't it seem like a bug?\n\nMatz\n\nOn Tue, 19 Feb 2019 at 17:51, Mateusz Loskot <mateusz@loskot.net> wrote:\n>\n> Hi,\n>\n> $ git version\n> git version 2.20.1.windows.1\n>\n> I'm running `git clone --recurse-submodules https://...`\n> The command seems to run well and completes with success.\n>\n> However, I noticed strange output from git:\n>\n> ```\n> Submodule 'Binary/A' (https://...) registered for path '!f() { ~/AppData/Local/Fork/Fork.exe $PWD; }; fBinary/A'\n> Submodule 'Binary/B' (https://...) registered for path 'À   ♂♦Binary/Intergraph'\n> Submodule 'Binary/C' (https://....) registered for path '!sh -c 'git log $1@{1}..$1@{0} $@'Binary/C'\n> Submodule 'Binary/D' (https://...) registered for path 'ls-files -o -i --exclude-standardBinary/D'\n> Submodule 'Binary/E' (https://...) registered for path 'mergetool.TortoiseGitMerge.trustexitcodeBinary/E'\n> ```\n>\n> I managed to identify where that garbage injections come from:\n> from git aliases I've got configured [1]\n>\n> Could anyone explain what is happening here?\n> Is there anything wrong with my ~/.gitconfig [1] ?\n>\n>\n> [1] https://github.com/mloskot/wsl-config/tree/master/git\n>\n> --\n> Best regards,\n> Mateusz Łoskot, http://mateusz.loskot.net\n\n\n\n-- \nMateusz Loskot, http://mateusz.loskot.net\n"},{"id":"369755","messageId":"20190221021825.GB488342@genre.crustytoothpaste.net","threadId":"50540","inReplyTo":"CABUeae_N3NFXn-E1+LHORL3RDf5iTCFn=zyuOo3c2Aot2QF7pg@mail.gmail.com","subject":"Re: \"Submodule registered for path\" output with config aliases mixed in","fromName":"brian m. carlson","fromEmail":"sandals@crustytoothpaste.net","sentAt":"2019-02-21T02:18:25Z","receivedAt":"2019-02-21T02:18:34Z","isPatch":false,"sender":{"key":"sandals@crustytoothpaste.net","avatar":"https://avatars.githubusercontent.com/u/497054?v=4"},"body":"On Tue, Feb 19, 2019 at 05:51:05PM +0100, Mateusz Loskot wrote:\n> Hi,\n> \n> $ git version\n> git version 2.20.1.windows.1\n> \n> I'm running `git clone --recurse-submodules https://XXX`\n> The command seems to run well and completes with success.\n> \n> However, I noticed strange output from git:\n> \n> ```\n> Submodule 'Binary/A' (https://XXX) registered for path '!f() {\n> ~/AppData/Local/Fork/Fork.exe $PWD; }; fBinary/A'\n> Submodule 'Binary/B' (https://XXX) registered for path 'À   ♂♦Binary/Intergraph'\n> Submodule 'Binary/C' (https://XXX) registered for path '!sh -c 'git\n> log $1@{1}..$1@{0} $@'Binary/C'\n> Submodule 'Binary/D' (https://XXX) registered for path 'ls-files -o -i\n> --exclude-standardBinary/D'\n> Submodule 'Binary/E' (https://XXX) registered for path\n> 'mergetool.TortoiseGitMerge.trustexitcodeBinary/E'\n> ```\n> \n> I managed to identify where that garbage injections come from:\n> from git aliases I've got configured [1]\n> \n> Could anyone explain what is happening here?\n> Is there anything wrong with my ~/.gitconfig [1] ?\n\nI see the right behavior on my system (Debian amd64/sid) with\n2.21.0.rc0.258.g878e2cd30e and with master. I've built with ASan and\ndon't see any memory warnings on my system.\n\nDoes your .gitmodules file appear to be correct in the main repository\nwhen you clone, or does it contain unexpected entries?\n-- \nbrian m. carlson: Houston, Texas, US\nOpenPGP: https://keybase.io/bk2204\n"},{"id":"369756","messageId":"CACsJy8BKhroHhVWCj5fvwxk2z4XsdSK_pSQ=o8hxZcZsAn3R-w@mail.gmail.com","threadId":"50540","inReplyTo":"20190221021825.GB488342@genre.crustytoothpaste.net","subject":"Re: \"Submodule registered for path\" output with config aliases mixed in","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2019-02-21T03:47:45Z","receivedAt":"2019-02-21T03:48:14Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Thu, Feb 21, 2019 at 9:19 AM brian m. carlson\n<sandals@crustytoothpaste.net> wrote:\n>\n> On Tue, Feb 19, 2019 at 05:51:05PM +0100, Mateusz Loskot wrote:\n> > Hi,\n> >\n> > $ git version\n> > git version 2.20.1.windows.1\n> >\n> > I'm running `git clone --recurse-submodules https://XXX`\n> > The command seems to run well and completes with success.\n> >\n> > However, I noticed strange output from git:\n> >\n> > ```\n> > Submodule 'Binary/A' (https://XXX) registered for path '!f() {\n> > ~/AppData/Local/Fork/Fork.exe $PWD; }; fBinary/A'\n> > Submodule 'Binary/B' (https://XXX) registered for path 'À   ♂♦Binary/Intergraph'\n> > Submodule 'Binary/C' (https://XXX) registered for path '!sh -c 'git\n> > log $1@{1}..$1@{0} $@'Binary/C'\n> > Submodule 'Binary/D' (https://XXX) registered for path 'ls-files -o -i\n> > --exclude-standardBinary/D'\n> > Submodule 'Binary/E' (https://XXX) registered for path\n> > 'mergetool.TortoiseGitMerge.trustexitcodeBinary/E'\n> > ```\n> >\n> > I managed to identify where that garbage injections come from:\n> > from git aliases I've got configured [1]\n> >\n> > Could anyone explain what is happening here?\n> > Is there anything wrong with my ~/.gitconfig [1] ?\n>\n> I see the right behavior on my system (Debian amd64/sid) with\n> 2.21.0.rc0.258.g878e2cd30e and with master. I've built with ASan and\n> don't see any memory warnings on my system.\n\nI ran valgrind yesterday (gentoo amd64) and didn't find the problem\neither even though it clearly looked like some dangling pointers. I\nalso audited this code and couldn't find anything obviously wrong. My\nonly suspicion is maybe some strange getenv() behavior on Windows\n(_if_ the super prefix is used, but I can't see how...) and that's\njust a dead end for me.\n-- \nDuy\n"},{"id":"369757","messageId":"20190221041756.GA28389@sigill.intra.peff.net","threadId":"50540","inReplyTo":"CACsJy8BKhroHhVWCj5fvwxk2z4XsdSK_pSQ=o8hxZcZsAn3R-w@mail.gmail.com","subject":"Re: \"Submodule registered for path\" output with config aliases mixed in","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2019-02-21T04:17:56Z","receivedAt":"2019-02-21T04:18:00Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Feb 21, 2019 at 10:47:45AM +0700, Duy Nguyen wrote:\n\n> > > I managed to identify where that garbage injections come from:\n> > > from git aliases I've got configured [1]\n> > >\n> > > Could anyone explain what is happening here?\n> > > Is there anything wrong with my ~/.gitconfig [1] ?\n> >\n> > I see the right behavior on my system (Debian amd64/sid) with\n> > 2.21.0.rc0.258.g878e2cd30e and with master. I've built with ASan and\n> > don't see any memory warnings on my system.\n> \n> I ran valgrind yesterday (gentoo amd64) and didn't find the problem\n> either even though it clearly looked like some dangling pointers. I\n> also audited this code and couldn't find anything obviously wrong. My\n> only suspicion is maybe some strange getenv() behavior on Windows\n> (_if_ the super prefix is used, but I can't see how...) and that's\n> just a dead end for me.\n\nCertainly we fixed a bunch of getenv() problems in the upcoming 2.21\nrelease, including 8aac69038f (get_super_prefix(): copy getenv() result,\n2019-01-11).\n\nMateusz, can you try with the one of the v2.21.0 release candidates (or\nthe current tip of \"master\")?\n\n-Peff\n"},{"id":"369767","messageId":"CABUeae81zRSZrAce86R7OQgGFoc3R1Lk2X8Ujg4ZPUAGVVSknw@mail.gmail.com","threadId":"50540","inReplyTo":"20190221041756.GA28389@sigill.intra.peff.net","subject":"Re: \"Submodule registered for path\" output with config aliases mixed in","fromName":"Mateusz Loskot","fromEmail":"mateusz@loskot.net","sentAt":"2019-02-21T10:53:12Z","receivedAt":"2019-02-21T10:53:42Z","isPatch":false,"sender":{"key":"mateusz@loskot.net","avatar":"https://gravatar.com/avatar/fe5a0ce8995d4986f5c46aad6353e21717808da58e4bc710da1a100e247be239?d=mp&s=160"},"body":"On Thu, 21 Feb 2019 at 05:17, Jeff King <peff@peff.net> wrote:\n> On Thu, Feb 21, 2019 at 10:47:45AM +0700, Duy Nguyen wrote:\n>\n> > > > I managed to identify where that garbage injections come from:\n> > > > from git aliases I've got configured [1]\n> > > >\n> > > > Could anyone explain what is happening here?\n> > > > Is there anything wrong with my ~/.gitconfig [1] ?\n> > >\n> > > I see the right behavior on my system (Debian amd64/sid) with\n> > > 2.21.0.rc0.258.g878e2cd30e and with master. I've built with ASan and\n> > > don't see any memory warnings on my system.\n> >\n> > I ran valgrind yesterday (gentoo amd64) and didn't find the problem\n> > either even though it clearly looked like some dangling pointers. I\n> > also audited this code and couldn't find anything obviously wrong. My\n> > only suspicion is maybe some strange getenv() behavior on Windows\n> > (_if_ the super prefix is used, but I can't see how...) and that's\n> > just a dead end for me.\n>\n> Certainly we fixed a bunch of getenv() problems in the upcoming 2.21\n> release, including 8aac69038f (get_super_prefix(): copy getenv() result,\n> 2019-01-11).\n>\n> Mateusz, can you try with the one of the v2.21.0 release candidates (or\n> the current tip of \"master\")?\n\nJeff, I have just tried git version 2.21.0.rc2.windows.1 and\nI no longer see the problem.\nIt seems like a bug in earlier version indeed, that has been fixed now.\n\nBrian, yes, the .gitmodules was perfectly correct.\n\nBrian, Duy, Jeff, thank you all for responding; the problem has\nbeen solved for me now.\n\nBest regards,\n-- \nMateusz Loskot, http://mateusz.loskot.net\n"},{"id":"369803","messageId":"20190221130717.GB20536@sigill.intra.peff.net","threadId":"50540","inReplyTo":"CABUeae81zRSZrAce86R7OQgGFoc3R1Lk2X8Ujg4ZPUAGVVSknw@mail.gmail.com","subject":"Re: \"Submodule registered for path\" output with config aliases mixed in","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2019-02-21T13:07:17Z","receivedAt":"2019-02-21T13:07:20Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Feb 21, 2019 at 11:53:12AM +0100, Mateusz Loskot wrote:\n\n> On Thu, 21 Feb 2019 at 05:17, Jeff King <peff@peff.net> wrote:\n> > On Thu, Feb 21, 2019 at 10:47:45AM +0700, Duy Nguyen wrote:\n> >\n> > > > > I managed to identify where that garbage injections come from:\n> > > > > from git aliases I've got configured [1]\n> > > > >\n> > > > > Could anyone explain what is happening here?\n> > > > > Is there anything wrong with my ~/.gitconfig [1] ?\n> > > >\n> > > > I see the right behavior on my system (Debian amd64/sid) with\n> > > > 2.21.0.rc0.258.g878e2cd30e and with master. I've built with ASan and\n> > > > don't see any memory warnings on my system.\n> > >\n> > > I ran valgrind yesterday (gentoo amd64) and didn't find the problem\n> > > either even though it clearly looked like some dangling pointers. I\n> > > also audited this code and couldn't find anything obviously wrong. My\n> > > only suspicion is maybe some strange getenv() behavior on Windows\n> > > (_if_ the super prefix is used, but I can't see how...) and that's\n> > > just a dead end for me.\n> >\n> > Certainly we fixed a bunch of getenv() problems in the upcoming 2.21\n> > release, including 8aac69038f (get_super_prefix(): copy getenv() result,\n> > 2019-01-11).\n> >\n> > Mateusz, can you try with the one of the v2.21.0 release candidates (or\n> > the current tip of \"master\")?\n> \n> Jeff, I have just tried git version 2.21.0.rc2.windows.1 and\n> I no longer see the problem.\n> It seems like a bug in earlier version indeed, that has been fixed now.\n\nGreat, thanks for confirming.\n\nJunio, Johannes: I don't know if there were particular plans to pick up\nthose getenv() fixes for \"maint\", but if we're going to do a v2.20.2, it\nmight be worth it.  It's also possible that it's just due to ca1b411648\n(mingw: safe-guard a bit more against getenv() problems, 2019-02-15),\nbut I'd consider that to be maint-worthy as well.\n\nIt may be that we just don't do a v2.20.2, though, since v2.21 is so\nclose to release.\n\n-Peff\n"},{"id":"369839","messageId":"xmqqpnrkud05.fsf@gitster-ct.c.googlers.com","threadId":"50540","inReplyTo":"20190221130717.GB20536@sigill.intra.peff.net","subject":"Re: \"Submodule registered for path\" output with config aliases mixed in","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-02-21T20:01:46Z","receivedAt":"2019-02-21T20:01:51Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King <peff@peff.net> writes:\n\n> Junio, Johannes: I don't know if there were particular plans to pick up\n> those getenv() fixes for \"maint\", but if we're going to do a v2.20.2, it\n> might be worth it.  It's also possible that it's just due to ca1b411648\n> (mingw: safe-guard a bit more against getenv() problems, 2019-02-15),\n> but I'd consider that to be maint-worthy as well.\n>\n> It may be that we just don't do a v2.20.2, though, since v2.21 is so\n> close to release.\n\nYeah, they are all low-risk and straight-forward changes, and were\nqueued on a topic branch forked at 2.19.2 to make it possible to\ndirectly merge to older maintenance tracks.  So if we are going to\nupdate 2.20.x (or even 2.19.x) series, we can certainly include\nthem.\n\nThanks.\n"}]}