Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: extrema - Extrema is a powerful visualization and data analysis tool https://bugzilla.redhat.com/show_bug.cgi?id=365171 adel.gadllah@xxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@xxxxxxxxxxxxxxxxx |adel.gadllah@xxxxxxxxx Flag| |fedora-review? ------- Additional Comments From adel.gadllah@xxxxxxxxx 2007-11-04 07:06 EST ------- Spec file looks ok. ------- md5sum is ok (3b6911525c1824ab50f82b6e72b05a19) ------- Package builds fine in mock (f7/x86_64). ------- rpmlint output: rpmlint extrema-help-4.2.10-1.fc7.x86_64.rpm extrema-help.x86_64: W: no-documentation (other packages none). (=ok) ------- Notes: License: GPLv2 Thats wrong the correct License is GPLv2+ : " Copyright (C) 2005 Joseph L. Chuma, TRIUMF This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version." (notice the 'any later version') ------ BuildRequires: desktop-file-utils /usr/bin/convert Please avoid file requires when possible, it causes yum to download the filelist. Use ImageMagick instead of /usr/bin/convert/ ------ The package runs fine, even thought it complains about missing help files on startup but I think thats ok. ------ The package is under development in the menu, but it isn't really a development tool. ------ ================== Please comment on/fix the above issues and I will approve this package. -- 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, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review