Skip to content

Bugfix/docker bake linux amd64 macos#384

Open
esalituro wants to merge 4 commits into
AcademySoftwareFoundation:mainfrom
esalituro:bugfix/docker-bake-linux-amd64-macos
Open

Bugfix/docker bake linux amd64 macos#384
esalituro wants to merge 4 commits into
AcademySoftwareFoundation:mainfrom
esalituro:bugfix/docker-bake-linux-amd64-macos

Conversation

@esalituro
Copy link
Copy Markdown

Adds and explicit platforms for MacOS builds. Adds test to confirm it only affects MacOS.
Updates README.md with MacOS notes.

jfpanisset
jfpanisset previously approved these changes Apr 11, 2026
Copy link
Copy Markdown
Contributor

@jfpanisset jfpanisset left a comment

Choose a reason for hiding this comment

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

Thank you for this Eric, I've had to resolve some merge conflicts since your branch ran into some recent cleanup changes (getting rid of the --use-conan option in aswfdocker), can you take a look and make sure I didn't break anything before merging?

@jfpanisset
Copy link
Copy Markdown
Contributor

Ugh, I may have messed something up, I've tried to resolve the conflicts, but GitHub is still showing "This branch cannot be rebased due to conflicts".

Any chance you could resubmit the changes against a fresh checkout from top of main? I don't want to mess up the repo by attempting to do things I don't fully understand.

esalituro and others added 2 commits May 5, 2026 19:59
Signed-off-by: Eric Salituro <esalituro@icloud.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
- README: Linux artifacts on macOS, Docker 8c/16GB, buildx on Docker VMM vs Rosetta

Signed-off-by: Eric Salituro <esalituro@icloud.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@esalituro esalituro force-pushed the bugfix/docker-bake-linux-amd64-macos branch from 76d1ba8 to 12702de Compare May 6, 2026 03:37
@jfpanisset
Copy link
Copy Markdown
Contributor

Looks good, as soon as I have a working set of latest releases done I'll merge this. Thank you!

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