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=659082 Orcan Ogetbil <oget.fedora@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review? --- Comment #1 from Orcan Ogetbil <oget.fedora@xxxxxxxxx> 2011-01-04 21:48:20 EST --- Hi Thomas, here are my findings from my review. It needs a little work: * "rpmbuild -bs redland-bindings" fails with $ rpmbuild -bs redland-bindings.spec sh: php-config: command not found error: Macro %php_extdir has empty body sh: /usr/bin/ruby: No such file or directory sh: /usr/bin/ruby: No such file or directory Please fix these lines by using the macros as defined in http://fedoraproject.org/wiki/Packaging:PHP#Other_Modules http://fedoraproject.org/wiki/Packaging:Ruby#Ruby_packages_with_binary_content.2Fshared_libraries Otherwise these might cause an issue in koji. * Patches should be explained and be submitted to upstream. What is their status? * rpmlint says: python-redland.x86_64: W: private-shared-object-provides /usr/lib64/python2.7/site-packages/Redland.so Redland.so()(64bit) perl-redland.x86_64: W: private-shared-object-provides /usr/lib64/perl5/auto/RDF/Redland/CORE/CORE.so CORE.so()(64bit) perl-redland.x86_64: W: spurious-executable-perm /usr/share/doc/perl-redland-1.0.11.1/example.pl perl-redland.x86_64: W: doc-file-dependency /usr/share/doc/perl-redland-1.0.11.1/example.pl /usr/bin/perl php-redland.x86_64: W: unstripped-binary-or-object /usr/lib64/php/modules/redland.so php-redland.x86_64: W: spurious-executable-perm /usr/share/doc/php-redland-1.0.11.1/example.pl php-redland.x86_64: W: doc-file-dependency /usr/share/doc/php-redland-1.0.11.1/example.pl perl(RDF::Redland) php-redland.x86_64: W: doc-file-dependency /usr/share/doc/php-redland-1.0.11.1/example.pl /usr/bin/perl As far as I can tell, these all need to be fixed. * We prefer %global over %defined ! The .html files in the root directory of the tarball could be packaged (except INSTALL.html) * There are many files under perl/ and php/ directories, which have the license text: # This package is Free Software or Open Source available under the # following licenses (these are alternatives): # 1. GNU Lesser General Public License (LGPL) # 2. GNU General Public License (GPL) # 3. Mozilla Public License (MPL) These do not state any license versions. It would be good to ask upstream. On the other hand, for example, python/RDF.py says that it is LGPLv2+ or GPLv2+ or ASL 2.0 or MIT, which makes the licensing scenario even more complicated. Please go through all the source files and document their respective licenses. ! The files AUTHORS COPYING COPYING.LIB ChangeLog LICENSE.txt NEWS README LICENSE-2.0.txt NOTICE are being duplicated. This is okay with license files. However I'm not sure about the other ones. Maybe put everything in the main package and make all the subpackages require it? * The subpackages have Requires: redland = %{redland_version} which won't be satisfied with the current redland version we have in F-14+. Don't we need >= * Parallel make must be supported whenever possible. If it is not supported, this should be noted in the SPEC file as a comment. * The following BuildRequires seem redundant to me: perl php openssl-devel mysql-devel postgresql-devel sqlite-devel db4-devel * Python guidelines are not obeyed: - Need to use BuildRequires: python2-devel - The macros do not match the guidelines. Please see http://fedoraproject.org/wiki/Packaging:Python * Ruby package must indicate the Ruby ABI version it depends on. Please see http://fedoraproject.org/wiki/Packaging:Ruby ? What is this for: /usr/lib/rpm/brp-python-bytecompile -- 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