New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dockerTools: normalize arch to GOARCH #106703
Conversation
ab738f0
to
4d0a688
Compare
Tested and improvement suggested here terinjokes#1. @adisbladis could you have a look and confirm if these changes still work for you? |
Docker (via containerd) and the the OCI Image Configuration imply and suggest, respectfully, that the architecture set in images matches those of GOARCH in the Go Language document. This changeset updates the implimentation of getArch in dockerTools to return GOARCH values, to satisfy Docker. Fixes: NixOS#106695
Not everyone has a suitable remote builder set up, so the cross-compilation tests that _include_ running the result are separate. That way, most people can run the majority of the test suite without the extra setup.
4d0a688
to
5cacf0f
Compare
@ofborg build nixosTests.docker-tools nixosTests.docker-tools-cross |
Fair enough, ofborg doesn't have remote builders set up, but tests pass locally, ehm remotely. |
If GOARCH is taking on a significance beyond |
It does seems so. Taking it from a package is not a problem here. I'm pretty sure that container stuff is strict in |
Docker (via containerd) and the the OCI Image Configuration imply and
suggest, respectfully, that the architecture set in images matches those
of GOARCH in the Go Language document.
This changeset updates the implimentation of getArch in dockerTools to
return GOARCH values, to satisfy Docker.
Fixes: #106695
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)