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=472848 --- Comment #8 from Dominik 'Rathann' Mierzejewski <rpm@xxxxxxxxxxxxxx> 2008-11-26 21:17:40 EDT --- Full review, relevant items only (OK'd items omitted). MUST Items: - MUST: rpmlint must be run on every package. The output should be posted in the review. rpmlint output clean - MUST: The package must be named according to the Package Naming Guidelines . The project is named "jeuclid". Why is the package named jeuclid-core? The Java packaging guidelines (https://fedoraproject.org/wiki/Packaging/Java) say that you should follow the project's name and provide a symlink if the commonly used jar name is different. - MUST: The package must meet the Packaging Guidelines . Group: Applications/Text License: ASL 2.0 The above lines have trailing whitespace at the end. # this patch points the ant to the correct jars Patch0: jeuclid-core-build.patch You could use build-classpath or build-jar-repository instead of the above patch. # this patch removes FreeHep support as per the build README Patch1: jeuclid-core-FreeHep.patch You don't have to say "this patch", it's redundant. What you should say, instead, is whether or not the patches have been sent upstream (and if not, why). - MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines . The following files have no licensing information, please ask upstream to provide it (not a blocker). $ licensecheck.pl -r . | grep -v Apache ./jeuclid-core/src/test/java/net/sourceforge/jeuclid/test/LayoutTest.java: *No copyright* UNKNOWN ./jeuclid-core/src/test/java/net/sourceforge/jeuclid/test/ViewerTest.java: *No copyright* UNKNOWN ./jeuclid-core/src/test/java/net/sourceforge/jeuclid/test/ConverterTest.java: *No copyright* UNKNOWN ./jeuclid-core/src/test/java/net/sourceforge/jeuclid/test/DOMBuilderTest.java: *No copyright* UNKNOWN The rest seems to be under ASL 2.0, thus OK. - MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. Reviewers should use md5sum for this task. If no upstream URL can be specified for this package, please see the Source URL Guidelines for how to deal with this. OK: ff3690e649bf0ead5fd2a03c732dc1ce jeuclid-parent-3.1.3-src.zip - MUST: A package must own all directories that it creates. If it does not create a directory that it uses, then it should require a package which does create that directory. Refer to the Guidelines for examples. You should use %{_javadir}/%{name}-%{version}.jar instead of %{_javadir}/* in %files section. It'll let you detect stray files if they ever make their way into buildroot in some future release. -- 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