Skip to content
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

msmtp: hotfix for broken queueing in 1.8.8 #85846

wants to merge 1 commit into from


Copy link

peterhoeg commented Apr 23, 2020

1.8.8 breaks msmtp-queue causing mails to be continuously resent as they are not removed from the queue.

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits
Copy link

FRidh commented Apr 23, 2020

Add a comment in the code along with a long to an upstream issue.

@peterhoeg peterhoeg force-pushed the peterhoeg:d/msmtp branch from afa9b5d to dd80359 Apr 23, 2020
@peterhoeg peterhoeg changed the title Revert "msmtp: 1.8.7 -> 1.8.8" msmtp: hotfix for broken queueing in 1.8.8 Apr 23, 2020
rycee added a commit that referenced this pull request Apr 23, 2020
PR #85846
Copy link

rycee commented Apr 23, 2020

Thanks for the quick discovery and fix! I've tried it out a bit and it Works For Me™. Rebased to master in 920c51e.

@rycee rycee closed this Apr 23, 2020
@peterhoeg peterhoeg deleted the peterhoeg:d/msmtp branch Apr 23, 2020
dtzWill added a commit to dtzWill/nixpkgs that referenced this pull request Jun 5, 2020
PR NixOS#85846

(cherry picked from commit 920c51e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.