Skip to content

Commit

Permalink
Merge pull request #6212 from mistydemeo/allow_skipping_or_later_bottles
Browse files Browse the repository at this point in the history
Bottles: allow skipping or_later tags
  • Loading branch information
mistydemeo authored Jun 6, 2019
2 parents 80230ec + 5ab34ab commit 1475554
Show file tree
Hide file tree
Showing 3 changed files with 30 additions and 5 deletions.
4 changes: 4 additions & 0 deletions Library/Homebrew/extend/ARGV.rb
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,10 @@ def homebrew_developer?
!ENV["HOMEBREW_DEVELOPER"].nil?
end

def skip_or_later_bottles?
homebrew_developer? && !ENV["HOMEBREW_SKIP_OR_LATER_BOTTLES"].nil?
end

def no_sandbox?
include?("--no-sandbox") || !ENV["HOMEBREW_NO_SANDBOX"].nil?
end
Expand Down
9 changes: 7 additions & 2 deletions Library/Homebrew/extend/os/mac/utils/bottles.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,13 @@ class Collector
alias generic_find_matching_tag find_matching_tag

def find_matching_tag(tag)
generic_find_matching_tag(tag) ||
find_older_compatible_tag(tag)
# Used primarily by developers testing beta macOS releases.
if OS::Mac.prerelease? && ARGV.skip_or_later_bottles?
generic_find_matching_tag(tag)
else
generic_find_matching_tag(tag) ||
find_older_compatible_tag(tag)
end
end

def tag_without_or_later(tag)
Expand Down
22 changes: 19 additions & 3 deletions Library/Homebrew/test/utils/bottles/collector_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,25 @@
end

it "uses older tags when needed", :needs_macos do
subject[:yosemite] = "foo"
expect(subject.fetch_checksum_for(:yosemite)).to eq(["foo", :yosemite])
expect(subject.fetch_checksum_for(:mavericks)).to be nil
subject[:mavericks] = "foo"
expect(subject.send(:find_matching_tag, :mavericks)).to eq(:mavericks)
expect(subject.send(:find_matching_tag, :yosemite)).to eq(:mavericks)
end

it "does not use older tags when requested not to", :needs_macos do
allow(ARGV).to receive(:skip_or_later_bottles?).and_return(true)
allow(OS::Mac).to receive(:prerelease?).and_return(true)
subject[:mavericks] = "foo"
expect(subject.send(:find_matching_tag, :mavericks)).to eq(:mavericks)
expect(subject.send(:find_matching_tag, :yosemite)).to be_nil
end

it "ignores HOMEBREW_SKIP_OR_LATER_BOTTLES on release versions", :needs_macos do
allow(ARGV).to receive(:skip_or_later_bottles?).and_return(true)
allow(OS::Mac).to receive(:prerelease?).and_return(false)
subject[:mavericks] = "foo"
expect(subject.send(:find_matching_tag, :mavericks)).to eq(:mavericks)
expect(subject.send(:find_matching_tag, :yosemite)).to eq(:mavericks)
end
end
end

0 comments on commit 1475554

Please sign in to comment.