On Wed, Aug 17, 2016 at 5:17 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Stefan Beller <sbeller@xxxxxxxxxx> writes: > >> and now with error handling of invalid options. > > Thanks. Well this was not the end of the story. I sent that version out, before doing a final test run, because changing a few lines is trivial right? :( I'll resend with: diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c index a1da3ea..a366757 100644 --- a/builtin/submodule--helper.c +++ b/builtin/submodule--helper.c @@ -562,7 +562,7 @@ static void prepare_possible_alternates(const char *sm_name, if (!strcmp(sm_alternate, "superproject")) foreach_alt_odb(add_possible_reference_from_superproject, &sas); - else if (!strcmp(sm_alternate, "no") + else if (!strcmp(sm_alternate, "no")) ; /* do nothing */ else die(_("Value '%s' for submodule.alternateLocation is not recognized"), sm_alternate); -- 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