-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
nixos/services.mosquitto: remove with lib;
#338022
nixos/services.mosquitto: remove with lib;
#338022
Conversation
${cfg.package}/bin/mosquitto_passwd -U "$file" | ||
'' | ||
++ hashedLines)); | ||
|
||
authPluginOptions = with types; submodule { | ||
authPluginOptions = with lib.types; submodule { |
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.
watch out, under this with
there's no automatic checking.
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.
is your suggestion to get rid of this with
statement 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.
No, no suggestion as such.
I've found that when I make errors, it's in the following conditions:
nix-instantiate --parse
succeeds- Because there's a symbol that was previously looked up in the top-level with
- That now is "looked up" in the lower with
- And there's no test of the NixOS module that would exercise the code.
nixosTests.mosquitto
exists, so I'll definitely run that before merging. But the other "watch out" signals are there.
e255741
to
481df95
Compare
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.
Running nix-build -A nixosTests.mosquitto
now
Description of changes
part of #208242
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.