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=504489 Orcan 'oget' Ogetbil <oget.fedora@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |oget.fedora@xxxxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |oget.fedora@xxxxxxxxx Flag| |fedora-review? --- Comment #1 from Orcan 'oget' Ogetbil <oget.fedora@xxxxxxxxx> 2009-06-07 18:08:26 EDT --- Here are my notes for this package: - rpmlint is silent. - koji rawhide build seems fine http://koji.fedoraproject.org/koji/taskinfo?taskID=1397983 ! Please explain in the specfile as comments what Sources 1-3 are for. ! Not a blocker, but in the docs/ and examples/ directories, there are html, xml and pro files that refer to /usr/local/. You might want to fix them. ? Any reason why you don't put the jar files directly in /usr/share/java/ ? If you definitely need to put the jar files in /usr/share/java/proguard/ , can you replace %{_javadir}/%{name}* with %{_javadir}/%{name}/ in %files to indicate that this is a directory? * If a package contains a GUI application, then it needs to also include a properly installed .desktop file. Please follow http://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files and http://fedoraproject.org/wiki/Packaging/ScriptletSnippets ! You need to specify a specific java version in BR and R. See: http://fedoraproject.org/wiki/Packaging/Java#BuildRequires_and_Requires In your case this ought to be 1.5 * GCJ AOT bits SHOULD be built and included in packages. Since this package builds with java 1.5, this will bring great performance improvements on ppc* architectures. Please follow: http://fedoraproject.org/wiki/Packaging/GCJGuidelines Can you review my package (bug #504521 )? It is java too. -- 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