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

Recognize arm64 as a valid goarch #2928

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

aycyang
Copy link
Contributor

@aycyang aycyang commented May 1, 2023

The itch app queries the OS for its platform architecture in order to decide which builds of butler and itch-setup to download from broth.itch.ovh. There is a bit of ad-hoc logic in the itch app to translate the architecture string into the format used by Go, which is what butler and itch-setup are written in and packaged with. This ad-hoc logic did not recognize arm64 as a valid architecture, likely because broth.itch.ovh does not currently serve arm64 builds of butler and itch-setup. This commit prepares the itch app for a future where such builds are available.

The itch app queries the OS for its platform architecture in order to
decide which builds of butler and itch-setup to download from
broth.itch.ovh. There is a bit of ad-hoc logic in the itch app to
translate the architecture string into the format used by Go, which is
what butler and itch-setup are written in and packaged with. This ad-hoc
logic did not recognize arm64 as a valid architecture, likely because
broth.itch.ovh does not currently serve arm64 builds of butler and
itch-setup. This commit prepares the itch app for a future where such
builds are available.
@aycyang
Copy link
Contributor Author

aycyang commented May 1, 2023

In my manual local tests on my M1 Mac, observed the following behavior change:

Before:

ERROR (🔧 setup) setup got error: Error: Could not retrieve latest version of (butler):
got HTTP 404 while fetching (https://broth.itch.ovh/butler/darwin-unknown-head/LATEST)

After:

ERROR (🔧 setup) setup got error: Error: Could not retrieve latest version of (butler):
got HTTP 404 while fetching (https://broth.itch.ovh/butler/darwin-arm64-head/LATEST)

@aycyang aycyang mentioned this pull request May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant