Fix sendmail key#3261
Conversation
change key for additional_parameters argument fix: sendmail error sendmail: illegal option -- r sendmail: usage: sendmail [ -t ] [ -fsender ] [ -Fname ] [ -bp ] [ -bs ] [ arg ... ]
|
There's a bigger problem -- the flag differs based on the sendmail implementation. If you use postfix, sendmail is symlinked to postfix, and -r is correct. If you use sendmail itself, "-f". (The postfix sendmail compat script actually can use either -f or -r, which tells me we should use -f by default.) As such, we should likely specify the flag to use with the from address, as well as whether or not whitespace is allowed (postfix requires a space between the flag and the address; sendmail does not). The problem, of course, is figuring out what the most common options are. |
- "-r" is evidently a deprecated/obsolete synonym for "-f", and not present on current versions of sendmail (though still present on many clones, including postfix, exim, and qmail) - updated tests to refer to "-f" instead of "-r"
|
Actually, it looks like "-r"has been deprecated in sendmail for some time, in favor of "-f". Most MTAs with a sendmail compat layer allow both, but defaulting to "-f" is a sane change. |
- "-r" is evidently a deprecated/obsolete synonym for "-f", and not present on current versions of sendmail (though still present on many clones, including postfix, exim, and qmail) - updated tests to refer to "-f" instead of "-r"
change key for additional_parameters argument
fix:
sendmail error
sendmail: illegal option -- r
sendmail: usage: sendmail [ -t ] [ -fsender ] [ -Fname ] [ -bp ] [ -bs ] [ arg ... ]