Skip to content

feat(docker): add support for specifying Docker platform in build and…#1008

Merged
YCC3741 merged 2 commits into
mainfrom
docker-comp
Jun 14, 2026
Merged

feat(docker): add support for specifying Docker platform in build and…#1008
YCC3741 merged 2 commits into
mainfrom
docker-comp

Conversation

@yu2C

@yu2C yu2C commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

… run scripts

Introduce DOCKER_PLATFORM variable to ensure compatibility with arm64/aarch64 hosts. Update Makefile and various scripts to utilize this variable for Docker commands, enhancing cross-platform support.

… run scripts

Introduce DOCKER_PLATFORM variable to ensure compatibility with arm64/aarch64 hosts. Update Makefile and various scripts to utilize this variable for Docker commands, enhancing cross-platform support.
@YCC3741 YCC3741 self-requested a review June 14, 2026 13:36
@YCC3741

YCC3741 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Doc note (suggestion, non-blocking): This PR adds a new DOCKER_PLATFORM override (defaulting to --platform linux/amd64) that affects the build/run workflow for arm64/aarch64 hosts. Per our READMERef/DOCUMENTATION_POLICY.md, workflow/option changes should be reflected in READMERef/, but no docs are updated here.

Could we add a short note to READMERef/RELEASE_AND_PUBLISH.md (which already documents make release-target / make run-release-target) describing the DOCKER_PLATFORM env var and the arm64/Apple Silicon use case? Happy to hear if you think an env-var-only addition does not warrant a doc change.

…ntation

Enhance READMERef/RELEASE_AND_PUBLISH.md with details on the DOCKER_PLATFORM variable for cross-platform compatibility, particularly for arm64/aarch64 hosts. Update READMERef/VALIDATE_FUZZ.md to reference the new Docker platform section, ensuring clarity on local run requirements.
@yu2C yu2C requested a review from YCC3741 June 14, 2026 15:58
@YCC3741 YCC3741 merged commit 6bc1fbd into main Jun 14, 2026
8 checks 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