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

Re: [PATCH] Install git-sh-setup.sh into $(prefix)/share/git-core. Call with explicit path.

From
Han-Wen Nienhuys <hanwen@xs4all.nl>
Date
Dec 9, 2006, 23:40 UTC
Message-ID
<457B4981.2030408@xs4all.nl>
In-Reply-To
<7v1wn8ah2k.fsf@assigned-by-dhcp.cox.net>
Junio C Hamano escreveu:
Show 9 quoted lines
> Han-Wen Nienhuys <hanwen@xs4all.nl> writes:
> 
>> And how about coding the path of git-sh-setup (in <prefix>/bin/git-sh-setup) 
>> in the shell scripts? This will ensure that the sh-setup is the one
>> from the right package, but still retain backward compatibility for 
>> scripts.
> 
> Hardcoding $(bin)/git-sh-setup is a non-starter; it breaks
> GIT_EXEC_PATH convention to allow users to override it.
> In other words, I do not know what real problem you are trying
> to solve.

The real problem is that git is nonstandard to wrap around: scripts like git-sh-setup need special attention, because they don't work with

  exec /some/where/else/git-sh-setup "$@"
I admit is a small problem, but IMO that's not an argument not to solve it.
Previous: Junio C HamanoNext: Junio C Hamano
Message 8 of 9 in “Install git-sh-setup.sh into $(prefix)/share/git-core. Call with explicit path.”
  1. Install git-sh-setup.sh into $(prefix)/share/git-core. Call with explicit path.Han-Wen Nienhuys, Dec 8, 2006
  2. Nguyen Thai Ngoc DuyDec 8, 2006
  3. Han-Wen NienhuysDec 8, 2006
  4. Eric WongDec 8, 2006
  5. Junio C HamanoDec 8, 2006
  6. Han-Wen NienhuysDec 9, 2006
  7. Junio C HamanoDec 9, 2006
  8. Han-Wen NienhuysDec 9, 2006
  9. Junio C HamanoDec 10, 2006

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.