dev-cmd/dispatch-build-bottle: set ARM labels.

Set the label properly for ARM.

Tested with inputs `11-arm` and `arm64_big_sur`.

Fixes https://github.com/Homebrew/brew/issues/9434
This commit is contained in:
Mike McQuaid 2020-12-07 13:20:01 +00:00
parent 62de53a515
commit 09cca671bd
No known key found for this signature in database
GPG Key ID: 48A898132FD8EE70

View File

@ -35,13 +35,31 @@ module Homebrew
def dispatch_build_bottle def dispatch_build_bottle
args = dispatch_build_bottle_args.parse args = dispatch_build_bottle_args.parse
# Fixup version for ARM/Apple Silicon
arm_regex = Regexp.union(/^arm64_/, /-arm$/)
arm_label = if arm_regex.match?(args.macos)
args.macos&.gsub!(arm_regex, "")
true
else
false
end
macos = args.macos&.yield_self do |s| macos = args.macos&.yield_self do |s|
MacOS::Version.from_symbol(s.to_sym) MacOS::Version.from_symbol(s.to_sym)
rescue MacOSVersionError rescue MacOSVersionError
MacOS::Version.new(s) MacOS::Version.new(s)
end end
raise UsageError, "Must specify --macos option" unless macos raise UsageError, "Must specify --macos option" if macos.blank?
# Fixup label for ARM/Apple Silicon
macos_label = if arm_label
"#{macos}-arm"
else
macos.to_s
end
p macos_label
odie "no"
tap = Tap.fetch(args.tap || CoreTap.instance.name) tap = Tap.fetch(args.tap || CoreTap.instance.name)
user, repo = tap.full_name.split("/") user, repo = tap.full_name.split("/")
@ -53,7 +71,7 @@ module Homebrew
# Required inputs # Required inputs
inputs = { inputs = {
formula: formula.name, formula: formula.name,
macos: macos.to_s, macos: macos_label,
} }
# Optional inputs # Optional inputs