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=481508 --- Comment #2 from Fabian Affolter <fabian@xxxxxxxxxxxxxxxxx> 2009-01-27 15:51:43 EDT --- Marcela, thanks for the review. (In reply to comment #1) > FAIL specfile is properly named, is cleanly written and uses macros > consistently. > - RPM_BUILD_ROOT vs. buildroot - use only one of these macros a stupid copy-and-paste-error, fixed > FAIL license field matches the actual license. > - You can find GPLv3+ in COPYING and PKG-INFO fixed Updated files: Spec URL: http://fab.fedorapeople.org/packages/SRPMS/pipviewer.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/pipviewer-0.3.9-2.fc10.src.rpm -- 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