diff options
| author | Mike McQuaid | 2015-03-17 09:03:13 +0000 | 
|---|---|---|
| committer | Mike McQuaid | 2015-03-17 09:16:57 +0000 | 
| commit | 20b4b344e69f14aa1ce6ba2ec76f0413a3c290fc (patch) | |
| tree | 08b4257363e772c04a0fb853368719637c02fe75 | |
| parent | a33c74123c4f139b4ecbc217af6dc512b9140d34 (diff) | |
| download | homebrew-20b4b344e69f14aa1ce6ba2ec76f0413a3c290fc.tar.bz2 | |
software_spec: check if bottles are compatible.
The `bottled?` method makes more sense when it also checks for
compatibility.
This is particularly useful for the case in `brew info` so it prints out
`(bottled)` correctly depending on the `Cellar`.
Closes #37636.
Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
| -rw-r--r-- | Library/Homebrew/software_spec.rb | 10 | 
1 files changed, 8 insertions, 2 deletions
| diff --git a/Library/Homebrew/software_spec.rb b/Library/Homebrew/software_spec.rb index b6ec3b485..569ea7988 100644 --- a/Library/Homebrew/software_spec.rb +++ b/Library/Homebrew/software_spec.rb @@ -62,7 +62,8 @@ class SoftwareSpec    end    def bottled? -    bottle_specification.tag?(bottle_tag) +    bottle_specification.tag?(bottle_tag) && \ +      bottle_specification.compatible_cellar?    end    def bottle &block @@ -220,6 +221,7 @@ class Bottle      @name = formula.name      @resource = Resource.new      @resource.owner = formula +    @spec = spec      checksum, tag = spec.checksum_for(bottle_tag) @@ -234,7 +236,7 @@ class Bottle    end    def compatible_cellar? -    cellar == :any || cellar == HOMEBREW_CELLAR.to_s +    @spec.compatible_cellar?    end    def stage @@ -265,6 +267,10 @@ class BottleSpecification      @collector = BottleCollector.new    end +  def compatible_cellar? +    cellar == :any || cellar == HOMEBREW_CELLAR.to_s +  end +    def tag?(tag)      !!checksum_for(tag)    end | 
