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
criu: 2.0 -> 2.12.1 #30408
criu: 2.0 -> 2.12.1 #30408
Conversation
substituteInPlace ./criu/Makefile --replace "-I/usr/include/libnl3" "-I${libnl.dev}/include/libnl3" | ||
substituteInPlace ./Makefile --replace "tar-name := $(shell git tag -l v$(CRIU_VERSION))" "tar-name = 2.0" # --replace "-Werror" "" | ||
substituteInPlace ./Makefile --replace "head-name := \$(shell git tag -l v\$(CRIU_VERSION))" "head-name = ${version}.0" # --replace "-Werror" "" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder, why not hard-code this to ${version}
instead?
Needs rebasing |
}; | ||
|
||
enableParallelBuilding = true; | ||
nativeBuildInputs = [ pkgconfig docbook_xsl ]; | ||
buildInputs = [ protobuf protobufc asciidoc xmlto libpaper libnl libcap python ]; | ||
buildInputs = [ protobuf protobufc asciidoc xmlto libpaper libnl libcap libnet python git ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can tell the git here is not actually used, presumably because of the substitution in the patch phase… At least I was able to build it OK without git and
nix-store -q --references /nix/store/609y5nypwvwmm48ivj3lq7vrpp3px7fm-criu-2.12.1
doesn't show any reference to the git package.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I see now, I do actually substituteInPlace the git call from Makefile so it's not needed. Will patch that.
Thanks! Rebased to master in 22beeea. |
Motivation for this change
criu: 2.0 -> 2.12.1
No luck with upgrading to 3.x
cc @thoughtpolice
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)