Skip to content

[nrf noup] Use proper ZAP package for Linux/Windows with ARM CPUs#601

Merged
ArekBalysNordic merged 1 commit into
nrfconnect:masterfrom
kendallgoto:kgoto/fix-zap-linux-arch
May 9, 2025
Merged

[nrf noup] Use proper ZAP package for Linux/Windows with ARM CPUs#601
ArekBalysNordic merged 1 commit into
nrfconnect:masterfrom
kendallgoto:kgoto/fix-zap-linux-arch

Conversation

@kendallgoto
Copy link
Copy Markdown
Contributor

Currently x86 ZAP is always downloaded on Linux & Windows. This change looks at the architecture of the system to download either x86_64 or arm64 binaries as provided by ZAP releases

This is the Linux / Windows equivalent to #574 which similarly downloads ARM64 binaries for Darwin systems on Apple Silicon.

@kendallgoto kendallgoto requested a review from a team as a code owner April 24, 2025 02:28
@NordicBuilder
Copy link
Copy Markdown

none

Note: This comment is automatically posted and updated by the Contribs GitHub Action.

Copy link
Copy Markdown
Contributor

@kkasperczyk-no kkasperczyk-no left a comment

Choose a reason for hiding this comment

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

It looks good to me 👍 Thank you for the contribution!

Currently x86 ZAP is always downloaded on Linux/Windows. This change
looks at the architecture of the system to download either
x86_64 or arm64 binaries as provided by ZAP releases

Signed-off-by: Kendall Goto <kendall@level.co>
@kendallgoto kendallgoto force-pushed the kgoto/fix-zap-linux-arch branch from e591f08 to b2c7790 Compare April 25, 2025 09:01
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 25, 2025

CLA assistant check
All committers have signed the CLA.

@ArekBalysNordic ArekBalysNordic merged commit cb45489 into nrfconnect:master May 9, 2025
8 of 9 checks passed
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.

5 participants