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

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat