From: Junio C Hamano <gitster@pobox.com>
To: nadim khemir <nadim@khemir.net>
Cc: "git list" <git@vger.kernel.org>
Subject: Re: Adding Exit status documentation to all git commands starting with git status
Date: Sat, 13 Dec 2008 12:04:52 -0800 [thread overview]
Message-ID: <7v4p176duj.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <200812132036.39318.nadim@khemir.net> (nadim khemir's message of "Sat, 13 Dec 2008 20:36:39 +0100")
nadim khemir <nadim@khemir.net> writes:
> There are different styles to add exit status, give me your input on why and
> why not using one or the other. I list 3 solutions and what I think about
> them.
No matter what you do, I think EXIT STATUS section should consistently
come near the end of the document, immediately before SEE ALSO, which is
where people who know how manual pages are written expect to find it.
> EXIT STATUS
> -----------
> The command exits with non-zero status if there is no path that is
> different between the index file and the current HEAD commit (i.e.,
> there is nothing to commit by running `git commit`).
Prose is much easier to read as long as it is brief enough, than two-item
enumeration:
> EXIT STATUS
> -----------
> Zero status: There is a different between the index file and HEAD.
> Non-zero status: There is nothing to commit by running `git commit`.
whose use of "Zero status" makes it look doubly funny (traditionally
manual pages do not seem to spell out exit status 0 as "zero", but
"non-zero" is Ok).
Avoid talking about only one side of the condition if you can without
being too verbose.
The command exits with status 0 if there is something to commit by
running `git commit` with corresponding arguments, and non-zero
otherwise.
prev parent reply other threads:[~2008-12-13 20:06 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-13 19:36 Adding Exit status documentation to all git commands starting with git status nadim khemir
2008-12-13 20:04 ` Junio C Hamano [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7v4p176duj.fsf@gitster.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=nadim@khemir.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).