This is the mail archive of the
cygwin-apps@cygwin.com
mailing list for the Cygwin project.
Re: [ITP] email-2.3.0 (review)
On Thu, Oct 21, 2004 at 01:12:55PM -0800, Ross Smith II wrote:
>Corinna Vinschen wrote:
>
>
>> On Oct 20 18:32, Ross Smith II wrote:
>> I don't understand this one. Exim as well as ssmtp both have a config
>> script which sets /usr/sbin/sendmail so that it points to the real
>> executable, if /usr/sbin/sendmail doesn't exists.
>>
>> Wouldn't it be better to use /usr/sbin/sendmail as default and either
>> to complain if /usr/sbin/sendmail doesn't exist?
>
>You're right. It now defaults to /usr/sbin/sendmail. I have uploaded the
>corrected files (though I didn't bump the release number):
>
>http://smithii.com/files/cygwin/email/email-2.3.0-2.tar.bz2
>http://smithii.com/files/cygwin/email/email-2.3.0-2-src.tar.bz2
>http://smithii.com/files/cygwin/email/setup.hint
>
>Thanks for all the help. This was my first release, and I learned alot.
I've uploaded this but I made a few changes to the setup.hint file.
1) I removed the extraneous '@ chere' from the beginning. This field should
only be used when absolutely necessary and that is only when the name of
the package doesn't match the name of the package directory. This field
is only very rarely needed.
2) I removed the "curr" tag. There is no reason to use it unless you are
trying to subvert the way that upset would normally handle the version
numbering. Usually, the decisions that upset makes are correct.
3) Please don't indent the "ldesc" field. Although nothing displays this
currently, indenting the text to some arbitrary level doesn't do anything.
4) The setup.hint file should not have CRLF line endings.
Other than these minor issues, the package is fine, AFAICT.
Please send a release announcement ASAP. Use a recent release
announcement as a guide. Make sure that you include unsubscribe
instructions.
Thanks,
cgf