Larry D'Anna <larry@xxxxxxxxxxxxxx> writes:
> * Junio C Hamano (gitster@xxxxxxxxx) [100208 17:48]:
>> *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...
$ while sh t5401-*.sh -i; do :; done
... wait for a while ...
* FAIL 12: send-pack stderr contains hook messages
grep ^remote: send.err | sed "s/ *\$//" >actual &&
test_cmp - actual <expect
$ t/(364643e...); cat trash\ directory.t5401-update-hooks/actual
remote: STDOUT pre-receive
remote: STDERR pre-receive
remote: STDOUT update refs/heads/master
remote: STDERR update refs/heads/master
remote: STDOUT update refs/heads/tofail
remote: STDOUT post-receive
remote: STDERR post-receive
remote: STDOUT post-update
remote: STDERR post-update
$ t/(364643e...); cat trash\ directory.t5401-update-hooks/expect
remote: STDOUT pre-receive
remote: STDERR pre-receive
remote: STDOUT update refs/heads/master
remote: STDERR update refs/heads/master
remote: STDOUT update refs/heads/tofail
remote: STDERR update refs/heads/tofail
remote: STDOUT post-receive
remote: STDERR post-receive
remote: STDOUT post-update
remote: STDERR post-update
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html