admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Only error out for sudo brew install

Browse Source 
Note this only happens if Homebrew is *not* installed sudo. Which is most of the time true.

Fixes Homebrew/homebrew#6899
This commit is contained in:
Max Howell 2011-08-17 12:30:21 +01:00
parent 32c746ef45
commit df113d3ce6
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Library/Homebrew/cmd/install.rb
View File

@ -9,6 +9,12 @@ module Homebrew extend self
raise "No available formula for #{name}\n#{msg}" if msg raise "No available formula for #{name}\n#{msg}" if msg
end unless ARGV.force? end unless ARGV.force?
if Process.uid.zero? and not File.stat(HOMEBREW_BREW_FILE).uid.zero?
# note we only abort if Homebrew is *not* installed as sudo and the user
# calls brew as root. The fix is to chown brew to root.
abort "Cowardly refusing to `sudo brew install'"
end
install_formulae ARGV.formulae install_formulae ARGV.formulae
end end

5
bin/brew
View File

@ -26,11 +26,6 @@ when '-v'
end end
end end
if Process.uid.zero? and not File.stat(HOMEBREW_BREW_FILE).uid.zero?
# note we only abort if Homebrew is *not* installed as sudo and the user
# calls brew as root. The fix is to chown brew to root.
abort "Cowardly refusing to `sudo brew'"
end
case HOMEBREW_PREFIX.to_s when '/', '/usr' case HOMEBREW_PREFIX.to_s when '/', '/usr'
# it may work, but I only see pain this route and don't want to support it # it may work, but I only see pain this route and don't want to support it
abort "Cowardly refusing to continue at this prefix: #{HOMEBREW_PREFIX}" abort "Cowardly refusing to continue at this prefix: #{HOMEBREW_PREFIX}"