deac/gitolite
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

a bit of optimisation to help another one along

Browse source 
(thanks to reuss for telling me about the problem, although setting
BIG_CONFIG worked just as well in his case, and this patch was not
really needed)
This commit is contained in:
Sitaram Chamarty 2010-08-25 23:10:31 +05:30
parent 8808158bd8
commit 1d566ac46b
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/gl-compile-conf
View file

@ -512,13 +512,18 @@ my %projlist = ();
# for each real repo (and remember this will be empty, thus skipping all this, # for each real repo (and remember this will be empty, thus skipping all this,
# if $GL_NO_DAEMON_NO_GITWEB is on!) # if $GL_NO_DAEMON_NO_GITWEB is on!)
# note: we do them in 2 separate loops to avoid breaking the optimisation in
# sub parse_acl (look for variable $saved_crwu)
for my $repo (keys %repo_patts) { for my $repo (keys %repo_patts) {
wrap_chdir("$ENV{GL_REPO_BASE_ABS}/$repo.git"); wrap_chdir("$ENV{GL_REPO_BASE_ABS}/$repo.git");
# daemon is easy # daemon is easy
&setup_daemon_access($repo); &setup_daemon_access($repo);
}
for my $repo (keys %repo_patts) {
wrap_chdir("$ENV{GL_REPO_BASE_ABS}/$repo.git");
# gitweb is a little more complicated. Here're some notes: # gitweb is a little more complicated. Here're some notes:
# - "setup_gitweb_access" also sets "owner", despite the name # - "setup_gitweb_access" also sets "owner", despite the name
# - specifying a description also counts as enabling gitweb # - specifying a description also counts as enabling gitweb
@ -535,7 +540,7 @@ for my $repo (keys %repo_patts) {
# implementation note: this must happen *after* one of the previous 2 # implementation note: this must happen *after* one of the previous 2
# calls (setup daemon or gitweb). The reason is that they call # calls (setup daemon or gitweb). The reason is that they call
# "can_read", which eventually calls parse_acl with the right "creator" # "can_read", which eventually calls parse_acl with the right "creator"
# set for the *current* repo, which in turn stores translated values fr # set for the *current* repo, which in turn stores translated values for
# $creator in the repo_config hash, which, (phew!) is needed for a match # $creator in the repo_config hash, which, (phew!) is needed for a match
# that eventually gets you a valid $repo_config{} below # that eventually gets you a valid $repo_config{} below
&setup_repo_configs($repo, \%repo_config) if $repo_config{$repo}; &setup_repo_configs($repo, \%repo_config) if $repo_config{$repo};