From: Junio C Hamano <junio@pobox.com>
To: Jon Loeliger <jdl@jdl.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Clarify and fix English in git-rm documentation.
Date: Wed, 16 Apr 2008 00:06:40 -0700 [thread overview]
Message-ID: <7vve2is2gv.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <E1Jl91r-0000yT-8T@jdl.com> (Jon Loeliger's message of "Sun, 13 Apr 2008 15:42:23 -0500")
Jon Loeliger <jdl@jdl.com> writes:
> diff --git a/Documentation/git-rm.txt b/Documentation/git-rm.txt
> index dc36c66..2066def 100644
> --- a/Documentation/git-rm.txt
> +++ b/Documentation/git-rm.txt
> @@ -11,28 +11,34 @@ SYNOPSIS
>
> DESCRIPTION
> -----------
> +Remove files from the index, or from the working tree and the index.
> +`git rm` will not remove a file from just your working directory.
The last sentence made me stop reading and read the paragraph twice and
half. Perhaps it would make it easier to read if we make it
(there is no option to remove a file only from work tree and keep
it in the index; use regular "/bin/rm" if you want to do that).
a parenthesized note as the tail part of the previous sentence?
> +The files being removed have to be identical to the tip of the branch,
> +and no updates to their contents can be staged in the index.
Perhaps add "This can be overridden with the `-f` option" here?
next prev parent reply other threads:[~2008-04-16 7:07 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-13 20:42 [PATCH] Clarify and fix English in git-rm documentation Jon Loeliger
2008-04-16 7:06 ` Junio C Hamano [this message]
2008-04-16 13:29 ` Jon Loeliger
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=7vve2is2gv.fsf@gitster.siamese.dyndns.org \
--to=junio@pobox.com \
--cc=git@vger.kernel.org \
--cc=jdl@jdl.com \
/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.