Hi everyone, >> >> @@ -958,6 +946,8 @@ int cmd_grep(int argc, const char **argv, const char *prefix) >> /* die the same way as if we did it at the beginning */ >> setup_git_directory(); >> } >> + if (!use_index) >> + recurse_submodules = 0; /* ignore */ >> >> /* >> * skip a -- separator; we know it cannot be > > Yeah, this seems more meaningful, IMHO, as we can easily see that the > recurse_submodules option was dropped in favor of using --no-index. > I agree. I’ll send a v2 of my patch with this added. Philippe.