This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ITP] email-2.3.0 (review)


Sorry for all those glitches. I just took another setup.hint and modified it. I must not have grabbed a "nice" one.

Thanks again for all the help,

Ross

Christopher Faylor wrote:
> 
> 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.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]