Spec URL: http://akurtakov.fedorapeople.org/slf4j.spec SRPM URL: http://akurtakov.fedorapeople.org/slf4j-1.5.8-2.fc11.src.rpm Description: The Simple Logging Facade for Java or (SLF4J) is intended to serve as a simple facade for various logging APIs allowing to the end-user to plug in the desired implementation at deployment time. SLF4J also allows for a gradual migration path away from Jakarta Commons Logging (JCL).
- package builds and installs fine on x86 X please make the description for the javadoc package "API documentation for %{name}" X many of the lines are > 80 characters; please fix X I think we need some Requires on java and jpackage-utils - licensing good (some files are missing license headers but I'm willing to accept [1]. we should perhaps press to get these added in git master) - rpmlint clean (except ignorable maven thing): $ rpmlint /home/overholt/rpmbuild/SRPMS/slf4j-1.5.8-2.fc11.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint /home/overholt/rpmbuild/RPMS/noarch/slf4j-1.5.8-2.fc11.noarch.rpm slf4j.noarch: W: non-conffile-in-etc /etc/maven/fragments/slf4j 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint /home/overholt/rpmbuild/RPMS/noarch/slf4j-javadoc-1.5.8-2.fc11.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint /home/overholt/rpmbuild/RPMS/noarch/slf4j-manual-1.5.8-2.fc11.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. - macros and clean sections present and fine - sources match upstream (md5sum) X please move to (-,root,root,-) for the %files %defattrs [1] http://marc.info/?l=slf4j-user&m=119773193423413&w=2
(In reply to comment #1) > - package builds and installs fine on x86 > X please make the description for the javadoc package "API documentation for > %{name}" Done. > X many of the lines are > 80 characters; please fix Done wherever possible > X I think we need some Requires on java and jpackage-utils Done > - licensing good (some files are missing license headers but I'm willing to > accept [1]. we should perhaps press to get these added in git master) > - rpmlint clean (except ignorable maven thing): > > $ rpmlint /home/overholt/rpmbuild/SRPMS/slf4j-1.5.8-2.fc11.src.rpm > 1 packages and 0 specfiles checked; 0 errors, 0 warnings. > > $ rpmlint /home/overholt/rpmbuild/RPMS/noarch/slf4j-1.5.8-2.fc11.noarch.rpm > slf4j.noarch: W: non-conffile-in-etc /etc/maven/fragments/slf4j > 1 packages and 0 specfiles checked; 0 errors, 1 warnings. > > $ rpmlint > /home/overholt/rpmbuild/RPMS/noarch/slf4j-javadoc-1.5.8-2.fc11.noarch.rpm > 1 packages and 0 specfiles checked; 0 errors, 0 warnings. > > $ rpmlint > /home/overholt/rpmbuild/RPMS/noarch/slf4j-manual-1.5.8-2.fc11.noarch.rpm > 1 packages and 0 specfiles checked; 0 errors, 0 warnings. > > - macros and clean sections present and fine > - sources match upstream (md5sum) > X please move to (-,root,root,-) for the %files %defattrs Done > > [1] > http://marc.info/?l=slf4j-user&m=119773193423413&w=2 New sources: Spec URL: http://akurtakov.fedorapeople.org/slf4j.spec SRPM URL: http://akurtakov.fedorapeople.org/slf4j-1.5.8-2.fc11.src.rpm
(In reply to comment #1) > - package builds and installs fine on x86 > X please make the description for the javadoc package "API documentation for > %{name}" Done. > X many of the lines are > 80 characters; please fix Done wherever possible > X I think we need some Requires on java and jpackage-utils Done > - licensing good (some files are missing license headers but I'm willing to > accept [1]. we should perhaps press to get these added in git master) > - rpmlint clean (except ignorable maven thing): > > $ rpmlint /home/overholt/rpmbuild/SRPMS/slf4j-1.5.8-2.fc11.src.rpm > 1 packages and 0 specfiles checked; 0 errors, 0 warnings. > > $ rpmlint /home/overholt/rpmbuild/RPMS/noarch/slf4j-1.5.8-2.fc11.noarch.rpm > slf4j.noarch: W: non-conffile-in-etc /etc/maven/fragments/slf4j > 1 packages and 0 specfiles checked; 0 errors, 1 warnings. > > $ rpmlint > /home/overholt/rpmbuild/RPMS/noarch/slf4j-javadoc-1.5.8-2.fc11.noarch.rpm > 1 packages and 0 specfiles checked; 0 errors, 0 warnings. > > $ rpmlint > /home/overholt/rpmbuild/RPMS/noarch/slf4j-manual-1.5.8-2.fc11.noarch.rpm > 1 packages and 0 specfiles checked; 0 errors, 0 warnings. > > - macros and clean sections present and fine > - sources match upstream (md5sum) > X please move to (-,root,root,-) for the %files %defattrs Done > > [1] > http://marc.info/?l=slf4j-user&m=119773193423413&w=2 New sources: Spec URL: http://akurtakov.fedorapeople.org/slf4j.spec SRPM URL: http://akurtakov.fedorapeople.org/slf4j-1.5.8-3.fc11.src.rpm
Thanks. There are still some lines that could be shortened (104-108, 115-118, 120, 167. Otherwise, we're good to go.
(In reply to comment #4) > Thanks. There are still some lines that could be shortened (104-108, 115-118, > 120, 167. Otherwise, we're good to go. Done New sources: Spec URL: http://akurtakov.fedorapeople.org/slf4j.spec SRPM URL: http://akurtakov.fedorapeople.org/slf4j-1.5.8-4.fc11.src.rpm
104-112 could still be shortened but now we're just getting silly :) Fix if you'd like once it's in CVS. Approved.
New Package CVS Request ======================= Package Name: slf4j Short Description: Simple Logging Facade for Java Owners: akurtakov Branches: InitialCC:
CVS Done
Build in rawhide.
Any reason not to build this for F-11?
(In reply to comment #10) > Any reason not to build this for F-11? It requires maven >= that which is in F-11. A back-port of all the maven packages and dependencies would be a lot of work.
Package Change Request ====================== Package Name: slf4j New Branches: el5 el6 Owners: shughes
Have you asked the existing Fedora maintainers if they would like to maintain this package in EPEL?
Yes, He asked me and I can't dedicate any time to maintain it in EPEL.