diff options
| author | Jack Nagel | 2011-10-05 23:50:29 -0500 |
|---|---|---|
| committer | Jack Nagel | 2011-10-06 00:11:34 -0500 |
| commit | df2017c06b8dc5a4c863ebf7d1381484a8fd2f30 (patch) | |
| tree | d4166c1df63fc8409897992823e72c148e13f4e5 | |
| parent | 8dfaafeb9e41e3a1142cb80d4279b7999864652b (diff) | |
| download | homebrew-df2017c06b8dc5a4c863ebf7d1381484a8fd2f30.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 #7989.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
| -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 |