On Tue, Jan 29, 2008 at 10:16:16PM -0800, Junio C Hamano wrote: > Bruno Ribas <ribas@xxxxxxxxxxxx> 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. > > The project list page does that for every project, and it > actually is worse because it also needs to open yet another file > "description" from the directory. It almost makes me wonder if > are much better of to have a single file per project to read all > the necessary information off of, instead of having to open many > little files (currently it is only two---owner and description. > But who knows what other little pieces of information you would > want to add next week). wow, just me adds more little information?! 8-P (I have my own git.acl which is the "simple" way to my SSH scripts control acl via public_key, but that's another story). Having a single file is already done, having the config file but parser may be bad too (or not for common case), we can have a "gitweb.config" which contains all those necessary information like owner, description, cloneurl ... Another "option" is to make cache of then project list page =) From time to time (each push ?!) gitweb creates project list page again and generates a new project_list file to have easy access when entering summary page, or make summary page cached too -- Bruno Ribas - ribas@xxxxxxxxxxxx http://web.inf.ufpr.br/ribas C3SL: http://www.c3sl.ufpr.br - 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
