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) > if (revs.bisect) > bisect_list = 1; > > + if(revs.first_parent_only && revs.bisect) I should have added a space after the if. > + die(_("--first-parent is incompattible with --bisect")); > + > if (DIFF_OPT_TST(&revs.diffopt, QUICK)) > info.flags |= REV_LIST_QUIET; > for (i = 1 ; i < argc; i++) { -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html