Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct GOARCH docs #733

Merged
merged 1 commit into from Nov 6, 2023
Merged

Correct GOARCH docs #733

merged 1 commit into from Nov 6, 2023

Conversation

cognifloyd
Copy link
Contributor

The docs had an extra v in the GOARCH+GOARM values.

Drop the v to match the actual translation tables:

nfpm/apk/apk.go

Lines 65 to 66 in 6c24137

"arm6": "armhf",
"arm7": "armv7",

nfpm/arch/arch.go

Lines 46 to 48 in 6c24137

"arm7": "armv7h",
"arm6": "armv6h",
"arm5": "arm",

nfpm/deb/deb.go

Lines 40 to 42 in 6c24137

"arm5": "armel",
"arm6": "armhf",
"arm7": "armhf",

nfpm/rpm/rpm.go

Lines 63 to 65 in 6c24137

"arm5": "armv5tel",
"arm6": "armv6hl",
"arm7": "armv7hl",

@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 6, 2023
@caarlos0 caarlos0 merged commit c4f2da3 into goreleaser:main Nov 6, 2023
1 check passed
@caarlos0
Copy link
Member

caarlos0 commented Nov 6, 2023

good catch, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants