Skip to content

Conversation

@reinhrst
Copy link

@reinhrst reinhrst commented Jan 7, 2023

This PR contains the changes necessary to build the container for arm64. On my M1 Macbook Pro, compiling in the arm64 container is 2.5 times faster than the amd64 one. On other devices (like the RPI4, or linux on an M1 Macbook), arm64 is the only container that will run.

I'm not suggesting pulling in this PR verbatim, but either as a separate branch, or maybe add an extra parameter to the build scripts (I'd be happy to do that if you want) to switch between the architectures (or maybe even have switches in the Dockerfiles themselves for different architectures if this is possible?).

Eventually ideally the pre-built container will be available for arm64 as well.

I just want to make this work available for the next person who needs this.

@CLAassistant
Copy link

CLAassistant commented Jan 7, 2023

CLA assistant check
All committers have signed the CLA.

@reinhrst reinhrst marked this pull request as draft January 7, 2023 13:18
&& chmod 777 /tmp && apt-get update \
&& apt-get -y install autoconf \
&& apt-get -y install --no-install-recommends sudo make g++ g++-multilib \
&& apt-get -y install --no-install-recommends sudo make g++ gcc-multilib-arm-linux-gnueabi gcc-multilib-arm-linux-gnueabihf \
Copy link
Author

Choose a reason for hiding this comment

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

Actually, you probably don't need both, but I didn't spend the time to check which one is needed.

@Damian-Nordic
Copy link
Collaborator

@reinhrst Thanks for the contribution. We'll think how to provide builds for arm64 architecture as well.

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