diff options
author | Riyad Preukschas <riyad@informatik.uni-bremen.de> | 2013-02-01 23:56:03 +0400 |
---|---|---|
committer | Riyad Preukschas <riyad@informatik.uni-bremen.de> | 2013-02-01 23:56:03 +0400 |
commit | 003c3121049aef98bf37f048563b2ab892207620 (patch) | |
tree | 1cca26705d983ff55b2e2b97a4efd0d2954f9fcf /lib | |
parent | 0965aeb2ea3a7f89b0a74b746ec9aee063b0713c (diff) | |
parent | f4175219fb7a1cb93f7eed54bd6510a85345096e (diff) |
Merge pull request #2440 from klamontagne/master
Don't setuid the repositories on installation
Diffstat (limited to 'lib')
-rw-r--r-- | lib/tasks/gitlab/check.rake | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/tasks/gitlab/check.rake b/lib/tasks/gitlab/check.rake index 1ca723f271a..5e25d1b5d4b 100644 --- a/lib/tasks/gitlab/check.rake +++ b/lib/tasks/gitlab/check.rake @@ -716,7 +716,7 @@ namespace :gitlab do end def check_repo_base_permissions - print "Repo base access is drwsrws---? ... " + print "Repo base access is drwxrws---? ... " repo_base_path = Gitlab.config.gitolite.repos_path unless File.exists?(repo_base_path) @@ -724,12 +724,14 @@ namespace :gitlab do return end - if File.stat(repo_base_path).mode.to_s(8).ends_with?("6770") + if File.stat(repo_base_path).mode.to_s(8).ends_with?("2770") puts "yes".green else puts "no".red try_fixing_it( - "sudo chmod -R ug+rwXs,o-rwx #{repo_base_path}" + "sudo chmod -R ug+rwX,o-rwx #{repo_base_path}", + "sudo chmod -R u-s #{repo_base_path}", + "find #{repo_base_path} -type d -print0 | sudo xargs -0 chmod g+s" ) for_more_information( see_installation_guide_section "Gitolite" |