diff options
| author | Charlie Sharpsteen | 2011-04-01 12:30:02 -0700 |
|---|---|---|
| committer | Adam Vandenberg | 2011-04-01 13:00:07 -0700 |
| commit | 03c8d6bbf12f632e93dff0a0cd1a254b4da5560a (patch) | |
| tree | cc3aa52403c1c4d56e580fca52f651490c565767 /Library | |
| parent | 846cc0cba26383f4f92c50577149c28ea81cb43c (diff) | |
| download | homebrew-03c8d6bbf12f632e93dff0a0cd1a254b4da5560a.tar.bz2 | |
Fix brew-audit Fortran check
The Fortran regex only allowed for trailing whitespace after:
depends_on 'gfortran'
This missed conditional dependencies, such as:
depends_on 'gfortran' if something?
Such a dependency was present in the NetCDF formula.
Signed-off-by: Adam Vandenberg <flangy@gmail.com>
Diffstat (limited to 'Library')
| -rwxr-xr-x | Library/Homebrew/cmd/audit.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb index 7df226109..b4cc01bf0 100755 --- a/Library/Homebrew/cmd/audit.rb +++ b/Library/Homebrew/cmd/audit.rb @@ -107,7 +107,7 @@ def audit_formula_text name, text end if strict? # Formula depends_on gfortran - if text =~ /\s*depends_on\s*(\'|\")gfortran(\'|\")\s*$/ + if text =~ /^\s*depends_on\s*(\'|\")gfortran(\'|\").*/ problems << " * Use ENV.fortran during install instead of depends_on 'gfortran'" end unless name == "gfortran" # Gfortran itself has this text in the caveats |