| Age | Commit message (Collapse) | Author |
|---|
|
|
|
|
|
Fixes #16989.
|
|
Fixes #17074.
Closes #17075.
Signed-off-by: Adam Vandenberg <flangy@gmail.com>
|
|
The DependencyCollector tests are really integration tests, while the
rest are closer to real unit tests. Split them up so that the tests can
be run in isolation on a per-class basis.
|
|
It is important that dep equality corresponds to the name attribute, but
we may want to use the Comparable interface to sort them by installation
order in the future. Code that needs to sort them alphabetically should
just use sort_by.
|
|
* Detect `latex' and `bibtex' commands.
* Recommend installing MacTeX when no LaTeX installation is found.
Signed-off-by: Adam Vandenberg <flangy@gmail.com>
|
|
Commit 30a08f5cc707 ("Don't attempt installation multiple times") was
correct in raising FormulaInstallationAlreadyAttemptedError early enough
to prevent the other exceptions in #check_install_sanity from being
raised, but it should not have moved the point at which formulae are
added to the attempted set. Doing so prevents the installer from being
instantiated multiple times on the same formula, even if installation
was never attempted.
Put it back where it belongs.
|
|
|
|
|
|
- Name of the brewed formula in "Summary" heading.
- At the beginning of the summary line a 🍺 is given out
(but only on Lion or above). This acts as a nice
visual marker for the end of a brew-ing process.
- When brewing a dependency, the name is highlighted in green.
- `oh1` is bold, too (like ohai).
Closes #16020.
Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
|
|
When a dependency of a formula specified on the command-line is also
specified, *after* the dependent formula, installation proceeds as part
of the dependent's dependency tree and then is attempted again because
the user asked for it explicitly. This results in the installer raising
a CannotInstallFormulaError because it has already been installed.
For example:
$ brew install graphviz pkg-config
==> Installing graphviz dependency: pkg-config
...
==> Installing graphviz
...
Error: pkg-config-0.27.1 already installed
We already have a mechanism for dealing with this, but it does not kick
in early enough. Move the installation attempt check into
FormulaInstaller#check_install_sanity and catch the exception in the
appropriate places.
Fixes #16957.
|
|
Closes #16409.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
|
|
|
|
Procs cannot be marshalled, but formula objects may be as part of the
BuildError exception. If the formula object has a reference to a Proc,
this will fail.
Work around it by not storing the test Procs during installations.
|
|
Ruby 1.8.6 doesn't have Symbol#to_proc, which allows things like
map(&:to_s) rather than map { |o| o.to_s }. 1.8.7 does, though, and
since it is used in a bunch of the superenv code we should attempt to
keep it compatible with 1.8.6.
Closes #16046.
|
|
`python --version` doesn't work in some (very) old versions of
python. While I doubt this is going to come up very often, it's
theoretically possible someone has an old python first in their
path. (python -V works on all versions I'm aware of, including
python3, and comes before --version in the --help anyway.)
Also skips the warning in case the regexp matches nothing, which
probably shouldn't happen. But if it does we shouldn't produce the
wrong warning.
|
|
|
|
|
|
Closes #16947.
Signed-off-by: Adam Vandenberg <flangy@gmail.com>
|
|
Fixes #16948.
|
|
|
|
This is needed for the intersection code in `brew deps` to work right,
but can hopefully be refactored away somehow eventually.
|
|
|
|
Inheriting from Array (and other core types) is problematic:
- It exposes a very wide interface with many methods that are not
really relevant to the subclass.
- It can cause some weird side effects, as many Array operations are
in C and have hardcoded return values; for example, combining two
array subclasses returns a new Array instead of the subclass.
Avoid these problems using delegation and the Enumerable module where
applicable.
|
|
|
|
|
|
Tests can now be specified as a block in the DSL. A temporary test
directory is set up automatically when calling Formula#test. The
semantics of the test remain the same: the block can either raise an
exception or return false to signal failure.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
|
|
|
|
Closes #16280.
Signed-off-by: Adam Vandenberg <flangy@gmail.com>
|
|
Closes #16901.
|
|
|
|
|
|
Superenv scripts have a stripped PATH, which may not include
the brew binary itself. Make this explicitly available to
superenv scripts.
Fixes bsdmake wrapper.
Closes #16805.
Closes #16846.
Signed-off-by: Adam Vandenberg <flangy@gmail.com>
|
|
Fixes #16857.
|
|
Probably a better approach than reverted e721c7.
Fixes #16604.
|
|
|
|
|
|
Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
|
|
|
|
Xcode.prefix and Xcode.installed? use slightly different heuristics to
find Xcode. In fact, .installed? basically duplicates a portion of the
.prefix logic. In practice, the methods results are usually consistent,
but .installed? does not handle non-standard prefixes if mdfind cannot
locate Xcode (for example, if the user has disabled Spotlight indexing).
Since .installed? is essentially a subset of the .prefix logic, we can
rely on the result from .prefix instead.
Fixes #16790.
|
|
|
|
@sceaga has discontinued his homebrew fork, so tigerbrew is now PPC/
Tiger users' best bet.
|
|
Originally written for tigerbrew, but useful enough for core.
Replaces the shelled-out which in utils.rb with a native-ruby
equivalent, which is moderately faster.
Closes #16659.
Signed-off-by: Misty De Meo <mistydemeo@gmail.com>
|
|
|
|
|
|
|
|
c.f. #16296.
|
|
|
