diff options
| author | Mike McQuaid | 2016-09-15 09:20:39 +0100 |
|---|---|---|
| committer | Mike McQuaid | 2016-09-15 09:20:39 +0100 |
| commit | 842d6ce8bdc2296869c41c5866bbf5f046ac6eb8 (patch) | |
| tree | c587130ef82ba07e9896afd7a56fb8e2c6d3687b /Library | |
| parent | 099295163412b19470c500406738c50c2fb2fc8f (diff) | |
| download | brew-842d6ce8bdc2296869c41c5866bbf5f046ac6eb8.tar.bz2 | |
keg: add ALL_TOP_LEVEL_DIRECTORIES constant.
This is used in diagnostic (and will be used in other places later)
rather than hard-coding other directories later.
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/diagnostic.rb | 3 | ||||
| -rw-r--r-- | Library/Homebrew/keg.rb | 1 |
2 files changed, 2 insertions, 2 deletions
diff --git a/Library/Homebrew/diagnostic.rb b/Library/Homebrew/diagnostic.rb index 1b3018251..7cc6371bd 100644 --- a/Library/Homebrew/diagnostic.rb +++ b/Library/Homebrew/diagnostic.rb @@ -308,8 +308,7 @@ module Homebrew def check_access_prefix_directories not_writable_dirs = [] - extra_dirs = ["lib/pkgconfig", "share/locale", "share/man", "opt"] - (Keg::TOP_LEVEL_DIRECTORIES + extra_dirs).each do |dir| + Keg::ALL_TOP_LEVEL_DIRECTORIES.each do |dir| path = HOMEBREW_PREFIX/dir next unless path.exist? next if path.writable_real? diff --git a/Library/Homebrew/keg.rb b/Library/Homebrew/keg.rb index bc4b9ed6e..d40994368 100644 --- a/Library/Homebrew/keg.rb +++ b/Library/Homebrew/keg.rb @@ -65,6 +65,7 @@ class Keg LOCALEDIR_RX = /(locale|man)\/([a-z]{2}|C|POSIX)(_[A-Z]{2})?(\.[a-zA-Z\-0-9]+(@.+)?)?/ INFOFILE_RX = %r{info/([^.].*?\.info|dir)$} TOP_LEVEL_DIRECTORIES = %w[bin etc include lib sbin share var Frameworks] + ALL_TOP_LEVEL_DIRECTORIES = (TOP_LEVEL_DIRECTORIES + %w[lib/pkgconfig share/locale share/man opt]).freeze PRUNEABLE_DIRECTORIES = %w[bin etc include lib sbin share Frameworks LinkedKegs].map do |d| case d when "LinkedKegs" then HOMEBREW_LIBRARY/d else HOMEBREW_PREFIX/d end end |