Daniel,
On Feb 18 00:04, Corinna Vinschen wrote:
On Feb 17 23:51, Christian Franke wrote:
D. Boland wrote:
Hi Corinna,
Corinna Vinschen wrote:
Only two smaller problems:
- The mailq and newaliases symlinks in /usr/bin must not be part
of the package, otherwise they potentially overwrite an existing
configuration. They are created by sendmail-config anyway, if
the user chooses to run the script.
- Probably less important, but the other MTAs are installed into
/usr/sbin, not into /usr/libexec. Fedora installs the sendmail
binary as /usr/sbin/sendmail.sendmail, analog to postfix, which
is installed on Cygwin and Linux as /usr/sbin/sendmail.postfix.
But that's not that important, as long as there's no collision.
Other than that, I think the package is GTG.
Thanks for the sharp eye. I'll remove the links.
Another minor issue in the last line /etc/postinstall/sendmail.sh:
chown SYSTEM:Administrators $UBINDIR/sendmail-config
This does not work with all versions of Windows because the "Administrators"
(S-1-5-32-544) group name may be localized.
Indeed. That should use the gid 544 instead.
I removed the just uploaded 32 bit sendmail package for now.
For one thing, the 64 bit package is missing for some reason, the
other is that you didn't take the above into account. The
Administrators account is only called Administrators on certain
language versions of Windows. On other language versions, the
name "Administrators" is localized, for instance "Administratoren"
on german versions.
Please fix this before re-uploading.
Thanks,
Corinna