Skip to content

Conversation

mcamou
Copy link
Contributor

@mcamou mcamou commented Oct 21, 2024

No description provided.

@mcamou
Copy link
Contributor Author

mcamou commented Oct 25, 2024

@PsypherPunk would you mind having a look at this? I'd like to have it count towards Hacktoberfest.

@PsypherPunk
Copy link
Collaborator

@mcamou, this doesn't quite work: the platforms addition is correct as per the docs. but your changes fail with an error:

ERROR: Multi-platform build is not supported for the docker driver.

Based on those same docs. and some testing, it looks like we need the docker/setup-buildx-action too:

- name: Set up Docker Buildx
        uses: docker/setup-buildx-action@v3

With that in place both images appear to build correctly.

@mcamou mcamou force-pushed the add-arm64-docker-build branch from d7f0dfa to ea428a1 Compare October 26, 2024 10:19
@mcamou
Copy link
Contributor Author

mcamou commented Oct 26, 2024

@PsypherPunk thank you for that. I did it from memory and had forgotten about that. I've amended the commit and force-pushed to keep the commit history clean.

@PsypherPunk
Copy link
Collaborator

Looks good (and linking to this as evidence)—thanks.

Happy Hacktoberfest!

Copy link
Collaborator

@PsypherPunk PsypherPunk left a comment

Choose a reason for hiding this comment

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

Verified successful build here.

@PsypherPunk PsypherPunk merged commit aed0d71 into bee-san:master Oct 26, 2024
1 check passed
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