diff options
| author | Adam Vandenberg | 2010-07-20 08:49:42 -0700 |
|---|---|---|
| committer | Adam Vandenberg | 2010-07-20 08:58:37 -0700 |
| commit | 070421f4413e32d14252275c4169bb7daeff6806 (patch) | |
| tree | 9466bc620d4b923133302b6fb2d02cb4fc704811 /Library | |
| parent | 14f0556d1ca2b67301aee221d3a5919fca60143e (diff) | |
| download | brew-070421f4413e32d14252275c4169bb7daeff6806.tar.bz2 | |
Fix top-level formula having duplicate deps
If the top-level formula being installed has any keg-only
dependencies, the lib/include/bin/pkg-config paths for those
deps will be added twice to the ENV vars.
This doesn't break anything, but does make debug output somewhat longer
than it needs to be.
Tidy this up by calling uniq on deps before iterating.
(The cause of this duplication is related to how top-level formula are
run, to support keeping patches in __END__ blocks.)
Fixes Homebrew/homebrew#1110
Diffstat (limited to 'Library')
| -rwxr-xr-x | Library/Homebrew/install.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Library/Homebrew/install.rb b/Library/Homebrew/install.rb index ebd50d588..3f643e407 100755 --- a/Library/Homebrew/install.rb +++ b/Library/Homebrew/install.rb @@ -54,7 +54,7 @@ end def install f show_summary_heading = false - f.deps.each do |dep| + f.deps.uniq.each do |dep| dep = Formula.factory dep if dep.keg_only? ENV.prepend 'LDFLAGS', "-L#{dep.lib}" |