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

Re: [PATCH trailing ws fixed] use only the PATH for exec'ing git commands

From
Alex Riesen <raa.lkml@gmail.com>
Date
Oct 23, 2007, 21:12 UTC
Message-ID
<20071023211202.GA729@steel.home>
In-Reply-To
<20071023040844.GQ16291@srparish.net>
Scott Parish, Tue, Oct 23, 2007 06:08:45 +0200:
Show 10 quoted lines
> We need to correctly set up PATH for non-c based git commands. Since we
> already do this, we can just use that PATH and execvp, instead of looping
> over the paths with execve.
> 
> This patch adds a setup_path() function to exec_cmd.c, which sets
> the PATH order correctly for our search order. execv_git_cmd() is
> stripped down to setting up argv and calling execvp(). git.c's main()
> only only needs to call setup_path().
> 
> Signed-off-by: Scott R Parish <srp@srparish.net>
Acked-by: Alex Riesen <raa.lkml@gmail.com>

Don't forget to mention it needs the patch from Johannes re "deducing exec_path from calls to git with a relative path" and your "current_exec_path"-patch. Took me a while to figure these out...

Previous: Scott ParishNext: Scott R Parish
Message 6 of 7 in “use only the PATH for exec'ing git commands”
  1. use only the PATH for exec'ing git commandsScott R Parish, Oct 22, 2007
  2. Johannes SchindelinOct 22, 2007
  3. Alex RiesenOct 22, 2007
  4. Eric MerrittOct 22, 2007
  5. use only the PATH for exec'ing git commandsScott Parish, Oct 23, 2007
  6. Alex RiesenOct 23, 2007
  7. Scott R ParishOct 23, 2007

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.