Skip to content

Add substitution for aarch64 to installation bash script#363

Merged
nimakaviani merged 1 commit intocnoe-io:mainfrom
andrew-rosca:patch-1
Aug 26, 2024
Merged

Add substitution for aarch64 to installation bash script#363
nimakaviani merged 1 commit intocnoe-io:mainfrom
andrew-rosca:patch-1

Conversation

@andrew-rosca
Copy link
Copy Markdown
Contributor

On some ARM64 Linux instances, uname -m returns aarch64 instead of arm64. This causes the installation script to fail.

This change adds an additional substitution for this scenario

@nabuskey
Copy link
Copy Markdown
Collaborator

@nimakaviani Please review.

@nabuskey nabuskey requested a review from nimakaviani August 22, 2024 19:10
@nimakaviani
Copy link
Copy Markdown
Contributor

LGTM.

can you please sign the DCO? thanks!

On some ARM64 Linux instances, `uname -m` returns `aarch64` instead of `arm64`. This causes the installation script to fail.

This change adds an additional substitution for this scenario

Signed-off-by: Andrew Rosca <andrew@rosca.net>
@andrew-rosca
Copy link
Copy Markdown
Contributor Author

@nimakaviani done!

Copy link
Copy Markdown
Contributor

@nimakaviani nimakaviani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nimakaviani nimakaviani merged commit 78be2c2 into cnoe-io:main Aug 26, 2024
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.

3 participants