| Age | Commit message (Collapse) | Author |
|---|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
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, Homebrew/homebrew#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 Homebrew/homebrew#10380.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Currently FormulaInstaller "forgets" about the requirements of
dependencies, as dependencies are recursively expanded by the first
FormulaInstaller instance, and then ignored by subsequent instances to
prevent duplicate installation attempts. These requirements then have
their messages displayed, but the fatal check is skipped and thus no
exception is raised.
Now that we have Formula#recursive_requirements, we can mirror what we
are doing with dependencies and recursively expand requirements up
front.
While at it, fix a bug in determining Requirement equality that resulted
in unique requirements being eliminated from a formula's set of
recursive requirements.
Fixes Homebrew/homebrew#12290.
Fixes Homebrew/homebrew#14084.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Apple's X11 is XQuartz, but this can be confusing, and is ultimately
unnecessary for debugging purposes.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
This is coming up a lot at the moment, so here is an attempt to be more helpful.
|
|
-D is now the switch for diagnostic-dumps. Let it be so.
Shows how long each doctor method takes in a sorted table at end. I used this to move the two slowest methods to the end of the doctor run so that as much useful information can be shown as quickly as possible.
Also now possible to specify on command line which tests should be run.
|
|
Broke when I removed `require 'formula'` from ARGV.rb.
|
|
This was a regression which caused the bash completion script to
fail to complete formula options.
|
|
Previously if error occurred in code around the logic you'd get a keg that is installed without a tab, which breaks brew.
|
|
|
|
Probably slow, but we hate these fuckers.
|
|
|
|
|
|
Using :when_xquartz_installed will tell the keg-only machinery to activate
if XQuartz is installed.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
This was slowing down every instantiation of brew significantly.
Hopefully doesn't break anything that had become accustomed to not having to require 'formula' or 'keg'.
/cc @mikemcquaid
|
|
exceptions.rb now needs String#undent when it is loaded.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
As options are stored in an object owned by the eigenclass of a formula,
options defined in the Formula#options method can be added multiple
times if the formula is instantiated multiple times.
Store them in a set to prevent duplicates.
Fixes Homebrew/homebrew#14133.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
When combining the set of old-style and new-style options, make sure
that the leading "--" is stripped.
Fixes displaying options in `brew options`, and the exotic case of
declaring options using the old syntax and then checking them with
`build.include?`
|
|
|
|
Fixes Homebrew/homebrew#14121.
|
|
Closes Homebrew/homebrew#14124.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
|
|
If you google for "Cowardly refusing to sudo brew" you get a lot of confused users who didn't read any of the Homebrew documentation and then had a hissy-fit.
|
|
Closes Homebrew/homebrew#13444.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
|
|
|
|
A user on IRC was getting strange results from MacOS.dev_tools_path.
It turns out that xcrun's exit status is not always reliable - if
xcrun is a shim and not able to locate the real xcrun, it will exit
0 despite not actually doing what it was asked. Instead check to see
if the stout is empty.
|
|
|
|
|
|
|
|
Closes Homebrew/homebrew#14066.
Signed-off-by: Max Howell <mxcl@me.com>
|
|
|
|
- So that Xcode-only systems don't fail to find svn.
Closes Homebrew/homebrew#14080.
Signed-off-by: Adam Vandenberg <flangy@gmail.com>
|
|
|
|
Closes Homebrew/homebrew#9982
|
|
Fixes Homebrew/homebrew#13638.
|
|
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
|
|
I managed to remove a return while fixing this yesterday. The whitespace changes made the diff unreadable so I missed this regression.
Why does this sort of thing still happen?
|
|
|
|
This stuff sucks. I'd like to just replace the dash with a directory division or tilde character. But this makes the code even more complicated, unless we can figure out how to migrate the taps.
|
