Spec URL: http://oget.fedorapeople.org/review/jorbis.spec SRPM URL: http://oget.fedorapeople.org/review/jorbis-0.0.17-1.fc10.src.rpm Description: JOrbis is a pure Java Ogg Vorbis decoder. JOrbis accepts Ogg Vorbis bitstreams and decodes them to raw PCMs. There are these kind of rpmlints: jorbis.src: W: libdir-macro-in-noarch-package which can be ignored since the package is not noarch by default
Fedora review jorbis-0.0.17-1.fc10.src.rpm 2009-04-29 rpmlint output: [ellert@ellert jorbis]$ rpmlint 1256512409979681881233819/result/jorbis/*.rpm jorbis.spec jorbis.src:156: W: libdir-macro-in-noarch-package (main package) %dir %{_libdir}/gcj/%{name} jorbis.src:157: W: libdir-macro-in-noarch-package (main package) %{_libdir}/gcj/%{name}/%{name}-%{version}.jar.* jorbis.src:170: W: libdir-macro-in-noarch-package player %{_libdir}/gcj/%{name}/%{name}-player*.jar.* jorbis.src:178: W: libdir-macro-in-noarch-package comment %{_libdir}/gcj/%{name}/%{name}-comment*.jar.* jorbis.spec:156: W: libdir-macro-in-noarch-package (main package) %dir %{_libdir}/gcj/%{name} jorbis.spec:157: W: libdir-macro-in-noarch-package (main package) %{_libdir}/gcj/%{name}/%{name}-%{version}.jar.* jorbis.spec:170: W: libdir-macro-in-noarch-package player %{_libdir}/gcj/%{name}/%{name}-player*.jar.* jorbis.spec:178: W: libdir-macro-in-noarch-package comment %{_libdir}/gcj/%{name}/%{name}-comment*.jar.* 6 packages and 1 specfiles checked; 0 errors, 8 warnings. * OK ! needs attention * rpmlint warnings are bogus - rpmlint does not understand the conditional BuildArch. * Package is named according to the guidelines * The specfile is named after the package * The package follows the guidelines for a native Java package * Package license is Fedora approved (LGPLv2+) * The README file says that the license is LGPL with no version, but that is equivalent to LGPLv2+ according to the Fedora License list * The license file (COPYING.LIB) in the sources is installed as %doc * The specfile is written in legible English * Sources matches upstream: 6141edfb16d34412a86c95a97fd4b10f jorbis-0.0.17.zip 6141edfb16d34412a86c95a97fd4b10f SRPM/jorbis-0.0.17.zip * Package builds in mock (Fedora 10) * BuildRequires are sane * No shared libraries * Package owns the directories it creates ! The jars in -comment and -player are also in main (duplicate files) * Permissions are sane and %files has %defattr * %clean clears buildroot * Consistent use of macros * Documentation in -javadoc package * %doc is not runtime essential * subpackages require main with fully versioned dependencies * Package does not own other's directories * %install clears buildroot * Installed filenames are valid UTF-8
Ah, * is greedy :) Spec URL: http://oget.fedorapeople.org/review/jorbis.spec SRPM URL: http://oget.fedorapeople.org/review/jorbis-0.0.17-2.fc10.src.rpm Changelog: 0.0.17-2 - Fix duplicate files issue
Package approved.
Thank you for the review! New Package CVS Request ======================= Package Name: jorbis Short Description: Pure Java Ogg Vorbis Decoder Owners: oget Branches: F-9 F-10 F-11 InitialCC:
CVS Done
jorbis-0.0.17-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/jorbis-0.0.17-2.fc11
jorbis-0.0.17-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/jorbis-0.0.17-2.fc10
jorbis-0.0.17-2.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/jorbis-0.0.17-2.fc9
jorbis-0.0.17-2.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.
jorbis-0.0.17-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
jorbis-0.0.17-2.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.