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
DNSChain: fix broken package and improve service #22041
Conversation
''; | ||
|
||
dns.address = mkOption { | ||
type = types.string; |
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.
Probably should be types.str
unless you want the peculiar merge semantics of string
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.
Right, I meant to use str
.
|
||
let | ||
np = nodePackages.override { generated = ./package.nix; self = np; }; | ||
in | ||
nodePackages = callPackage (import ../../top-level/node-packages.nix) { |
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.
What is the motivation for the move to this construction instead of taking the package set as a parameter? Does override not work?
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.
Yes, it produces the error:
called with unexpected argument ‘self’, at <nixpkgs>/lib/customisation.nix:56:12
I looked how other nodejs packages handle this and switched to use callPackage
.
Looks like there is an error building this:
|
Strange, it builds for me. |
Motivation for this change
See #21999
Things done