-
-
Notifications
You must be signed in to change notification settings - Fork 14.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
technititum-dns-server: 13.2 -> 13.2.2 #363529
technititum-dns-server: 13.2 -> 13.2.2 #363529
Conversation
Co-authored-by: Sandro <[email protected]>
@SuperSandro2000 Thank you for your review! I've applied all changes, but now the build is failing. Perhaps because of
? Any suggestions what to edit/add, please? |
Yes, you need to pass in the package at the top so callPackage picks it up but then you might have an infinite recursion.. Which would speak for merging the packages again.. And then create an update script that updates both |
OK. I'd like to wait if @SuperSandro2000 has an idea how to solve this elegantly. If not, I'd close this PR and probably go back to joining the two packages again. :-( |
I asked in matrix, there doesn't seem to be a way. You can join the packages again and I will try to create an update script that covers both |
Update Technitium DNS Server and its library to 13.2.2.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.