Skip to content

fix: update architecture handling#40

Open
EzraRT wants to merge 1 commit intoargoproj:masterfrom
EzraRT:master
Open

fix: update architecture handling#40
EzraRT wants to merge 1 commit intoargoproj:masterfrom
EzraRT:master

Conversation

@EzraRT
Copy link
Copy Markdown

@EzraRT EzraRT commented Sep 16, 2025

No description provided.

…S versions

Signed-off-by: EzraRT <ezrart@ezrart.com>
@komapa
Copy link
Copy Markdown

komapa commented Oct 22, 2025

Can this be merged so we do not install amd64 executables on Macs?

@EzraRT
Copy link
Copy Markdown
Author

EzraRT commented Oct 22, 2025

Can this be merged so we do not install amd64 executables on Macs?

Yeah, now we should download the correct arch exec on both Intel CPUs and Apple Silicon.

Copy link
Copy Markdown
Member

@chansuke chansuke left a comment

Choose a reason for hiding this comment

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

lgtm too

@zachaller
Copy link
Copy Markdown
Contributor

This needs to be updated in the script: https://github.com/argoproj/homebrew-tap/blob/master/update.sh

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.

4 participants