-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
nsd: 4.1.15 -> 4.1.16 #26002
nsd: 4.1.15 -> 4.1.16 #26002
Conversation
pkgs/servers/dns/nsd/default.nix
Outdated
''; | ||
|
||
buildInputs = [ libevent ]; | ||
propagatedBuildInputs = [ openssl ]; |
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.
If you're already replacing the openssl path in the shell script calling it, putting it in propagatedBuildInputs
is not necessary except there are more scripts/tools depending on openssl being present.
pkgs/servers/dns/nsd/default.nix
Outdated
}; | ||
|
||
buildInputs = [ libevent openssl ]; | ||
patchPhase = '' |
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.
You should use prePatch
or postPatch
here so you can still pass patches
to the mkDerivation
call which is done in the default patchPhase
that you're replacing here.
pkgs/servers/dns/nsd/default.nix
Outdated
buildInputs = [ libevent openssl ]; | ||
patchPhase = '' | ||
substituteInPlace nsd-control-setup.sh.in --replace openssl ${openssl}/bin/openssl | ||
''; |
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.
Minor indention nitpick. 2 spaces to the left please. :)
Fixed comments by @fpletz (thanks by the way)! |
Built and tested locally. Thank you for your contribution! |
Motivation for this change
Minor version update. Plus (in a second commit) a fix to nsd-control-setup. This convenience script was dependent on openssl being already installed, now it is a propagatedBuildInput.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)