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

Re: Issue with 2.11.0 and GIT_EXEC_PATH with multiple entries

From
Nate Mueller <nate@retailnext.net>
Date
Mar 31, 2017, 00:39 UTC
Message-ID
<CAJF7t-cvCN1wMN1FzNPTG5zOC-d1g0GNxL-necZF7BOfUyrKog@mail.gmail.com>
In-Reply-To
<20170330233228.dfv5mxnwlppghmua@sigill.intra.peff.net>
Got it.  Thanks!
On Thu, Mar 30, 2017 at 4:32 PM, Jeff King <peff@peff.net> wrote:
Show 20 quoted lines
> On Thu, Mar 30, 2017 at 04:00:41PM -0700, Nate Mueller wrote:
>
>> Really?  My config has been set this way for years and it's never
>> caused problems before.  I have subcommands in both of those
>> directories and all work.
>
> Really. It did happen to work most of the time before (because most uses
> involved just appending it to $PATH). But it was never intended to work
> with multiple paths. The:
>
>   . "$(git --exec-path)/git-sh-whatever"
>
> advice has been advertised in the documentation for years. E.g., see
> bd870878f (Documentation: don't assume git-sh-setup and git-parse-remote
> are in PATH, 2008-06-29). So even if we wanted to relax the rules in our
> scripts, it seems like a potential hazard for 3rd party scripts.
>
> The recommended way is to just put your ~/.git-exec into your $PATH.
>
> -Peff
-- 
Nate Mueller - Head of Engineering - RetailNext - 406-356-6283
Previous: Jeff King
Message 5 of 5 in “Issue with 2.11.0 and GIT_EXEC_PATH with multiple entries”
  1. Nate MuellerMar 30, 2017
  2. Junio C HamanoMar 30, 2017
  3. Nate MuellerMar 30, 2017
  4. Jeff KingMar 30, 2017
  5. Nate MuellerMar 31, 2017

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.