Re: [PATCH] Added sub get_owner_file which checks if there's a file with project owner name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Nagy Balázs wrote:
> Jakub Narebski wrote:

>> But I think if IO matters it is better to generate projects list; you
>> can even use gitweb for that, or you can simply add a line with URL
>> escaped project name (project path) relative to $projectroot, separated
>> by space from the URL escaped (URI-encoded) project owner.  See also
>> "Gitweb repositories" section in gitweb/INSTALL.  Adding projects is
>> rare event.
>>   
> Are you talking about I/O of an all-in CGI script?  

I am talking there between I/O difference between situation
(configuration) when $projects_list is a directory (default),
or is a file. If $projects_list is a directory, gitweb scans
directory structure to find git repositories, which for large
number of repositories might take time, even with filesystem
cache, and with depth of searching bound by $project_maxdepth.
Add to that finding symbolic name of the owner of repository
directory, or (with the patch) reading a file per repo with repo
owner.

Reading and pasing single text file avoids this; it is faster.
But for small repos it is easier to scan directory, and difference
in performance is not much.

> We can tune the  
> performance of this script, but changing the GIT_DIR structure just 
> because of a simple script is a bit overkill to me.
> 
> What if this script creates the $projects_list file, for example when 
> the $projectroot's mtime changes?  We can even hold mtime info for every 
> project's config file.

I don't understand what you wanted to say here. $projects_list file
lists only project path (project name) and project owner.

-- 
Jakub Narebski
Poland
-
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux