-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
RFC42 tracking issue #144575
Comments
I think some of the modules in the list are false positives, some already implement structural settings but provide |
With PR #139075 merged, we're going to have |
Logrotate can be checked as it was done in e92c053 |
NFS is in progress, #177601 |
Grafana is done. |
The big one is all the programs under the systemd project umbrella. It should be probably be done piecemeal but it requires a fair amount of work. |
This replaces the krb5 module's options with RFC 42-style krb5.settings option*, while greatly simplifying the code and fixing a few bugs, namely: - NixOS#243068 krb5: Configuration silently gets ignored when set by multiple modules - not being able to use mkIf etc. inside subattributes of krb5.libdefaults, e.g. krb5.libdefaults.default_realm = mkIf ... * leaving an escape hatch in form of krb5.extraConfig in case extra syntax is needed (e.g. `include' directives) See NixOS#144575. Closes NixOS#243068. Co-authored-by: h7x4 <h7x4@nani.wtf>
Now that #253428 got merged, the "programs/tsm-client.nix" checkbox may be ticked. |
This replaces the krb5 module's options with RFC 42-style krb5.settings option, while greatly simplifying the code and fixing a few bugs, namely: - NixOS#243068 krb5: Configuration silently gets ignored when set by multiple modules - not being able to use mkIf etc. inside subattributes of krb5.libdefaults, e.g. krb5.libdefaults.default_realm = mkIf ... See NixOS#144575. Closes NixOS#243068. Co-authored-by: h7x4 <h7x4@nani.wtf>
This replaces the krb5 module's options with RFC 42-style krb5.settings option*, while greatly simplifying the code and fixing a few bugs, namely: - NixOS#243068 krb5: Configuration silently gets ignored when set by multiple modules - not being able to use mkIf etc. inside subattributes of krb5.libdefaults, e.g. krb5.libdefaults.default_realm = mkIf ... * leaving an escape hatch in form of krb5.extraConfig in case extra syntax is needed (e.g. `include' directives) See NixOS#144575. Closes NixOS#243068.
virtualisation/xen-dom0.nix is partially done with #324911. Since the |
I'd like to mention that PHP (not the phpfpm module) also uses extraConfig. There it also would make sense to migrate to RFC42. |
RFC42 is the future, let's migrate to it
List generated with
grep -rn "extraConfig\|extraOptions" nixos/modules | grep "=" | sed -r "s|(.*.nix):.+|\1|g" | sed "s|nixos/modules/|- [ ] |g" | grep -v pl | grep -v xml | uniq
The text was updated successfully, but these errors were encountered: