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
Postgrey #21586
Postgrey #21586
Conversation
The option renames will be painful for any existing users, but I admit that it makes slightly more sense, the way you did it. Hoping that I'm the only existing user, a tentative +1 from me. |
You could add entries like those in https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/rename.nix to smoothe things out. |
@bendlas are you okay with this? I have no stake in this module. |
LGTM, thanks for the compat options! |
Motivation for this change
The postgrey module was rather rudimentary
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)configuration.nix
does not showsocket
nicely, I worked around that by making sure the default and example cover all options available in both submodules – is there a prettier solution?