diff options
| author | Jack Nagel | 2011-10-05 23:50:29 -0500 |
|---|---|---|
| committer | Jack Nagel | 2011-10-06 00:11:34 -0500 |
| commit | 74207f71e61a59be797f6139db5bcc8e73f7310c (patch) | |
| tree | e1c0a9e588eba328e9078095e1d8968569cdc8b5 /Library | |
| parent | e7aaeda953e84fdc1965fb27cc0af579224ed3a5 (diff) | |
| download | brew-74207f71e61a59be797f6139db5bcc8e73f7310c.tar.bz2 | |
cleaner: correctly obey HOMEBREW_KEEP_INFO
The advertised default is that anything in share/info is removed unless
the user sets HOMEBREW_KEEP_INFO, but we've actually been installing the
files *unless* the variable is set. To illustrate:
$ unset HOMEBREW_KEEP_INFO
$ brew install -v wdiff
[...]
ln /usr/local/share/locale/af/LC_MESSAGES/wdiff-gnulib.mo
ln /usr/local/share/info/wdiff.info
ln /usr/local/share/info/dir
==> Summary
[...]
$ HOMEBREW_KEEP_INFO=1 brew install -v wdiff
ln /usr/local/share/locale/af/LC_MESSAGES/wdiff-gnulib.mo
==> Summary
[...]
Obviously not what we wanted.
Closes Homebrew/homebrew#7989.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/cleaner.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Library/Homebrew/cleaner.rb b/Library/Homebrew/cleaner.rb index 36bb74f4b..426fbec6a 100644 --- a/Library/Homebrew/cleaner.rb +++ b/Library/Homebrew/cleaner.rb @@ -3,7 +3,7 @@ class Cleaner @f = Formula.factory f [f.bin, f.sbin, f.lib].select{ |d| d.exist? }.each{ |d| clean_dir d } - unless ENV['HOMEBREW_KEEP_INFO'].nil? + unless ENV['HOMEBREW_KEEP_INFO'] f.info.rmtree if f.info.directory? and not f.skip_clean? f.info end |