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

Re: [PATCH] run-command: treat inaccessible directories as ENOENT

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 30, 2012, 16:21 UTC
Message-ID
<7vmx6ym54m.fsf@alter.siamese.dyndns.org>
In-Reply-To
<CAH6sp9N=JsWp7iQ=AAdXHe0J+aB5L9cBq2_0BJgUO=Y-vgAbNg@mail.gmail.com>
Frans Klaver <fransklaver@gmail.com> writes:
> isn't it rather more sensible from a unit-test perspective to see
> if start_command returns 127 instead of 128 in this specific case?

You are welcome to add another test that checks lower level implementation detail, but this specific test is to make sure the gripe "Why does git deny my aliases when I have inaccessible directory on my PATH?" will never come back.

Previous: Frans KlaverNext: Frans Klaver
Message 3 of 5 in “run-command: treat inaccessible directories as ENOENT”
  1. run-command: treat inaccessible directories as ENOENTJeff King, Mar 30, 2012
  2. Frans KlaverMar 30, 2012
  3. Junio C HamanoMar 30, 2012
  4. Frans KlaverMar 30, 2012
  5. Junio C HamanoMar 30, 2012

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.