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/ssmtp: add settings option #89662
Conversation
ping @sjau for testing and approval. Usage as follows:
|
For me it's fine. I just wanted to be able to also debug if necessary :) |
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.
Nice. Apart form two minor this seems fine to me :)
…es.ssmtp.settings.AuthPass if absolutely required
@Ma27 after the bot finishes I'll look to you for final approval and merge. Thank you! |
Motivation for this change
#89594
I'm not sure if I'm currently running
ssmtp
on any machines so I didn't test the execution of the program, though I did validatessmtp.conf
is equivalent before and after this change with a few example configurations.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)