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

Re: PATH modifications for git-hook processes

From
Jeff King <peff@peff.net>
Date
Apr 16, 2015, 06:31 UTC
Message-ID
<20150416063137.GA7761@peff.net>
In-Reply-To
<20150416061732.GA5612@peff.net>
On Thu, Apr 16, 2015 at 02:17:32AM -0400, Jeff King wrote:
Show 11 quoted lines
> So it uses a special git-specific version of exec, and doesn't touch the
> PATH. Later, we did 231af83 (Teach the "git" command to handle some
> commands internally, 2006-02-26), which says at the end:
> 
>    There's one other change: the search order for external programs is
>    modified slightly, so that the first entry remains GIT_EXEC_DIR, but
>    the second entry is the same directory as the git wrapper itself was
>    executed out of - if we can figure it out from argv[0], of course.
> 
> There was some question of whether this would be a problem[1], but we
> realized it is OK due to 77cb17e, mentioned above.
I forgot to include my footnote, which was a link to the thread:
  http://thread.gmane.org/gmane.comp.version-control.git/16798
-Peff
Previous: Jeff KingNext: David Rodríguez
Message 5 of 12 in “PATH modifications for git-hook processes”
  1. Matthew RothenbergApr 14, 2015
  2. Junio C HamanoApr 14, 2015
  3. Matthew RothenbergApr 15, 2015
  4. Jeff KingApr 16, 2015
  5. Jeff KingApr 16, 2015
  6. David RodríguezApr 22, 2015
  7. Junio C HamanoApr 22, 2015
  8. stop putting argv[0] dirname at front of PATHJeff King, Apr 22, 2015
  9. Eric SunshineApr 22, 2015
  10. Jeff KingApr 22, 2015
  11. Jonathan NiederApr 22, 2015
  12. Jeff KingApr 22, 2015

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.