Re: [PATCH] rev-list: refuse --first-parent combined with --bisect

Kevin Daudt writes: > On Sat, Mar 07, 2015 at 10:31:16PM +0100, Kevin Daudt wrote: >> diff --git a/builtin/rev-list.c b/builtin/rev-list.c >> index ff84a82..c271e15 100644 >> --- a/builtin/rev-list.c >> +++ b/builtin/rev-list.c >> @@ -291,6 +291,9 @@ int cmd_rev_list(int argc, const char **argv,

Re: [PATCH] rev-list: refuse --first-parent combined with --bisect

On Sat, Mar 07, 2015 at 10:31:16PM +0100, Kevin Daudt wrote: > diff --git a/builtin/rev-list.c b/builtin/rev-list.c > index ff84a82..c271e15 100644 > --- a/builtin/rev-list.c > +++ b/builtin/rev-list.c > @@ -291,6 +291,9 @@ int cmd_rev_list(int argc, const char **argv, const char > *prefix) >

[PATCH] rev-list: refuse --first-parent combined with --bisect

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 --- This is my first code patch, and thoug