On Sun, Mar 8, 2015 at 11:03 AM, Kevin Daudt <m...@ikke.info> wrote:
> rev-list --bisect is used by git bisect, but never together with
> --first-parent. Because rev-list --bisect together with --first-parent
> is not handled currently, and even leads to segfaults, refuse to use
> both options together.
>
> Signed-off-by: Kevin Daudt <m...@ikke.info>
> Suggested-by: Junio C. Hamano <gits...@pobox.com>

It's customary for your sign-off to be last.

> ---
> diff --git a/Documentation/rev-list-options.txt 
> b/Documentation/rev-list-options.txt
> index 4ed8587..05c3f6d 100644
> --- a/Documentation/rev-list-options.txt
> +++ b/Documentation/rev-list-options.txt
> @@ -123,7 +123,8 @@ parents) and `--max-parents=-1` (negative numbers denote 
> no upper limit).
>         because merges into a topic branch tend to be only about
>         adjusting to updated upstream from time to time, and
>         this option allows you to ignore the individual commits
> -       brought in to your history by such a merge.
> +       brought in to your history by such a merge. Cannot be
> +       combined with --bisect.

A couple questions:

Should the documentation for ---bisect be updated to mention this
restriction also?

Should this change be protected by a "ifndef::git-rev-list[]" as are
all other mentions of "bisect" in rev-list-options.txt?

>  --not::
>         Reverses the meaning of the '{caret}' prefix (or lack thereof)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to