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

Re: [PATCH] add --porcelain option to git-push

From
Marc Branchaud <marcnarc@xiplink.com>
Date
Jun 23, 2009, 17:09 UTC
Message-ID
<4A410C64.9030903@xiplink.com>
In-Reply-To
<7v4ou79cga.fsf@alter.siamese.dyndns.org>
Both good points.  --porcelain is fine by me.
		M.
Junio C Hamano wrote:
Show 33 quoted lines
> Marc Branchaud <marcnarc@xiplink.com> writes:
> 
>> Shouldn't this option be named "--plumbing" since it's making 'git push'
>> act like plumbing?  Actually, neither name seems intuitively descriptive
>> to me...
> 
> Perhaps.  But asking for output format designed for Porcelain
> implementions to read with --porcelain option has precedence.
> 
>> Why not teach 'git push' to change its output format if it's writing to a pipe?
> 
> That is ugly.  Besides, "writing to a pipe" would not be a right criteria,
> if you want to do
> 
>           git push >log
>           if grep blah log
>           then
>                 do blah thing
>           fi
>           if grep baa log
>           then
>                 do baa thing, too
>           fi
> 
> When you make a program behave differently depending on where your stdout
> goes, typically you see if it is going to the terminal (e.g. isatty(3)),
> but even then you would need an explicit override from the command line
> when stdout is a tty and you do not want "for humans" frills (e.g. color),
> and when stdout is not a tty and you do want such frills.
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous: Junio C HamanoNext: Markus Heidelberg
Message 7 of 13 in “I'd like to be able to know what git-push will do”
  1. Larry D'AnnaJun 18, 2009
  2. Jeff KingJun 22, 2009
  3. add --porcelain option to git-pushLarry D'Anna, Jun 23, 2009
  4. Marc BranchaudJun 23, 2009
  5. add --plumbing option to git-pushLarry D'Anna, Jun 23, 2009
  6. Junio C HamanoJun 23, 2009
  7. Marc BranchaudJun 23, 2009
  8. Markus HeidelbergJun 23, 2009
  9. Constantine PlotnikovJun 23, 2009
  10. Larry D'AnnaJun 24, 2009
  11. Junio C HamanoJun 25, 2009
  12. Larry D'AnnaJun 25, 2009
  13. add --summary option to git-pushLarry D'Anna, Jun 27, 2009

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.