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

Replace all stat command line calls with ruby equivalents

Browse source
This commit is contained in:
Riyad Preukschas 2013-01-12 00:44:40 +01:00
parent f73dd4af5e
commit 147dfd1656
2 changed files with 17 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
lib/tasks/gitlab/check.rake
View file

@ -502,7 +502,7 @@ namespace :gitlab do
return return
end end
if `stat --printf %a #{gitolite_config_path}` == "750" if File.stat(gitolite_config_path).mode.to_s(8).ends_with?("750")
puts "yes".green puts "yes".green
else else
puts "no".red puts "no".red
@ -526,12 +526,11 @@ namespace :gitlab do
return return
end end
if `stat --printf %U #{gitolite_config_path}` == gitolite_ssh_user && # user if File.stat(gitolite_config_path).uid == uid_for(gitolite_ssh_user) &&
`stat --printf %G #{gitolite_config_path}` == gitolite_ssh_user #group File.stat(gitolite_config_path).gid == gid_for(gitolite_ssh_user)
puts "yes".green puts "yes".green
else else
puts "no".red puts "no".red
puts "#{gitolite_config_path} is not owned by #{gitolite_ssh_user}".red
try_fixing_it( try_fixing_it(
"sudo chown -R #{gitolite_ssh_user}:#{gitolite_ssh_user} #{gitolite_config_path}" "sudo chown -R #{gitolite_ssh_user}:#{gitolite_ssh_user} #{gitolite_config_path}"
) )
@ -722,7 +721,7 @@ namespace :gitlab do
return return
end end
if `stat --printf %a #{repo_base_path}` == "6770" if File.stat(repo_base_path).mode.to_s(8).ends_with?("6770")
puts "yes".green puts "yes".green
else else
puts "no".red puts "no".red
@ -746,12 +745,11 @@ namespace :gitlab do
return return
end end
if `stat --printf %U #{repo_base_path}` == gitolite_ssh_user && # user if File.stat(repo_base_path).uid == uid_for(gitolite_ssh_user) &&
`stat --printf %G #{repo_base_path}` == gitolite_ssh_user #group File.stat(repo_base_path).gid == gid_for(gitolite_ssh_user)
puts "yes".green puts "yes".green
else else
puts "no".red puts "no".red
puts "#{repo_base_path} is not owned by #{gitolite_ssh_user}".red
try_fixing_it( try_fixing_it(
"sudo chown -R #{gitolite_ssh_user}:#{gitolite_ssh_user} #{repo_base_path}" "sudo chown -R #{gitolite_ssh_user}:#{gitolite_ssh_user} #{repo_base_path}"
) )
@ -832,7 +830,8 @@ namespace :gitlab do
next next
end end
if run_and_match("stat --format %N #{project_hook_file}", /#{hook_file}.+->.+#{gitolite_hook_file}/) if File.lstat(project_hook_file).symlink? &&
File.realpath(project_hook_file) == File.realpath(gitolite_hook_file)
puts "ok".green puts "ok".green
else else
puts "not a link to Gitolite's hook".red puts "not a link to Gitolite's hook".red

9
lib/tasks/gitlab/task_helpers.rake
View file

@ -45,6 +45,15 @@ namespace :gitlab do
end end
end end
def uid_for(user_name)
run("id -u #{user_name}").chomp.to_i
end
def gid_for(group_name)
group_line = File.read("/etc/group").lines.select{|l| l.start_with?("#{group_name}:")}.first
group_line.split(":")[2].to_i
end
def warn_user_is_not_gitlab def warn_user_is_not_gitlab
unless @warned_user_not_gitlab unless @warned_user_not_gitlab
current_user = run("whoami").chomp current_user = run("whoami").chomp