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=573294 --- Comment #11 from Radek L <radoslaw.lisowski@xxxxxxxxx> 2010-03-28 12:20:39 EDT --- Ok, I've checked whole spec file, and fixed things that needed it. URL to plus_sms module is now correct, added License tags to modules, that haven't got one. (Found them in source files of the modules). Updated descriptions and summaries as Dominik noticed, and removed those if-endif clauses. Everything builds on fc12, couldn't test it on koji, couldn't even setup enviroment by using fedora-packager-setup. Receiving following error: $ fedora-packager-setup Setting up Fedora packager environment Traceback (most recent call last): File "/usr/bin/fedora-packager-setup", line 140, in <module> main() File "/usr/bin/fedora-packager-setup", line 114, in main if certificate_expired(): NameError: global name 'certificate_expired' is not defined Here are SPEC and SRPM files: http://ds.pk.edu.pl/~radekl/kadu/kadu.spec http://ds.pk.edu.pl/~radekl/kadu/kadu-0.6.5.4-3.fc12.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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review