Johan Herland <johan@xxxxxxxxxxx> writes: >> - $sub1sha1 sub1 (${sub1sha1:0:7}) >> - $sub2sha1 sub2 (${sub1sha1:0:7}) >> + $sub1sha1 sub1 ($(echo $sub1sha1 | cut -c 1-7)) >> + $sub2sha1 sub2 ($(echo $sub1sha1 | cut -c 1-7)) > > Typo (both in the original, and the patch), should be: > $sub2sha1 sub2 ($(echo $sub2sha1 | cut -c 1-7)) > >> $sub3sha1 sub3 (heads/master) >> EOF > > Otherwise: > > Acked-by: Johan Herland <johan@xxxxxxxxxxx> Hmm, what does the command use to shorten these object names? It may be safer and more correct to use "rev-parse --short" in case these object names were ambigous in their first 7 hexdigits. -- 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