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] libungif-4.1.0-1


Charles Wilson wrote:

> Okay, while I think this package should be added to the distro, I
> really dislike the way it has been ported (sorry Lapo).
>
> Steven's 'mkdll.sh' script is just plain wrong. libungif is a
> libtoolized package, you should just relibtoolize.
>
> I've had I version of libungif around for a while -- it's here:
> http://www.neuro.gatech.edu/users/cwilson/cygutils/testing/libungif/
> I posted it some time ago when Harold thought he had more time -- I
> thought you were basing your version on that, rather than using
> Steven's mkdll.sh.
>
> BTW, you can also relibtoolize glib (and probably gtk) in a similar
> way. Did I mention that I *really* dislike that mkdll.sh script?

Lapo,

I have to agree with Chuck, using a script is not the way to go. You should try to get the package working within the constructs provided. Using helper scripts can (and often will) cause problems when (and if) the package releases a new version. It is better to release a well tuned and fully compatible package then to hastily release one based on (no offense) kludges. If you run into any problems, post to the main list and I'm sure one of us, who has seen it before, will be able to help you fix it. As for the descriptions, I usually just download the SRPM of the package I am building from RedHat. I then use the short and long description from the spec file as my basis. Plus, as an added bonus, you can find nice patches for security and functionality that the
good folks at RedHat have provided.

Cheers,
Nicholas



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