Skip to content

Conversation

@manugupt1
Copy link
Contributor

No description provided.

@manugupt1
Copy link
Contributor Author

cc @AkihiroSuda is something like this acceptable.
We build only for the TARGETARCH.
Just doing that saves more than 2 minutes per build.

@manugupt1 manugupt1 force-pushed the build-arch branch 5 times, most recently from 710d0db to eeab95e Compare February 15, 2023 07:45
@manugupt1 manugupt1 changed the title build for specific arch only Build containerd and runc for specific arch with xx Feb 15, 2023
@manugupt1 manugupt1 marked this pull request as ready for review February 15, 2023 08:14
cp -a bypass4netns bypass4netnsd /out/arm64
RUN xx-go --wrap
RUN GO=xx-go GOARCH=${TARGETARCH} CC=$(xx-info)-gcc make static && \
xx-verify --static bypass4netns && cp -a bypass4netns bypass4netnsd /out/${TARGETARCH}
Copy link
Member

Choose a reason for hiding this comment

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

When GO=xx-go is set, no need to run --wrap and specify GOARCH, CC, etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks! sorry! misunderstood earlier. Thanks for helping with this PR throughout

Dockerfile Outdated
cp -a bypass4netns bypass4netnsd /out/amd64
RUN GOARCH=arm64 CC=aarch64-linux-gnu-gcc make static && \
cp -a bypass4netns bypass4netnsd /out/arm64
RUN xx-go --wrap
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove

Signed-off-by: Manu Gupta <manugupt1@gmail.com>
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit 2e5cdac into containerd:main Feb 16, 2023
@manugupt1 manugupt1 deleted the build-arch branch February 16, 2023 23:00
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.

2 participants