Spec URL: http://www.cora.nwra.com/~orion/fedora/jilter.spec SRPM URL: http://www.cora.nwra.com/~orion/fedora/jilter-1.2-1.fc11.src.rpm Description: Sendmail-Jilter is an Open Source implementation of the Sendmail milter protocol, for implementing milters in Java that can interface with the Sendmail MTA.
Fails to build for me on rawhide: runtests: BUILD FAILED /builddir/build/BUILD/jilter-1.2/build.xml:56: Problem: failed to create task or type junit Cause: the class org.apache.tools.ant.taskdefs.optional.junit.JUnitTask was not found. This looks like one of Ant's optional components. Action: Check that the appropriate optional JAR exists in -/usr/share/ant/lib -/builddir/.ant/lib -a directory added on the command line with the -lib argument Do not panic, this is a common problem. The commonest cause is a missing JAR. This is not a bug; it is a configuration problem Total time: 2 seconds A scratch build showing the failure: http://koji.fedoraproject.org/koji/taskinfo?taskID=2574543 Please clear the Whiteboard if providing a package that builds.
http://www.cora.nwra.com/~orion/fedora/jilter.spec http://www.cora.nwra.com/~orion/fedora/jilter-1.2-2.fc14.src.rpm * Wed Nov 3 2010 Orion Poplawski <orion.com> - 1.2-2 - Add ant-junit to BR - Add log4j to Requires - Build and ship javadoc
I don't normally review Java packages since I know very little about them. However, this has been sitting around for so long and the Java SIG folks are busy, so even though there's no possibility for me to test this, I'll review it. Builds fine (on F14 since rawhide is hosed today), rpmlint is silent. There's some stuff that's unnecessary on modern Fedora releases (BuildRoot:, buildroot cleaning in %install, the %clean section) which you can remove if you like. Of course, you can't if you intend to target EPEL with the same spec. Otherwise, I see no issues. * source files match upstream. sha256sum: a0f367f1103208e767b3e32e981ee4cdcc62f998cef1c455c11ac093288275b9 jilter-1.2-src.tar.gz * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * summary is OK. * description is OK. * dist tag is present. * license field matches the actual license. * license is open source-compatible. * license text included in package. * latest version is being packaged. * BuildRequires are proper. * package builds in mock (f14, x86_64). * package installs properly. * rpmlint is silent. * final provides and requires are sane: jilter-1.2-2.fc14.noarch.rpm jilter = 1.2-2.fc14 = java jpackage-utils log4j jilter-javadoc-1.2-2.fc14.noarch.rpm jilter-javadoc = 1.2-2.fc14 = jpackage-utils * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files (except for the license text) * file permissions are appropriate. * code, not content. * %docs are not necessary for the proper functioning of the package. Java-specific bits: * no pre-built jars * single jar, named after the package * jarfiles are under _javadir. * javadocs are under _javadocdir. * ant called properly. * no wrapper script necessary. * gcj not called. APPROVED
Thanks Jason! New Package SCM Request ======================= Package Name: jilter Short Description: Sendmail milter protocol for Java Owners: orion Branches: f14 f13 el6 el5 InitialCC:
Git done (by process-git-requests).
Checked in and built. Thanks!