On Mon, Feb 4, 2019 at 1:47 PM Junio C Hamano <gitster@xxxxxxxxx> wrote: > Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > > On Sun, Feb 3, 2019 at 4:01 PM Sven van Haastregt <svenvh@xxxxxxxxx> wrote: > >> We cannot always rely on successfully invoking `git rev-parse` in the > >> submodule directory. Keep the old method using `cut` as a fallback. > > > > Reviewers and future readers of this patch are left in the dark > > regarding the circumstances in which git-rev-parse may fail in the > > submodule directory. It would be helpful for the commit message to > > explain this potential failure in enough detail for someone working in > > this area in the future to understand any implications of changes to > > this code. > > Replace that problematic sentence with something like > > As a submodule may not be initialized with "submodule init" > or not cloned, `git rev-parse` may not work in it yet; as a > fallback, use the original method of cutting at 7 hexdigits. > > perhaps? That's better.