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

Re: [PATCH 2/2 v2] run-command (Windows): Run dashless "git <cmd>" (solves part of problem with system_path)

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jul 29, 2008, 11:13 UTC
Message-ID
<alpine.DEB.1.00.0807291310390.4631@eeepc-johanness>
In-Reply-To
<42EF8777-BABC-4757-AD49-627320F24D39@zib.de>
Hi,
On Tue, 29 Jul 2008, Steffen Prohaska wrote:
Show 7 quoted lines
> On Jul 29, 2008, at 7:42 AM, Junio C Hamano wrote:
> 
> >   We prefer running the dashless form, and POSIX side already does so; 
> >   we should use it in MinGW's start_command(), too.
> 
> Thanks for reading my mind ;-)  This was the alternative justification I 
> had in mind after reading my patch again.

Well, given that the justification you gave had the obvious flaw -- which you even pointed out -- that non-builtins are _still_ affected, i.e. that you leave that bug unfixed (but your description purports that you want to fix that bug), it would have been wiser to give the alternative justification, which makes the commit obviously correct.

Ciao, Dscho

Previous: Steffen ProhaskaNext: Steffen Prohaska
Message 6 of 9 in “run-command (Windows): Run dashless "git <cmd>" (solves part of problem with system_path)”
  1. 2/2 run-command (Windows): Run dashless "git <cmd>" (solves part of problem with system_path)Steffen Prohaska, Jul 29, 2008
  2. [FOR DEBUGGING] Stop installing BUILT_INSSteffen Prohaska, Jul 29, 2008
  3. Shawn O. PearceJul 29, 2008
  4. Junio C HamanoJul 29, 2008
  5. Steffen ProhaskaJul 29, 2008
  6. Johannes SchindelinJul 29, 2008
  7. Steffen ProhaskaJul 29, 2008
  8. Steffen ProhaskaJul 29, 2008
  9. Johannes SchindelinJul 29, 2008

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.