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

Fix arch detection of Linux/aarch64 #59

Merged
merged 1 commit into from
Feb 27, 2024
Merged

Conversation

nirs
Copy link
Contributor

@nirs nirs commented Feb 21, 2024

Fix arch detection on Linux/aarch64 (e.g. Asahi Linux on MacBook M*). From Go point of view, this is arm64.

$ uname -m
aarch64

$ go env GOOS
linux

$ go env GOARCH
arm64

Tested by running directly from source.

Fixes #58

@submariner-bot
Copy link

🤖 Created branch: z_pr59/nirs/aarch64

@nirs
Copy link
Contributor Author

nirs commented Feb 21, 2024

Example run:

$ bash < index.html 
Installing subctl version latest
  OS detected:           linux
  Architecture detected: arm64
  Download URL:          https://github.com/submariner-io/releases/releases/download/v0.16.3/subctl-v0.16.3-linux-arm64.tar.gz

Downloading...
Download attempt #1...
subctl has been installed as /home/nir/.local/bin/subctl
This provides subctl version: v0.16.3

$ which subctl
~/.local/bin/subctl

$ subctl version
subctl version: v0.16.3

Fix arch detection on Linux/aarch64 (e.g. Asahi Linux on MacBook M*).
From Go point of view, this is arm64.

    $ uname -m
    aarch64

    $ go env GOOS
    linux

    $ go env GOARCH
    arm64

Tested by running directly from source.

Fixes submariner-io#58

Signed-off-by: Nir Soffer <[email protected]>
@dfarrell07 dfarrell07 merged commit 6bf5b5b into submariner-io:devel Feb 27, 2024
10 checks passed
@submariner-bot
Copy link

🤖 Closed branches: [z_pr59/nirs/aarch64]

@dfarrell07
Copy link
Member

Thanks for the PR @nirs!

@nirs nirs deleted the aarch64 branch February 27, 2024 15:16
tpantelis added a commit to tpantelis/submariner-website that referenced this pull request Mar 12, 2024
Release note for submariner-io/get.submariner.io#59

This isn't tied to an actual release but it most closely coincides with the
0.17 release.

Signed-off-by: Tom Pantelis <[email protected]>
tpantelis added a commit to submariner-io/submariner-website that referenced this pull request Mar 18, 2024
Release note for submariner-io/get.submariner.io#59

This isn't tied to an actual release but it most closely coincides with the
0.17 release.

Signed-off-by: Tom Pantelis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Install script fail to detect aarch64, call undefined function
5 participants