We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
It seems https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/network-filesystems/samba.nix was changed after 24.05 to respect https://github.com/NixOS/rfcs/blob/master/rfcs/0042-config-option.md. But some smb.conf options have a dependency of order (include=...). This option (as per doc) "[...] allows you to include one config file inside another. The file is included literally, as though typed in place." There doesn't seem to be a way to order parameters anymore. Looks like parameters are just ordered lexically now.
services.samba = { enable = true; settings = { global = { security = "user"; include = "/etc/smb2.conf"; }; }; };
gives
[global] include = /etc/smb2.conf security = "user";
[global] security = "user"; include=/etc/smb2.conf
@anthonyroussel @bachp
Note for maintainers: Please tag this issue in your PR.
Add a 👍 reaction to issues you find important.
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Describe the bug
It seems https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/network-filesystems/samba.nix was changed after 24.05 to respect https://github.com/NixOS/rfcs/blob/master/rfcs/0042-config-option.md. But some smb.conf options have a dependency of order (include=...). This option (as per doc) "[...] allows you to include one config file inside another. The file is included literally, as though typed in place." There doesn't seem to be a way to order parameters anymore. Looks like parameters are just ordered lexically now.
Steps To Reproduce
gives
Expected behavior
Notify maintainers
@anthonyroussel @bachp
Note for maintainers: Please tag this issue in your PR.
Add a 👍 reaction to issues you find important.
The text was updated successfully, but these errors were encountered: