diff options
| author | Alex Wang | 2016-09-13 12:36:56 -0400 |
|---|---|---|
| committer | Alex Wang | 2016-09-13 12:36:56 -0400 |
| commit | 71063aa2e39ed12a6e42ebd7e66d477b3baa02c8 (patch) | |
| tree | ed356a2116378d6b9073bd9c48b92a5d2f0408b1 | |
| parent | b505ee3fedc7309a4c1ba43ea304d7a0d3b70d80 (diff) | |
| download | brew-71063aa2e39ed12a6e42ebd7e66d477b3baa02c8.tar.bz2 | |
Make scons use stdenv only if immediate build dep
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.
| -rw-r--r-- | Library/Homebrew/formula_installer.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index 68c02aee3..cb62e6206 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -562,7 +562,7 @@ class FormulaInstaller if 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" end |