bottles: support :all tag.
This allows the creation of bottles which will be used by any macOS version, architecture or OS (i.e. macOS or Linux). Add `TODO` stubs for where the bottle generation logic should be implemented.
This commit is contained in:
parent
6b5213286c
commit
0cb1645d25
@ -585,6 +585,12 @@ module Homebrew
|
|||||||
def merge(args:)
|
def merge(args:)
|
||||||
bottles_hash = merge_json_files(parse_json_files(args.named))
|
bottles_hash = merge_json_files(parse_json_files(args.named))
|
||||||
|
|
||||||
|
# TODO: deduplicate --no-json bottles by:
|
||||||
|
# 1. throwing away bottles for newer versions of macOS if their SHA256 is
|
||||||
|
# identical
|
||||||
|
# 2. generating `all: $SHA256` bottles that can be used on macOS and Linux
|
||||||
|
# i.e. need to be `any_skip_relocation` and contain no ELF/MachO files.
|
||||||
|
|
||||||
any_cellars = ["any", "any_skip_relocation"]
|
any_cellars = ["any", "any_skip_relocation"]
|
||||||
bottles_hash.each do |formula_name, bottle_hash|
|
bottles_hash.each do |formula_name, bottle_hash|
|
||||||
ohai formula_name
|
ohai formula_name
|
||||||
|
|||||||
@ -493,37 +493,43 @@ describe "brew bottle" do
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
specify "::merge_json_files" do
|
describe "::merge_json_files" do
|
||||||
bottles_hash = homebrew.merge_json_files(
|
it "merges JSON files" do
|
||||||
[hello_hash_big_sur, hello_hash_catalina, unzip_hash_big_sur, unzip_hash_catalina],
|
bottles_hash = homebrew.merge_json_files(
|
||||||
)
|
[hello_hash_big_sur, hello_hash_catalina, unzip_hash_big_sur, unzip_hash_catalina],
|
||||||
|
)
|
||||||
|
|
||||||
hello_hash = bottles_hash["hello"]
|
hello_hash = bottles_hash["hello"]
|
||||||
expect(hello_hash["bottle"]["tags"]["big_sur"]["cellar"]).to eq("any_skip_relocation")
|
expect(hello_hash["bottle"]["tags"]["big_sur"]["cellar"]).to eq("any_skip_relocation")
|
||||||
expect(hello_hash["bottle"]["tags"]["big_sur"]["filename"]).to eq("hello-1.0.big_sur.bottle.tar.gz")
|
expect(hello_hash["bottle"]["tags"]["big_sur"]["filename"]).to eq("hello-1.0.big_sur.bottle.tar.gz")
|
||||||
expect(hello_hash["bottle"]["tags"]["big_sur"]["local_filename"]).to eq("hello--1.0.big_sur.bottle.tar.gz")
|
expect(hello_hash["bottle"]["tags"]["big_sur"]["local_filename"]).to eq("hello--1.0.big_sur.bottle.tar.gz")
|
||||||
expect(hello_hash["bottle"]["tags"]["big_sur"]["sha256"]).to eq(
|
expect(hello_hash["bottle"]["tags"]["big_sur"]["sha256"]).to eq(
|
||||||
"a0af7dcbb5c83f6f3f7ecd507c2d352c1a018f894d51ad241ce8492fa598010f",
|
"a0af7dcbb5c83f6f3f7ecd507c2d352c1a018f894d51ad241ce8492fa598010f",
|
||||||
)
|
)
|
||||||
expect(hello_hash["bottle"]["tags"]["catalina"]["cellar"]).to eq("any_skip_relocation")
|
expect(hello_hash["bottle"]["tags"]["catalina"]["cellar"]).to eq("any_skip_relocation")
|
||||||
expect(hello_hash["bottle"]["tags"]["catalina"]["filename"]).to eq("hello-1.0.catalina.bottle.tar.gz")
|
expect(hello_hash["bottle"]["tags"]["catalina"]["filename"]).to eq("hello-1.0.catalina.bottle.tar.gz")
|
||||||
expect(hello_hash["bottle"]["tags"]["catalina"]["local_filename"]).to eq("hello--1.0.catalina.bottle.tar.gz")
|
expect(hello_hash["bottle"]["tags"]["catalina"]["local_filename"]).to eq("hello--1.0.catalina.bottle.tar.gz")
|
||||||
expect(hello_hash["bottle"]["tags"]["catalina"]["sha256"]).to eq(
|
expect(hello_hash["bottle"]["tags"]["catalina"]["sha256"]).to eq(
|
||||||
"5334dd344986e46b2aa4f0471cac7b0914bd7de7cb890a34415771788d03f2ac",
|
"5334dd344986e46b2aa4f0471cac7b0914bd7de7cb890a34415771788d03f2ac",
|
||||||
)
|
)
|
||||||
unzip_hash = bottles_hash["unzip"]
|
unzip_hash = bottles_hash["unzip"]
|
||||||
expect(unzip_hash["bottle"]["tags"]["big_sur"]["cellar"]).to eq("any_skip_relocation")
|
expect(unzip_hash["bottle"]["tags"]["big_sur"]["cellar"]).to eq("any_skip_relocation")
|
||||||
expect(unzip_hash["bottle"]["tags"]["big_sur"]["filename"]).to eq("unzip-2.0.big_sur.bottle.tar.gz")
|
expect(unzip_hash["bottle"]["tags"]["big_sur"]["filename"]).to eq("unzip-2.0.big_sur.bottle.tar.gz")
|
||||||
expect(unzip_hash["bottle"]["tags"]["big_sur"]["local_filename"]).to eq("unzip--2.0.big_sur.bottle.tar.gz")
|
expect(unzip_hash["bottle"]["tags"]["big_sur"]["local_filename"]).to eq("unzip--2.0.big_sur.bottle.tar.gz")
|
||||||
expect(unzip_hash["bottle"]["tags"]["big_sur"]["sha256"]).to eq(
|
expect(unzip_hash["bottle"]["tags"]["big_sur"]["sha256"]).to eq(
|
||||||
"16cf230afdfcb6306c208d169549cf8773c831c8653d2c852315a048960d7e72",
|
"16cf230afdfcb6306c208d169549cf8773c831c8653d2c852315a048960d7e72",
|
||||||
)
|
)
|
||||||
expect(unzip_hash["bottle"]["tags"]["catalina"]["cellar"]).to eq("any")
|
expect(unzip_hash["bottle"]["tags"]["catalina"]["cellar"]).to eq("any")
|
||||||
expect(unzip_hash["bottle"]["tags"]["catalina"]["filename"]).to eq("unzip-2.0.catalina.bottle.tar.gz")
|
expect(unzip_hash["bottle"]["tags"]["catalina"]["filename"]).to eq("unzip-2.0.catalina.bottle.tar.gz")
|
||||||
expect(unzip_hash["bottle"]["tags"]["catalina"]["local_filename"]).to eq("unzip--2.0.catalina.bottle.tar.gz")
|
expect(unzip_hash["bottle"]["tags"]["catalina"]["local_filename"]).to eq("unzip--2.0.catalina.bottle.tar.gz")
|
||||||
expect(unzip_hash["bottle"]["tags"]["catalina"]["sha256"]).to eq(
|
expect(unzip_hash["bottle"]["tags"]["catalina"]["sha256"]).to eq(
|
||||||
"d9cc50eec8ac243148a121049c236cba06af4a0b1156ab397d0a2850aa79c137",
|
"d9cc50eec8ac243148a121049c236cba06af4a0b1156ab397d0a2850aa79c137",
|
||||||
)
|
)
|
||||||
|
end
|
||||||
|
|
||||||
|
# TODO: add deduplication tests e.g.
|
||||||
|
# it "deduplicates JSON files with matching macOS checksums"
|
||||||
|
# it "deduplicates JSON files with matching OS checksums" do
|
||||||
end
|
end
|
||||||
|
|
||||||
describe "#merge_bottle_spec" do
|
describe "#merge_bottle_spec" do
|
||||||
|
|||||||
@ -87,6 +87,8 @@ module Utils
|
|||||||
|
|
||||||
sig { params(value: Symbol).returns(T.attached_class) }
|
sig { params(value: Symbol).returns(T.attached_class) }
|
||||||
def self.from_symbol(value)
|
def self.from_symbol(value)
|
||||||
|
return new(system: :all, arch: :all) if value == :all
|
||||||
|
|
||||||
@all_archs_regex ||= begin
|
@all_archs_regex ||= begin
|
||||||
all_archs = Hardware::CPU::ALL_ARCHS.map(&:to_s)
|
all_archs = Hardware::CPU::ALL_ARCHS.map(&:to_s)
|
||||||
/
|
/
|
||||||
@ -118,7 +120,9 @@ module Utils
|
|||||||
|
|
||||||
sig { returns(Symbol) }
|
sig { returns(Symbol) }
|
||||||
def to_sym
|
def to_sym
|
||||||
if macos? && arch == :x86_64
|
if system == :all && arch == :all
|
||||||
|
:all
|
||||||
|
elsif macos? && arch == :x86_64
|
||||||
system
|
system
|
||||||
else
|
else
|
||||||
"#{arch}_#{system}".to_sym
|
"#{arch}_#{system}".to_sym
|
||||||
@ -219,7 +223,11 @@ module Utils
|
|||||||
private
|
private
|
||||||
|
|
||||||
def find_matching_tag(tag, no_older_versions: false)
|
def find_matching_tag(tag, no_older_versions: false)
|
||||||
tag if key?(tag.to_sym)
|
if key?(tag.to_sym)
|
||||||
|
tag
|
||||||
|
elsif key?(:all)
|
||||||
|
Tag.from_symbol(:all)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user