This is the mail archive of the
cygwin-apps@cygwin.com
mailing list for the Cygwin project.
Re: [ITP] gmp-4.1.2-1
Lapo Luchini wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Charles Wilson wrote:
| 1. I'd prefer if the sdesc were a little more specific: it's a GPL
| library (e.g. free as in speech), not BSD or LGPL (free as in beer).
|
I'll think of something.. what about "GMP is a LGPL'ed library for
arbitrary precision arithmetic"?
GPL. not LGPL.
| 2. As far as the source packaging goes, it looks fine to me (built
| okay). You probably want to fix this:
|
| configure: WARNING: If you wanted to set the --build type, don't use
| --host. If a cross compiler is detected then cross compile mode will
| be used.
I already pointed out that as a "bug" of the generic install script..
nobody replyed <grin> 0=)
Sorry, I must have missed it. I'll fix it now...
| 6. Are you sure about the requires? I got a dependency on
| cygreadline5.dll -- for the demo programs which you don't
| ship...Might it be a good idea to ship some of those? (course, you
| have to install by hand -- and note rename of calc.exe)
Mhh.. well if I divide the package as you suggest later (and I could
definitely do it, I only have to study a little better the way it can be
automatized in a build script ^^) I guess a "-examples" package could
contain them.
BTW: is there any "not error-prone" procedure to find which build
packages are actually needed?
Other that uninstalling all and install them one but one, as required...
Not that I know of. I generally just let mine evolve until I get 'em right.
| 8. Any thoughts on --enable-cxx ? --enable-mpfr ?
Oops again, of course it is useful.
- --enable-cxx
- --enable-fft (just to be sure)
- --enable-mpbsd (I guess it doesn't do harm, and may be useful)
- --enable-mpfr
- --enable-shared
- --disable-static
I don't know if the C++ library will be all that simple to build
dynamically. You might choose to pass on that one; which is why I asked.
| 9. GMP uses very old-school libtool-1.4. I know it *works* -- but
| have you considered following the instructions in doc/configuration
| to update to the newer autotools ?
I noticed that it worked and produced a good DLL, so I didn't think it
was needed to upgrade them.
But I guess that a little upgrade can't do any bad (and could, of
course, do some good).
It will probably help if you run into any difficulties building the C++
library shared; prior to 1.5, libtool did not understand how to build
things other than C code.
| 10. Finally, you might think about splitting this into multiple
| packages -- it's better to do so at first, than go thru the "split
| hell" later. Unless you're pretty sure there will never be a
| cyggmp-4.dll.
|
| I'd suggest: libgmp3 - contains cyggmp-3.dll libgmp-devel -
| contains import, libtool lib, includes perhaps named gmp-devel,
| instead gmp - everything else: docs, info, demo progs, etc
|
I'll take a look at it.
'Kay.
--
Chuck