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
[WIP] docker, dockerTools: Unpin Go 1.9 (remove Go from Docker closure) #41435
Conversation
2f9efce
to
f2624b4
Compare
It looks like |
@GrahamcOfBorg test docker-tools docker |
Yes, I think we should still attempt to use the same versions that upstream does. |
Success on aarch64-linux Attempted: tests.docker-tools, tests.docker No partial log is available. |
Failure on x86_64-linux (full log) Attempted: tests.docker-tools, tests.docker Partial log (click to expand)
|
f2624b4
to
eba064a
Compare
eba064a
to
ce7dd85
Compare
I am figuring-out why: { go ? null, go_1_9 ? null }: # <---
rec {
dockerGen = { ... }:
let
docker-containerd = containerd.overrideAttrs (oldAttrs: rec {
go = (if versionAtLeast version "18.05.5-ce" then go else go_1_9); # <---
buildInputs = [ removeReferencesTo go ];
});
in
stdenv.mkDerivation ( rec {
inherit docker-containerd;
buildInputs = [ removeReferences go ];
});
docker_18_03 = dockerGen rec { ... };
docker_18_05 = dockerGen rec { ... };
} Makes Go to come back. |
So the main question is: how to manage package version override, so that Maybe the real question is formulated backwards. It is a quirk, and we need to change |
You don't have to introduce
|
Putting it in
Putting it in
|
Since |
Closed in favor of: #41849 |
Motivation for this change
Go would not longer going to be present in the Docker closure.
Fixes:
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)