'mail' from mailutils hardcodes /usr/sbin/sendmail

[bjornfor]

... rendering the 'mail' command not working on NixOS.

Last bug report for today :-)

[mogria]

Well which sendmail should be used then? sendmail is provided by busybox, exim, postfix & ssmtp.
Should the installed one be used?

[bjornfor]

@mogria: I'd go for $PATH lookup at runtime.

[peterhoeg]

Please leave this alone until tomorrow as I'm giving a talk on NixOS and will be fixing this as part of it.

[mogria]

i wasn't able to fix it, so go on :) On 6 Apr 2017 11:16, "Peter Hoeg" <[email protected]> wrote: Can I ask for this bug to be left alone until tomorrow as I'm giving a talk on NixOS and will be fixing this as part of it? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#21008 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAuDUKlGaHLSj0wfQ7OMyAgyvyyGX_kzks5rtK1TgaJpZM4LIWxs> .

[unode]

Any fix in the works @peterhoeg ? Just hit this seeing:

Cannot open mailer: No such file or directory
cannot send message: No such file or directory

[unode]

Also am I wrong in interpreting this as a regression of #3619 ?

[bjornfor]

Oh, I didn't know we previously had hardcoded the sendmail path. Apparently that was removed in 27a9ef8.

[bjornfor]

@vrthra: Do you remember why you removed sendmailPath from mailutils? (Commit message doesn't say.)

[bjornfor]

I asked upstream about looking up sendmail in $PATH: https://www.mail-archive.com/[email protected]/msg01740.html

[vrthra]

@bjornfor I think this was a mistake in my commit.

[unode]

I can confirm that manually applying the changes introduced in #3619 works around the Cannot open mailer problem.

Having postfix configured locally I then ran into a problem of /var/spool/mail/user being a file instead of a folder (mbox vs maildir format?) leading to failed deliveries.
Removing the location manually and re-sending a message to this user allowed postfix to recreate the location in the correct format, solving the issue.

[unode]

Updated patch https://gist.github.com/cc29b145ddd6f45ae5701b4d31796c8b

[bjornfor]

Fixed by b2b44c9.