aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Formula/git.rb
diff options
context:
space:
mode:
authorJack Nagel2012-08-15 18:52:25 -0500
committerJack Nagel2012-08-15 22:39:52 -0500
commit97d3c12c7860060a68a85615c91b960d98628d1e (patch)
tree527b91fa44ec3812d2e172a3db2ff6f5ae467318 /Library/Formula/git.rb
parent8458b4092bba013853c088bb8cad9d4aa8dd02b0 (diff)
downloadhomebrew-97d3c12c7860060a68a85615c91b960d98628d1e.tar.bz2
FormulaInstaller: filter ARGV when testing for installed deps
In FormulaInstaller#install, needed_deps is created without first filtering ARGV for flags like --HEAD. In practice, this means that sometimes needed_deps will contain dependencies that are actually already installed; --HEAD causes Formula#installed_prefix to differ and thus checking Formula#installed? will result in false negatives. This can trigger weird bugs; for example, #10380, where the "Installing foo" header is displayed even though no dependencies were previously installed. Fix this by filtering ARGV before testing for installed dependencies, and do the same for requirements to maintain symmetry, and because some requiremnts check Formula#installed? as well. Fixes #10380. Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Diffstat (limited to 'Library/Formula/git.rb')
0 files changed, 0 insertions, 0 deletions