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