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

Re: [PATCH 1/3] Set cmdline globally, not in stop_here_user_resolve

From
Ramkumar Ramachandra <artagnon@gmail.com>
Date
Jun 1, 2010, 09:36 UTC
Message-ID
<AANLkTinY8Z-Y_1aFnJrojaAMs68FITt-whC0fLbpIR8h@mail.gmail.com>
In-Reply-To
<4C04D268.6070908@viscovery.net>
Hi,
> Did you test this? "$0" prints the full path to git-am. This does not
> count as user-friendly in my book!

Doesn't it print out exactly the way you invoke the application ie. either "./git-am.sh" or "git am"? Then again, when git-am is invoked internally by git, it possibly uses the full path instead of relying on $PATH, and this is quite ugly. I noticed that all the other other shell scripts in git.git also hardcode the string, and there must be a good reason for this. Therefore I think this particular change should be dropped.

Thanks for pointing this out.
-- Ram
Previous: Johannes SixtNext: Ramkumar Ramachandra
Message 4 of 7 in “Improve git-am error messages”
  1. 0/3 Improve git-am error messagesRamkumar Ramachandra, Jun 1, 2010
  2. 1/3 Set cmdline globally, not in stop_here_user_resolveRamkumar Ramachandra, Jun 1, 2010
  3. Johannes SixtJun 1, 2010
  4. Ramkumar RamachandraJun 1, 2010
  5. 2/3 Display some help text when patch is emptyRamkumar Ramachandra, Jun 1, 2010
  6. 3/3 Remove stray error message from sedRamkumar Ramachandra, Jun 1, 2010
  7. Junio C HamanoJun 1, 2010

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.