Subject: Re: [PATCH 19/19] gitweb: Remove creating directory for temporary files

From: Junio C Hamano <[email-masked]>
Date: Fri, 25 Aug 2006 19:05:56 -0700

"Marco Costalba" <[email-masked]> writes:

>> You can view new gitweb in work at
>>   http://front.fuw.edu.pl/cgi-bin/jnareb/gitweb.cgi
>
> Very nice job!
>
> Just a couple of suggestion regarding blame.
>
> - Instead of commit sha perhaps is more useful to show author name
> (linked to commit) and progressive number of revision.

Remember git history is not linear so progressive number does
not make much sense here.  I agree author and commit date would
be nice if they were readily available without cluttering.
A pop-up when hovered, perhaps?

> - Original code lines, ie. imported at the beginning and never
> modified, perhaps it is better to view without commit number, this
> could obfuscate the view and in any case is not an accurate info
> because the line was not modified during initial patch.

That holds only true for very young projects, or ones that were
perfect from the beginning and did not see much action ;-).