diff options
| author | Charlie Sharpsteen | 2011-04-01 12:30:02 -0700 |
|---|---|---|
| committer | Adam Vandenberg | 2011-04-01 13:00:07 -0700 |
| commit | 7776f7ee117eb3aa8a3b2a8eed70d96f66f9bba1 (patch) | |
| tree | 488783813fc2ed273cf3ef91459f83bfa794bf07 /Library/Homebrew/cmd | |
| parent | 78ace67b3fce405bdfe5384b61e471e94b698c9c (diff) | |
| download | brew-7776f7ee117eb3aa8a3b2a8eed70d96f66f9bba1.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/Homebrew/cmd')
| -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 |