-
-
Notifications
You must be signed in to change notification settings - Fork 13.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
nixos/dnsmasq: remove deprecated option "extraConfig" #335957
Conversation
e975116
to
2eccb4b
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.
Makes sense to me, I had a quick look at discourse in case anyone had popped up saying some odd configuration can't be represented, wrt deprecation. Also I can't think of any, given it's just command line flags, and we have a way of repeating command line flags (i.e. lists).
I can't merge, and honestly I'm sadly not very active in the NixOS community/development to give good code review, but LGTM.
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.
Looks good. But this change needs release notes and a mkRemovedOptionModule
definition to inform users.
2eccb4b
to
b9f607b
Compare
b9f607b
to
d22b374
Compare
fa5b2a2
to
019ab7a
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.
Apart from the wording, LGTM
019ab7a
to
b686e69
Compare
8ec70bc
to
9155fb6
Compare
9155fb6
to
bd326c2
Compare
@fpletz: That's done now. |
Release note and mkRemovedOptionModule added now.
Description of changes
Remove deprecated option
nixpkgs/nixos/doc/manual/release-notes/rl-2305.section.md
Lines 417 to 420 in f02fa2f
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.