From 31f51b7e085ad8142aef238c1ec4acf4ceb12bc7 Mon Sep 17 00:00:00 2001 From: "Robin H. Johnson" Date: Sat, 23 Aug 2014 10:08:16 -0700 Subject: Only change umask after wildrepos check. Signed-off-by: Robin H. Johnson --- src/gitolite.pm | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gitolite.pm b/src/gitolite.pm index 487f265..bc18418 100644 --- a/src/gitolite.pm +++ b/src/gitolite.pm @@ -286,11 +286,12 @@ sub new_repo { my ($repo, $hooks_dir, $creator, $umask) = @_; + die "wildrepos disabled, can't set creator $creator on new repo $repo\n" + if $creator and not $GL_WILDREPOS; + my $oldumask = umask(); my $newumask = $umask ? $umask : get_repo_umask($repo); umask($newumask); - die "wildrepos disabled, can't set creator $creator on new repo $repo\n" - if $creator and not $GL_WILDREPOS; system("mkdir", "-p", "$repo.git") and die "$ABRT mkdir $repo.git failed: $!\n"; # erm, note that's "and die" not "or die" as is normal in perl -- cgit v1.2.3-65-gdbad