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

Re: [PATCH] push: don't show Done with --quiet --porcelain

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 4, 2015, 21:42 UTC
Message-ID
<xmqqbndhg8un.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<CA+WR+g7V0Rez_oug5P_LDmafQKXqnDHDNN_vk=-1Sbj3-754DQ@mail.gmail.com>
Josh Rabinowitz <joshr@joshr.com> writes:
Show 26 quoted lines
> Hi Junio and other recipients:
>
> Junio, thanks for your response. I think you may have misunderstood my
> patch though (or I am misunderstanding your responses), because it
> seems we are actually in agreement.
>
> 1) My original patch is to make 'git push --porcelain --quiet' not
> emit 'Done' when there is no error. It would continue to emit "Done"
> when using 'git push --porcelain' without an error.
>
> 2) In your first paragraph, you seem to state that while printing
> "Done" is advantageous when using 'git push --porcelain' without
> --quiet, the "Done" output isn't needed when --quiet is used. This
> appears to agree with my patch's intent.
>
> 3) in your second paragraph, you seem to agree with me again, that
> "Done" is not needed when "git push --porcelain --quiet" is use
>
> 4) Then in your third paragraph, you say that you don't want to remove
> the "Done" output when using "git push --porcelain" without --quiet --
> which my patch preserves (again, it would only remove the "Done" text
> when 'git push --porcelain' is used with --quiet and there is no
> error.)
>
> In summary, I think we are in agreement that this patch is probably
> acceptable. Look forward to reading comments.
I think your 4. misinterprets what I meant to say.

Even if we agree 1 thru 3, changing the output, with or without "--quiet", is an unwelcome thing to do to existing scripts.

Previous: Josh RabinowitzNext: Josh Rabinowitz
Message 4 of 5 in “push: don't show Done with --quiet --porcelain”
  1. push: don't show Done with --quiet --porcelainJosh Rabinowitz, Aug 31, 2015
  2. Junio C HamanoSep 1, 2015
  3. Josh RabinowitzSep 4, 2015
  4. Junio C HamanoSep 4, 2015
  5. Josh RabinowitzSep 10, 2015

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.