diff options
| author | Jack Nagel | 2013-04-06 22:11:26 -0500 |
|---|---|---|
| committer | Jack Nagel | 2013-04-06 22:12:21 -0500 |
| commit | 8bd619e03b17f603027b0724e3d018dd241b6d23 (patch) | |
| tree | 100e83275c5b769a603078fc41793bf51ebef0f2 /Library | |
| parent | 4a8653ba7d226080018c6f3640b4eced38a338b8 (diff) | |
| download | brew-8bd619e03b17f603027b0724e3d018dd241b6d23.tar.bz2 | |
audit: clean up some regexps
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/cmd/audit.rb | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb index 296e79cee..8b773345d 100644 --- a/Library/Homebrew/cmd/audit.rb +++ b/Library/Homebrew/cmd/audit.rb @@ -430,16 +430,17 @@ class FormulaAuditor problem "Use MacOS.version instead of MACOS_VERSION" end - if text =~ /(MacOS.((snow_)?leopard|leopard|(mountain_)?lion)\?)/ - problem "#{$1} is deprecated, use a comparison to MacOS.version instead" + cats = %w{leopard snow_leopard lion mountain_lion}.join("|") + if text =~ /MacOS\.(?:#{cats})\?/ + problem "\"#{$&}\" is deprecated, use a comparison to MacOS.version instead" end if text =~ /skip_clean\s+:all/ problem "`skip_clean :all` is deprecated; brew no longer strips symbols" end - if text =~ /depends_on (.*)\.new$/ - problem "`depends_on` can take requirement classes directly" + if text =~ /depends_on [A-Z][\w:]+\.new$/ + problem "`depends_on` can take requirement classes instead of instances" end end |
