From: "Ping Yin" <pkufranky@gmail.com>
To: "Sverre Hvammen Johansen" <hvammen@gmail.com>
Cc: "Junio C Hamano" <gitster@pobox.com>, git@vger.kernel.org
Subject: Re: [PATCH 2/5] Introduce -ff=<fast forward option>
Date: Fri, 2 May 2008 20:33:00 +0800 [thread overview]
Message-ID: <46dff0320805020533u179e16cej2c4db3105252d0bd@mail.gmail.com> (raw)
In-Reply-To: <402c10cd0804232246i6f7cd946h955b75832cf32876@mail.gmail.com>
On Thu, Apr 24, 2008 at 1:46 PM, Sverre Hvammen Johansen
<hvammen@gmail.com> wrote:
> --ff now takes an argument allowing --ff to be written
> as --ff=allow and -no-ff to be written as --ff=never.
> This change allow other fast forward options to be
> introduced later.
>
> See the documentation for a further explanation of these options.
>
> Signed-off-by: Sverre Hvammen Johansen <hvammen@gmail.com>
> ---
> Documentation/fast-forward-options.txt | 44 ++++++++
> Documentation/git-merge.txt | 6 +-
> Documentation/git-pull.txt | 2 +
> Documentation/merge-options.txt | 9 +-
> git-merge.sh | 47 +++++---
> git-pull.sh | 4 +-
> t/t7601-merge-ff-options.sh | 188 ++++++++++++++++++++++++++++++++
> 7 files changed, 277 insertions(+), 23 deletions(-)
> create mode 100644 Documentation/fast-forward-options.txt
>
> diff --git a/Documentation/fast-forward-options.txt
> b/Documentation/fast-forward-options.txt
> new file mode 100644
> index 0000000..95d0e6f
> --- /dev/null
> +++ b/Documentation/fast-forward-options.txt
> @@ -0,0 +1,44 @@
> +FAST FORWARD OPTIONS
> +--------------------
> +
> +allow::
> +
> + Do not generate a merge commit if the merge resolves as a
> + fast-forward, only update the branch pointer. This option is
> + equivalent of '--ff' without any argument. This is the
> + default behavior.
only and never are both adverb,but allow is a verb. So how about using
auto instead of allow?
--
Ping Yin
next prev parent reply other threads:[~2008-05-02 12:34 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-24 5:46 [PATCH 2/5] Introduce -ff=<fast forward option> Sverre Hvammen Johansen
2008-05-02 12:33 ` Ping Yin [this message]
2008-05-03 22:04 ` Sverre Hvammen Johansen
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=46dff0320805020533u179e16cej2c4db3105252d0bd@mail.gmail.com \
--to=pkufranky@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=hvammen@gmail.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 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).