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

Re: "git stash" is not known to git

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 30, 2007, 17:52 UTC
Message-ID
<7vtzrlkccp.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<20070730100408.GA8829@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 17 quoted lines
> On a related note, is it just me, or is the following comment and related code
> in git.c (introduced by Linus in 231af832) totally bogus:
>
>   /*
>    * We search for git commands in the following order:
>    *  - git_exec_path()
>    *  - the path of the "git" command if we could find it
>    *    in $0
>    *  - the regular PATH.
>    */
>
> We never actually look in the regular PATH since we call execv_git_cmd
> (although we do still munge the PATH, apparently so shell scripts can
> use git-foo syntax; see 77cb17e9). This means you can't drop "git-foo"
> into your PATH and have it work as "git foo".
>
> What is the desired behavior?

I failed to spot patches to the area without updating the comment. What the code does is fine, the comment is stale.

Previous: Jeff KingNext: Junio C Hamano
Message 11 of 14 in “"git stash" is not known to git”
  1. David KastrupJul 30, 2007
  2. Matthieu MoyJul 30, 2007
  3. David KastrupJul 30, 2007
  4. Jeff KingJul 30, 2007
  5. David KastrupJul 30, 2007
  6. Jeff KingJul 30, 2007
  7. David KastrupJul 30, 2007
  8. Jeff KingJul 30, 2007
  9. David KastrupJul 30, 2007
  10. Jeff KingJul 30, 2007
  11. Junio C HamanoJul 30, 2007
  12. Junio C HamanoJul 31, 2007
  13. Jeff KingJul 31, 2007
  14. Junio C HamanoJul 30, 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.