diff options
| author | Jack Nagel | 2012-12-29 17:16:38 -0600 |
|---|---|---|
| committer | Jack Nagel | 2012-12-30 19:23:53 -0600 |
| commit | de1eabf22c0137e4e0ad8c34cc47fd86cb78e557 (patch) | |
| tree | 589daf938cc90ffb9f2b2f21ce1ff58e6f5a8d7a /Library/Homebrew/cmd | |
| parent | 9362a7c89732a1cea6a1ada1062fedcf24505b8a (diff) | |
| download | brew-de1eabf22c0137e4e0ad8c34cc47fd86cb78e557.tar.bz2 | |
Fix some doublethink in the Xcode module
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 Homebrew/homebrew#16790.
Diffstat (limited to 'Library/Homebrew/cmd')
0 files changed, 0 insertions, 0 deletions