Re: [Spacewalk-devel] [Fwd: removing log5j until we get fedora approval; removed velocity since its in tempjars...]

2010-07-09 Thread Shannon Hughes

right, log5j is really just a wrapper for log4j

On 07/09/10 11:21, Miroslav Suchý wrote:

On 07/09/2010 03:40 PM, Shannon Hughes wrote:

justin is working to get it approved into fedora but there was an
issue with the Source var in the spec file.


To be precise:
https://bugzilla.redhat.com/show_bug.cgi?id=574575

IMO it is good spec, but since it is Justin first package he needs 
sponsor. Only few people can sponsor others. One of them is Dennis 
Gilmore, who promised to sponsor Justin, but according the BZ it seems 
he did not response since April.


Dennis can you look at this review please?

> Shannon - do we still use/have log4j then?
Hmm log4j is not replacement for log4j, it is enhancement.
log5j requires log4j:
$ rpm -qpR /home/msuchy/rpmbuild/RPMS/noarch/log5j-1.2-1.el6.noarch.rpm
log4j
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1



___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel


Re: [Spacewalk-devel] [Fwd: removing log5j until we get fedora approval; removed velocity since its in tempjars...]

2010-07-09 Thread Miroslav Suchý

On 07/09/2010 03:40 PM, Shannon Hughes wrote:

justin is working to get it approved into fedora but there was an
issue with the Source var in the spec file.


To be precise:
https://bugzilla.redhat.com/show_bug.cgi?id=574575

IMO it is good spec, but since it is Justin first package he needs 
sponsor. Only few people can sponsor others. One of them is Dennis 
Gilmore, who promised to sponsor Justin, but according the BZ it seems 
he did not response since April.


Dennis can you look at this review please?

> Shannon - do we still use/have log4j then?
Hmm log4j is not replacement for log4j, it is enhancement.
log5j requires log4j:
$ rpm -qpR /home/msuchy/rpmbuild/RPMS/noarch/log5j-1.2-1.el6.noarch.rpm
log4j
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1

--
Miroslav Suchy
Red Hat Satellite Engineering

___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel


Re: [Spacewalk-devel] [Fwd: removing log5j until we get fedora approval; removed velocity since its in tempjars...]

2010-07-09 Thread Shannon Hughes

yes.

log5j was removed since its not yet in fedora and we are not using it in 
our code yet. i don't want to add it to ivy deps until its actually 
used. justin is working to get it approved into fedora but there was an 
issue with the Source var in the spec file.


On 07/09/10 09:19, Cliff Perry wrote:

Shannon - do we still use/have log4j then?


 



___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel


___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel


[Spacewalk-devel] [Fwd: removing log5j until we get fedora approval; removed velocity since its in tempjars...]

2010-07-09 Thread Cliff Perry

Shannon - do we still use/have log4j then?




___
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel