diff options
| author | Jack Nagel | 2011-09-28 14:28:14 -0500 |
|---|---|---|
| committer | Jack Nagel | 2011-09-28 14:42:04 -0500 |
| commit | be9d91b2b80fdfe82720b2e2d3ceec378e33b371 (patch) | |
| tree | cd5eeb8fcaf53bdb7dbed04cc1d2fa52c3e6c30d /Library/Homebrew/cmd | |
| parent | 9bce115e9a0f2c4b5759073e03ef8f6d4f749850 (diff) | |
| download | brew-be9d91b2b80fdfe82720b2e2d3ceec378e33b371.tar.bz2 | |
audit: match #{prefix}/libexec and prefix+'libexec'
These were being picked up as just "#{prefix}/lib" and prefix+'lib' due
to the eagerness of the regex. Reordering "lib" and "libexec" will take
care of it; I couldn't think of a more clever solution.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Diffstat (limited to 'Library/Homebrew/cmd')
| -rwxr-xr-x | Library/Homebrew/cmd/audit.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb index 5f4796a6e..cfa6879a0 100755 --- a/Library/Homebrew/cmd/audit.rb +++ b/Library/Homebrew/cmd/audit.rb @@ -50,7 +50,7 @@ def audit_formula_text name, text end # Prefer formula path shortcuts in Pathname+ - if text =~ %r{\(\s*(prefix\s*\+\s*(['"])(bin|include|lib|libexec|sbin|share))} + if text =~ %r{\(\s*(prefix\s*\+\s*(['"])(bin|include|libexec|lib|sbin|share))} problems << " * \"(#{$1}...#{$2})\" should be \"(#{$3}+...)\"" end @@ -59,7 +59,7 @@ def audit_formula_text name, text end # Prefer formula path shortcuts in strings - if text =~ %r[(\#\{prefix\}/(bin|include|lib|libexec|sbin|share))] + if text =~ %r[(\#\{prefix\}/(bin|include|libexec|lib|sbin|share))] problems << " * \"#{$1}\" should be \"\#{#{$2}}\"" end |