From: Thomas Rast <trast@student.ethz.ch>
To: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Cc: <git@vger.kernel.org>
Subject: Re: [PATCH 2/2] Documentation: make git-sh-setup docs less scary
Date: Tue, 20 Dec 2011 21:27:44 +0100 [thread overview]
Message-ID: <87bor3ounz.fsf@thomas.inf.ethz.ch> (raw)
In-Reply-To: <CACBZZX5BSfh8S9Kf1Wbi+NPEKpxJNXU8TD8-hkC2o1Mi91Or6A@mail.gmail.com> ("Ævar Arnfjörð Bjarmason"'s message of "Tue, 20 Dec 2011 20:45:05 +0100")
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> On Tue, Dec 20, 2011 at 12:09, Thomas Rast <trast@student.ethz.ch> wrote:
>> At least one IRC user was scared away by the introductory "This is not
>> a command the end user would want to run. Ever." to the point of not
>> reading on.
>
> Arguably that's the point isn't it? To not have people who aren't
> maintaining Git itself waste time on reading it.
Junio C Hamano writes:
> You would need to say what that IRC user needed to find out. Depending on
> that, letting the user know that there is no point reading on early and
> not waste his or her time may be a good thing. That was what the paragraph
> was designed for. IOW, it is not to "scare" away, but to allow the users
> to decide if they are intended audiences.
Well, the original question was [*]
<ribasushi> how do I write a batch-test (goes in a script, exit value matters only) to test if the current workdir is clean?
<ribasushi> i.e. nothing staged/unstaged to commit
<shruggar> last I looked, there was no "all in one" method :/
<shruggar> git diff -q && git diff —cached -q, perhaps
<charon> ribasushi: . "$(git --exec-path)/git-sh-setup" ; require_clean_worktree
<shruggar> but I think that's ignoring something
<ribasushi> lack of output of `git status -s` seems to be what I want
<ribasushi> hmmm
<ribasushi> charon: git-gh-setup's manpage does not seem to list what you gave me
<ribasushi> and the manpage says that I should not be touching it...
Leaving aside my worktree vs. work_tree mistake, he concluded that he
should not be using it even though he is a script writer.
Regardless, I don't really care enough about this one; let's just do the
first patch (v2 upcoming) so that we have documentation to point people
at.
[*] http://colabti.org/irclogger/irclogger_log/git?date=2011-12-20#l1284
--
Thomas Rast
trast@{inf,student}.ethz.ch
next prev parent reply other threads:[~2011-12-20 20:27 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-20 11:09 [PATCH 1/2] git-sh-setup: make require_clean_work_tree part of the interface Thomas Rast
2011-12-20 11:09 ` [PATCH 2/2] Documentation: make git-sh-setup docs less scary Thomas Rast
2011-12-20 19:20 ` Junio C Hamano
2011-12-20 19:45 ` Ævar Arnfjörð Bjarmason
2011-12-20 20:27 ` Thomas Rast [this message]
2011-12-20 19:25 ` [PATCH 1/2] git-sh-setup: make require_clean_work_tree part of the interface Junio C Hamano
2011-12-20 20:29 ` [PATCH v2] " Thomas Rast
2011-12-20 20:52 ` Junio C Hamano
2011-12-20 21:42 ` [PATCH v3] " Thomas Rast
2011-12-20 22:05 ` Junio C Hamano
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=87bor3ounz.fsf@thomas.inf.ethz.ch \
--to=trast@student.ethz.ch \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.