Jens Lehmann wrote: > In commit 0656781fa "git mv" learned to update the submodule path in the > .gitmodules file when moving a submodule in the work tree. But since that > commit update_path_in_gitmodules() gets called no matter if we moved a > submodule or a regular file, which is wrong and leads to a bogus warning > when moving a regular file in a repo containing a .gitmodules file: > > warning: Could not find section in .gitmodules where path=<filename> [...] > Reported-by: Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> > Signed-off-by: Jens Lehmann <Jens.Lehmann@xxxxxx> > --- [...] > And this is the fix for it, which I believe is stuff for maint. Thanks again, and sorry to leave this hanging. I had some vague ideas for improvement: * style nits: test is a little long, making it hard to take in at a glance; tests tend to try to avoid pipelines to avoid losing the exit code from git commands; usual style is to use "test" instead of "[" consistently * would be nice to have another test that makes sure the "move a file, not submodule" case keeps working But those can easily happen as changes on top, and as you mentioned, it's probably worth someone interested (e.g., me :)) making a pass to clean up the style of the rest of the script anyway. The patch as-is is obviously good. Thanks again for the quick diagnosis and fix. Reviewed-by: Jonathan Nieder <jrnieder@xxxxxxxxx> -- 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