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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxx>  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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]