camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Claus Ibsen (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CAMEL-10217) Remove logging implementations from compile scope
Date Wed, 03 Aug 2016 11:57:20 GMT

    [ https://issues.apache.org/jira/browse/CAMEL-10217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15405783#comment-15405783
] 

Claus Ibsen commented on CAMEL-10217:
-------------------------------------

Maybe some of the archetypes uses log4j 1.x in their generated sample code. It could be a
good idea to upgrade those to use lo4j v2 also

> 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)

Mime
View raw message