Re: [PATCH] gitweb: Use config file or file for repository owner's name.
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 31, 2008, 02:48 UTC
- Message-ID
- <7vsl0eg16c.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <20080131023629.GB7778@c3sl.ufpr.br>
Bruno Cesar Ribas <ribas@c3sl.ufpr.br> writes:
Show 14 quoted lines
> 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.