From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 31 Jan 2008 02:48:43 GMT
Subject: Re: [PATCH] gitweb: Use config file or file for repository owner's name.
Message-ID: <7vsl0eg16c.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20080131023629.GB7778@c3sl.ufpr.br>

Bruno Cesar Ribas <ribas@c3sl.ufpr.br> writes:

> On Tue, Jan 29, 2008 at 10:16:16PM -0800, Junio C Hamano wrote:
>> Bruno Ribas <ribas@c3sl.ufpr.br> writes:
>> 
>> <snip> 
>> I am not sure about the effect of this change on a large scale
>> site.  If you do not have the project list file, originally we
>> just needed a stat per project, but now you open an extra file
>> (either "owner" or "config") and read it, once per every
>> project.
>
> Opening the extra file has same problem as the description file. And, as
> gitweb allow us to create  "description" and "cloneurl" file there is no 
> problem having another file to open instead finding out who is the owner of
> the directory asking to the filesystem.

We heard the same argument when cloneurl was added, and a
newcomer who does not know that may rightly use the same
argument.  But I think we should work towards _reducing_ the
number of such ad-hoc one-line-per-information files, not using
existing ones as an excuse to add _more_ of them.