2008/6/3 Jakub Narebski <jnareb@xxxxxxxxx>: >>> I'd rather remove this, correct it, or make it optional (this is very >>> fork-heavy). >> >> Not sure how to do the same thing in pure Perl. > > I was thinking about extending git-blame porcelain format (and also > incremental format, of course) by 'parents' (and perhaps > 'original-parents') header... OK, I see. That would be nice. Also: currently taking "$full_rev^" directs the user to the parent commit, but it would be more user-friendly to point at the previous commit where the selected file was modified instead. >> We could however cache the results of git-rev-parse, since the same >> rev is likely to appear many times in the list. > > ...but starting with cache of git-rev-parse results, or optionally > allowing extended sha-1 syntax (including <hash>^) in hash* CGI > parameters in gitweb would be a good idea. > > But as I wrote, I'm fine with the patch as it is now. I've sent a new version (take 2) with caching. And comments, as Lea suggested :) -- 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
