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

Re: Bug: git branch returns EXIT_SUCCESS when no branches found

From
CSChristoffer Stjernlöf <a@xkqr.org>
Date
Apr 24, 2019, 10:53 UTC
Message-ID
<87wojjhdx7.fsf@xkqr.org>
In-Reply-To
<xmqqimv31y3n.fsf@gitster-ct.c.googlers.com>
Dear Junio,
Junio C Hamano (gitster@pobox.com) 2019-04-24:
Show 7 quoted lines
> if cs/topic branch does not exist, of if the branch
> does not contain 'master', the command will show "here are the
> branches that match the criteria you gave me" by giving no lines in
> its standard output, and exits with 0 to signal that what you see on
> the screen indeed was computed correctly (as opposed to the command
> internally crashing and failing to produce a result, in which case
> you would want to be told with non-zero exit status).

Thank you for taking the time to elaborate on your already detailed answer. I can see the concern about separating internal command errors from correctly computed results, even if those results seem to indicate something being amiss. Is this not a place where different error codes can be useful?

Sincerely yours, Christoffer

Previous: Junio C HamanoNext: Christoffer Stjernlöf
Message 4 of 6 in “Bug: git branch returns EXIT_SUCCESS when no branches found”
  1. Christoffer StjernlöfApr 24, 2019
  2. Junio C HamanoApr 24, 2019
  3. Junio C HamanoApr 24, 2019
  4. Christoffer StjernlöfApr 24, 2019
  5. Christoffer StjernlöfApr 24, 2019
  6. SZEDER GáborApr 24, 2019

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.