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

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

flat view

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:48:12

Possibly related (same subject, not in this thread)

Larry D'Anna [off-list ref] writes:
also these

X TAB ...
! TAB ...
* TAB ...
+ TAB ...
SPACE TAB ...
Is this documented anywhere?

Much of what your patches in this thread tried to do does not interest me
very much rignt now, i.e. during the pre-release freeze period [*1*].

If you are already showing '*' and '+' without telling the users in the
documentation, it is a much more urgent issue that needs fixing, and it
would be preferrable if we can do so before the release.

Documentation/git-push.txt seems to say that only ' ', '!' and '=' have
any defined meaning:

    flag::
            A single character indicating the status of the ref. This is
            blank for a successfully pushed ref, `!` for a ref that was
            rejected or failed to push, and '=' for a ref that was up to
            date and did not need pushing (note that the status of up to
            date refs is shown only when `git push` is running verbosely).


[Footnote]

*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).  Especially I
want to see the "send advice to the standard error stream" one as a
separate patch, if you insist including that change in the series.

But they are something we can decide after 1.7.0 happens.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help