Normalize arm64 to an empty variant
Signed-off-by: Derek McGowan <derek@mcgstyle.net>
This commit is contained in:
parent
47a128d455
commit
db3c5afc6d
@ -92,8 +92,8 @@ func normalizeArch(arch, variant string) (string, string) {
|
|||||||
case "aarch64", "arm64":
|
case "aarch64", "arm64":
|
||||||
arch = "arm64"
|
arch = "arm64"
|
||||||
switch variant {
|
switch variant {
|
||||||
case "", "8":
|
case "8", "v8":
|
||||||
variant = "v8"
|
variant = ""
|
||||||
}
|
}
|
||||||
case "armhf":
|
case "armhf":
|
||||||
arch = "arm"
|
arch = "arm"
|
||||||
@ -112,15 +112,3 @@ func normalizeArch(arch, variant string) (string, string) {
|
|||||||
|
|
||||||
return arch, variant
|
return arch, variant
|
||||||
}
|
}
|
||||||
|
|
||||||
// defaultVariant detects default variants on normalized arch/variant
|
|
||||||
func defaultVariant(arch, variant string) bool {
|
|
||||||
switch arch {
|
|
||||||
case "arm64":
|
|
||||||
return variant == "v8"
|
|
||||||
case "arm":
|
|
||||||
return variant == "v7"
|
|
||||||
default:
|
|
||||||
return true
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
@ -197,7 +197,7 @@ func Parse(specifier string) (specs.Platform, error) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
p.Architecture, p.Variant = normalizeArch(parts[0], "")
|
p.Architecture, p.Variant = normalizeArch(parts[0], "")
|
||||||
if defaultVariant(p.Architecture, p.Variant) {
|
if p.Architecture == "arm" && p.Variant == "v7" {
|
||||||
p.Variant = ""
|
p.Variant = ""
|
||||||
}
|
}
|
||||||
if isKnownArch(p.Architecture) {
|
if isKnownArch(p.Architecture) {
|
||||||
@ -211,7 +211,7 @@ func Parse(specifier string) (specs.Platform, error) {
|
|||||||
// about whether or not we know of the platform.
|
// about whether or not we know of the platform.
|
||||||
p.OS = normalizeOS(parts[0])
|
p.OS = normalizeOS(parts[0])
|
||||||
p.Architecture, p.Variant = normalizeArch(parts[1], "")
|
p.Architecture, p.Variant = normalizeArch(parts[1], "")
|
||||||
if defaultVariant(p.Architecture, p.Variant) {
|
if p.Architecture == "arm" && p.Variant == "v7" {
|
||||||
p.Variant = ""
|
p.Variant = ""
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -220,6 +220,9 @@ func Parse(specifier string) (specs.Platform, error) {
|
|||||||
// we have a fully specified variant, this is rare
|
// we have a fully specified variant, this is rare
|
||||||
p.OS = normalizeOS(parts[0])
|
p.OS = normalizeOS(parts[0])
|
||||||
p.Architecture, p.Variant = normalizeArch(parts[1], parts[2])
|
p.Architecture, p.Variant = normalizeArch(parts[1], parts[2])
|
||||||
|
if p.Architecture == "arm64" && p.Variant == "" {
|
||||||
|
p.Variant = "v8"
|
||||||
|
}
|
||||||
|
|
||||||
return p, nil
|
return p, nil
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user