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

formula_installer: fix --default-fortran-flags

Browse Source 
Setting the `FC` environment variable allows the user to override the
default Fortran compiler. The `SharedEnvExtension#fortran` method then
checks whether any of `FCFLAGS` and `FFLAGS` environment variables are
set and, if not, suggests to use `--default-fortran-flags` when running
`brew install`. However, `--default-fortran-flags` never makes it to
`build.rb` (that is executed by `FormulaInstaller`) because the flag is
not whitelisted in `FormulaInstaller#sanitized_ARGV_options`.

This change rectifies this and thereby eliminates the confusing message.

Fixes Homebrew/homebrew#47852.
This commit is contained in:
Martin Afanasjew 2016-01-10 02:41:00 +01:00
parent bcedfe64e8
commit 08fb36252c
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Library/Homebrew/formula_installer.rb
View File

@ -503,6 +503,7 @@ class FormulaInstaller
args << "--verbose" if verbose? args << "--verbose" if verbose?
args << "--debug" if debug? args << "--debug" if debug?
args << "--cc=#{ARGV.cc}" if ARGV.cc args << "--cc=#{ARGV.cc}" if ARGV.cc
args << "--default-fortran-flags" if ARGV.include? "--default-fortran-flags"
if ARGV.env if ARGV.env
args << "--env=#{ARGV.env}" args << "--env=#{ARGV.env}"