On Tuesday 24 August 2010, Jean Delvare wrote: > On Sat, 21 Aug 2010 12:10:22 +0200, Johan Herland wrote: > > To me, it seems what you want to do is convert the commit ID in > > front of every blame-line into the result of running 'git name-rev' > > (or 'git describe') on that line. > > > > To that effect something like this should work: > > > > git blame <file> | > > while read sha1 rest > > do > > tag=$(git name-rev --tags --name-only $sha1) && > > echo "$tag $rest" > > done > > > > Of course, if you're doing this at a bigger scale, you want to wrap > > this in a script that (1) caches commitID -> tag mappings, and that > > (2) runs 'git name-rev in its --stdin mode'. > > Thanks for the very valuable suggestion. Obviously, the fact that the > above command took over an hour to complete on a 1000-line file as > kind of an issue ;) I did suspect this performance issue originally, > which is why I thought it would be better if git itself would > implement the feature. Yes, everything would be easier and faster if git itself implemented it. Aka. infinite feature creep... This obviously does not scale. Instead, Git tries to provide the building blocks that you can string together to produce the exact result you're looking for. > That being said... git name-rev's --stdin > option seems to be doing all the hard caching work already: > > git blame -l <file> | git name-rev --tags --name-only --stdin > > does almost what I want, and is reasonably fast (13 seconds for the > same file.) I did the same experiment on a much larger file (~19000 lines), and got: 27.94s git blame -l <file> 28.31s git blame -l <file> | git name-rev --tags --name-only --stdin which suggests that the git name-rev process only adds ~1% to the total runtime. > I'll need to do some reformatting work to extract the tag > from the symbolic names (which in turn should almost fix the > alignment), ...only if all the tag names happen to have the same length. > but this is nothing a few lines of shell scripting can't do. I don't see why you need to remove the suffix from the tag name. You're simply removing information that could be used to look up the exact commit resposible for each line. And it's not like the tag name is completely unreadable unless you remove the suffix... ...Johan -- Johan Herland, <johan@xxxxxxxxxxx> www.herland.net -- 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