Re: [GSoC][PATCH v5 3/3] submodule: port subcommand foreach from shell to C
- From
- Brandon Williams <bmwill@google.com>
- Date
- May 26, 2017, 22:03 UTC
- Message-ID
- <20170526220319.GC4592@google.com>
- In-Reply-To
- <83801d05-e3a9-b7e4-8f29-cd8172257dac@kdbg.org>
On 05/26, Johannes Sixt wrote:
Show 15 quoted lines
> Am 26.05.2017 um 17:17 schrieb Prathamesh Chavan: > >+ argv_array_pushf(&cp.env_array, "path=%s", list_item->name); > > Not good! On Windows, environment variables are case insensitive. > The environment variable "path" has a very special purpose, although > it is generally spelled "PATH" (actually "Path" on Windows). > > Lowercase "path" may have worked as long as it was only used in a > shell script (and perhaps only by lucky coincidence), but this I can > pretty much guarantee to fail. (I haven't tested it, though.) > > The correct fix can only be to rename this variable here and in > shell scripts that need the value that is set here. > > -- Hannes
Nice catch, the only reason why this would have worked before was because it was a shell script before...
-- Brandon Williams