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
zstd: add static option #72069
zstd: add static option #72069
Conversation
CMake is used for static builds because the Makefile does not offer a way to disable the shared library.
2bdfef2
to
4833b69
Compare
Isn't it possible to build both shared & static in one common multi-outputs derivation (with CMake) ; and then install static lib in "static" output ? |
The regular Makefile based build already builds The purpose of this PR is mostly to get |
@@ -59,4 +60,15 @@ stdenv.mkDerivation rec { | |||
platforms = platforms.unix; | |||
maintainers = with maintainers; [ orivej ]; | |||
}; | |||
} | |||
} // stdenv.lib.optionalAttrs static { | |||
nativeBuildInputs = [ cmake ]; |
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.
It would be good if we can use cmake on non-static as well.
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.
Done in the second commit.
superseded by #75798. |
CMake is used for static builds because the Makefile does not
offer a way to disable the shared library.
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @orivej