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

Make scons use stdenv only if immediate build dep

Browse Source 
formula_installer will use stdenv if scons is anywhere in the formula's
recursive dependency list:

    https://github.com/Homebrew/legacy-homebrew/issues/40401#issuecomment-110066355

Having scons as a dependency should only require stdenv if it is an
immediate build dependency, as otherwise scons shouldn't be invoked.
This commit is contained in:
Alex Wang 2016-09-13 12:36:56 -04:00
parent b505ee3fed
commit 71063aa2e3
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Library/Homebrew/formula_installer.rb
View File

@ -562,7 +562,7 @@ class FormulaInstaller
if ARGV.env if ARGV.env
args << "--env=#{ARGV.env}" args << "--env=#{ARGV.env}"
elsif formula.env.std? || formula.recursive_dependencies.any? { |d| d.name == "scons" } elsif formula.env.std? || formula.deps.select(&:build?).any? { |d| d.name == "scons" }
args << "--env=std" args << "--env=std"
end end