Bug 502856

Good:
+ Basename of the SPEC file matches with package name
+ Package name fullfill with naming guidelines
+ URL tag shows on proper project homepage
+ Package contains most recent release of the application
+ Could download upstream sources via spectool -g
+ Package sources matches with upstream
(md5sum: 0ede5640b1c6f0f2f513e992b60a261f)
+ Package contains valid License tag
+ License tags excalim GPLV3+ as an valid OSS license
+ Package contains verbatin copy of the license text
+ Consistently rpm macro usage
+ Package doesn't contains subpackages
+ Proper Buildroot defintion
+ Buildroot will be cleaned on beginning of %clean and %install
+ RPM_OPT_FLAGS will be honour on build step
+ Build step support smp enabled make
+ Local build works fine
+ No complaints from rpmlint for source rpm
+ No complaints from rpmlint for binary rpm
+ No complaints from rpmlint for debuginfo rpm
+ Debuginfo package contains sources
+ Local install/uninstall works fine
+ Short test of the application looks fine.
+ Files has proper files permissions
+ All package files are owned by the package
+ %files stanza contains no duplicated entries
+ %doc stanzs is small, so we don't need an extra subpackage
+  Package contains proper Changelog


Bad:
- Source header indicates GPLv3 as license instead of GPLv3+
  Please talk with upstream for clarification
- Scratch build failed on koji
  Please see: http://koji.fedoraproject.org/koji/taskinfo?taskID=1380090

(In reply to comment #1)
> Bad:
> - Source header indicates GPLv3 as license instead of GPLv3+
>   Please talk with upstream for clarification

Good catch. The license tag is from Leigh's spec. Should've checked it myself. Send an email upstream.

> - Scratch build failed on koji
>   Please see: http://koji.fedoraproject.org/koji/taskinfo?taskID=1380090  

Duh, should've checked this too :)


OK, missing BR and license have been fixed. I set the license to GPLv3 as it is more restrictive than GPLv3+, I can change it when upstream replies.

http://theory.physics.helsinki.fi/~jzlehtol/rpms/pdfchain.spec
http://theory.physics.helsinki.fi/~jzlehtol/rpms/pdfchain-0.99-3.fc11.src.rpm

Ok, now the koji scratch build works fine, so I can APPROVE your package.

Thanks for the speedy review!

New Package CVS Request
=======================
Package Name: pdfchain
Short Description: A GUI for pdftk
Owners: jussilehtola
Branches: F-10 F-11
InitialCC:

CVS done.

pdfchain-0.99-3.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/pdfchain-0.99-3.fc10

pdfchain-0.99-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/pdfchain-0.99-3.fc11

pdfchain-0.99-3.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update pdfchain'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-5576

pdfchain-0.99-3.fc11 has been pushed to the Fedora 11 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update pdfchain'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-5591

pdfchain-0.99-3.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

pdfchain-0.99-3.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.