[Bug 504430] Review Request: healpy - A python wrapper of the healpix library

bugzilla at redhat.com bugzilla at redhat.com
Tue Jun 9 05:48:47 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=504430





--- Comment #10 from Jussi Lehtola <jussi.lehtola at iki.fi>  2009-06-09 01:48:46 EDT ---
Well, this should be Release 4, since:

-1 was initial release
-2 was build against healpix-c++ and cfitsio instead of the bundled ones
-3 was removing bundled healpix and cfitsio altogether and changing duplicated
BR:s into R:s
-4 dropping the extra R:s

**

- You should probably
 %build
 export CFLAGS="%{optflags} -fopenmp"
to enable threading support.

- g++ flags are not used. This needs to be fixed.

- The package *does not contain any license statement at all*. It cannot go in
until you get upstream to put the license in the tarball, the order of
preference is
1. source code headers stating the file(s) are under GPLv2+
2. COPYING stating healpy is under GPLv2+
3. plain GPLv(N) COPYING file, which means that code is under GPL+.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list