diff options
| author | Mike McQuaid | 2016-12-18 11:17:44 -0800 |
|---|---|---|
| committer | GitHub | 2016-12-18 11:17:44 -0800 |
| commit | e95c843c2e90eec35a1d8566c500aa4a91c3d3b3 (patch) | |
| tree | af94facb11c29b3efb28e8bb596e450b72cfb8ac /Library/Homebrew/dev-cmd | |
| parent | b87d27f939c6ed6d93bd80dc78240e25eb65ed49 (diff) | |
| parent | cf1fa0c5bc5bf80d21921e5187581de1a7baa08b (diff) | |
| download | brew-e95c843c2e90eec35a1d8566c500aa4a91c3d3b3.tar.bz2 | |
Merge pull request #1673 from MikeMcQuaid/audit-fails-with-llvm
audit: flag use of "fails_with :llvm".
Diffstat (limited to 'Library/Homebrew/dev-cmd')
| -rw-r--r-- | Library/Homebrew/dev-cmd/audit.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/Library/Homebrew/dev-cmd/audit.rb b/Library/Homebrew/dev-cmd/audit.rb index ee03c01ac..0112c524f 100644 --- a/Library/Homebrew/dev-cmd/audit.rb +++ b/Library/Homebrew/dev-cmd/audit.rb @@ -1000,6 +1000,10 @@ class FormulaAuditor problem "Use Language::Node for npm install args" end + if line.include?("fails_with :llvm") + problem "'fails_with :llvm' is now a no-op so should be removed" + end + return unless @strict if line =~ /system ((["'])[^"' ]*(?:\s[^"' ]*)+\2)/ |