Thread (14 messages) flat view 14 messages, 5 authors, 2016-06-15

Re: [PATCH v3 2/3] git-push: clean up some of the output from git push --porcelain

From: Larry D'Anna <hidden>
Date: 2016-06-15 22:48:12

Possibly related (same subject, not in this thread)

* Junio C Hamano (gitster@pobox.com) [100209 02:31]:
Larry D'Anna [off-list ref] writes:
quoted
* Junio C Hamano (gitster@pobox.com) [100208 17:48]:
quoted
*1* As I hinted repeatedly, I think many of them are mere churn, except
for "don't advice porcelain scripts" (good) and perhaps "exit with failure
status upon only this kind of failure" (I am undecided).  
What about the "To" lines?  It seems that they really should go to stdout (if
--porcelain is selected).  Otherwise, how does the reader of stdout know which
refs got pushed to which remote?
I think that probably is an improvement.

We would probably need documentation updates to cover the parsing of "To",
at least; it might be cleaner to have a separate section for porcelain
writers.  And tests.  But this series won't hit next until 1.7.0 ships so
we have enough time to polish the details.

I queued the series and once merged it to 'pu' but seeing some tests fail,
I reverted the merge.  The error is somewhat curious...
I bisected, and this was introduced by  

6d525d3 receive-pack: Send hook output over side band #2

  --larry
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help