Jens Lehmann <Jens.Lehmann@xxxxxx> writes: > diff --git a/submodule.c b/submodule.c > index b477c3c..ddb0a29 100644 > --- a/submodule.c > +++ b/submodule.c > @@ -263,6 +263,13 @@ void set_config_fetch_recurse_submodules(int value) > config_fetch_recurse_submodules = value; > } > > +static int is_submodule_commit_present(const char *path, unsigned char sha1[20]) > +{ > + if (!add_submodule_odb(path)) > + return lookup_commit_reference(sha1) != 0; > + return 0; > +} Don't you need to prove the usual "reachabile from the refs" here, instead of just the presense of a commit object? If not, why not? -- 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