You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@camel.apache.org by "Luca Burgazzoli (JIRA)" <ji...@apache.org> on 2016/08/05 11:47:20 UTC
[jira] [Comment Edited] (CAMEL-10217) Remove logging
implementations from compile scope
[ https://issues.apache.org/jira/browse/CAMEL-10217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15409352#comment-15409352 ]
Luca Burgazzoli edited comment on CAMEL-10217 at 8/5/16 11:46 AM:
------------------------------------------------------------------
[~davsclaus], [~nferraro]
I'm having a look on how complex it is to migrate all the test stuffs to log4j2 (CAMEL-10224), so far the properties migration seems to work fine.
I should have a branch ready for review by end of the day beginning next week.
was (Author: lb):
[~davsclaus][~nferraro]
I'm having a look on how complex it is to migrate all the test stuffs to log4j2 (CAMEL-10224), so far the properties migration seems to work fine.
I should have a branch ready for review by end of the day beginning next week.
> Remove logging implementations from compile scope
> -------------------------------------------------
>
> Key: CAMEL-10217
> URL: https://issues.apache.org/jira/browse/CAMEL-10217
> Project: Camel
> Issue Type: Bug
> Reporter: Nicola Ferraro
> Assignee: Nicola Ferraro
> Priority: Trivial
>
> Some components include logging implementations among their dependencies. If it is applicable (eg. if they are not hardcoded), they should be removed.
> A quick script:
> {code}
> for comp in $(ls | grep "camel-" | grep -v ".iml"); do
> cd $comp
> mvn dependency:tree | grep 'log4j\|logback\|slf4j-simple\|slf4j-jdk14\|slf4j-log4j12' | grep -v test | xargs -I line echo "$(pwd) line"
> cd ..
> done
> {code}
> Shows the following problems:
> {noformat}
> ./camel/components/camel-atmos [INFO] | +- log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-gora [INFO] | +- org.slf4j:slf4j-log4j12:jar:1.7.21:compile
> ./camel/components/camel-gora [INFO] | - log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-hbase [INFO] | +- log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-jbpm [INFO] +- org.slf4j:slf4j-log4j12:jar:1.7.21:compile
> ./camel/components/camel-jbpm [INFO] | - log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-jt400 [INFO] +- org.slf4j:slf4j-log4j12:jar:1.7.21:compile
> ./camel/components/camel-jt400 [INFO] | - log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-krati [INFO] | - log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-openshift [INFO] | - log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-salesforce [INFO] +- log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-scr [INFO] +- org.slf4j:slf4j-log4j12:jar:1.7.21:compile
> ./camel/components/camel-scr [INFO] +- log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-spark [INFO] | +- log4j:log4j:jar:1.2.17:compile
> ./camel/components/camel-spark [INFO] | | +- log4j:apache-log4j-extras:jar:1.2.17:compile
> ./camel/components/camel-spark-rest [INFO] | +- org.slf4j:slf4j-simple:jar:1.7.21:compile
> ./camel/components/camel-spring-boot [INFO] | | +- ch.qos.logback:logback-classic:jar:1.1.7:compile
> ./camel/components/camel-spring-boot [INFO] | | | - ch.qos.logback:logback-core:jar:1.1.7:compile
> ./camel/components/camel-spring-boot [INFO] | | - org.slf4j:log4j-over-slf4j:jar:1.7.21:compile
> ./camel/components/camel-spring-boot-starter [INFO] | | +- ch.qos.logback:logback-classic:jar:1.1.7:compile
> ./camel/components/camel-spring-boot-starter [INFO] | | | - ch.qos.logback:logback-core:jar:1.1.7:compile
> ./camel/components/camel-spring-boot-starter [INFO] | | - org.slf4j:log4j-over-slf4j:jar:1.7.21:compile
> ./camel/components/camel-zipkin-starter [INFO] | | +- ch.qos.logback:logback-classic:jar:1.1.7:compile
> ./camel/components/camel-zipkin-starter [INFO] | | | - ch.qos.logback:logback-core:jar:1.1.7:compile
> ./camel/components/camel-zipkin-starter [INFO] | | - org.slf4j:log4j-over-slf4j:jar:1.7.21:compile
> ./camel/components/camel-zookeeper [INFO] | +- log4j:log4j:jar:1.2.17:compile
> {noformat}
> Spring-boot starters are allowed to have logging implementations. _org.slf4j:log4j-over-slf4j_ is not an implementation.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)