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

Babak Vahdat edited comment on CAMEL-4955 at 3/7/12 9:47 AM:
-------------------------------------------------------------

@Kangsik

At least camel-jaxb *does* explicitly declare to depend both on a specific 
JAXB-API (2.1) as well as an implementation of it (Sun-Impl 2.1.13). And we 
could probably upgrade those dependencies so that there would be nothing the 
users should manually do about a potential NPE in JDK 7. But before that it 
should be made sure that this upgrade doesn't break anything existing.

Then the idea would be *even if* the users don't really need JaxbDataFormat 
they would still declare a dependency to camel-jaxb so that the proper jars 
would come along so that there would be no necessity to take over proper jars 
manually. However I see this option just as a workaorund until it's properly 
fixed inside JDK 7 itself.

{code}
~/dev/workspace/camel/components/camel-jaxb>mvn dependency:tree
[INFO] Scanning for projects...
[INFO]                                                                         
[INFO] ------------------------------------------------------------------------
[INFO] Building Camel :: JAXB 2.10-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO] 
[INFO] --- maven-dependency-plugin:2.1:tree (default-cli) @ camel-jaxb ---
[INFO] org.apache.camel:camel-jaxb:bundle:2.10-SNAPSHOT
[INFO] +- org.apache.camel:camel-core:jar:2.10-SNAPSHOT:compile
[INFO] |  \- org.slf4j:slf4j-api:jar:1.6.1:compile
[INFO] +- javax.xml.bind:jaxb-api:jar:2.1:compile
[INFO] |  +- javax.xml.stream:stax-api:jar:1.0-2:compile
[INFO] |  \- javax.activation:activation:jar:1.1:compile
[INFO] +- com.sun.xml.bind:jaxb-impl:jar:2.1.13:compile
[INFO] +- junit:junit:jar:4.8.1:test
[INFO] +- org.apache.camel:camel-test-spring:jar:2.10-SNAPSHOT:test
[INFO] |  +- org.apache.camel:camel-test:jar:2.10-SNAPSHOT:test
[INFO] |  \- org.apache.camel:camel-spring:jar:2.10-SNAPSHOT:test
[INFO] |     +- org.springframework:spring-context:jar:3.0.7.RELEASE:test
[INFO] |     |  +- org.springframework:spring-beans:jar:3.0.7.RELEASE:test
[INFO] |     |  +- org.springframework:spring-core:jar:3.0.7.RELEASE:test
[INFO] |     |  |  \- commons-logging:commons-logging:jar:1.1.1:test
[INFO] |     |  +- org.springframework:spring-expression:jar:3.0.7.RELEASE:test
[INFO] |     |  \- org.springframework:spring-asm:jar:3.0.7.RELEASE:test
[INFO] |     +- org.springframework:spring-aop:jar:3.0.7.RELEASE:test
[INFO] |     |  \- aopalliance:aopalliance:jar:1.0:test
[INFO] |     \- org.springframework:spring-tx:jar:3.0.7.RELEASE:test
[INFO] +- org.apache.camel:camel-spring-javaconfig:jar:2.10-SNAPSHOT:test
[INFO] |  \- 
org.apache.servicemix.bundles:org.apache.servicemix.bundles.cglib:jar:2.1_3_6:test
[INFO] +- org.slf4j:slf4j-log4j12:jar:1.6.1:test
[INFO] |  \- log4j:log4j:jar:1.2.16:test
[INFO] +- org.codehaus.woodstox:woodstox-core-asl:jar:4.1.1:test
[INFO] |  \- org.codehaus.woodstox:stax2-api:jar:3.1.1:test
[INFO] +- org.mockito:mockito-core:jar:1.8.5:test
[INFO] |  +- org.hamcrest:hamcrest-core:jar:1.1:test
[INFO] |  \- org.objenesis:objenesis:jar:1.0:test
[INFO] \- org.springframework:spring-test:jar:3.0.7.RELEASE:test
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1.377s
[INFO] Finished at: Wed Mar 07 09:03:22 CET 2012
[INFO] Final Memory: 10M/1011M
[INFO] ------------------------------------------------------------------------
{code} 
                
      was (Author: bvahdat):
    @Kangsik

Camel *does* explicitly declare to depend both on a specific JAXB API (2.1) as 
well as an implementation of it (Sun-Impl 2.1.13). And we could probably 
upgrade those dependencies so that there would be nothing the users should 
manually do about a potential NPE in JDK 7. But before that it should be made 
sure that this upgrade doesn't break anything existing.

{code}
~/dev/workspace/camel/components/camel-jaxb>mvn dependency:tree
[INFO] Scanning for projects...
[INFO]                                                                         
[INFO] ------------------------------------------------------------------------
[INFO] Building Camel :: JAXB 2.10-SNAPSHOT
[INFO] ------------------------------------------------------------------------
[INFO] 
[INFO] --- maven-dependency-plugin:2.1:tree (default-cli) @ camel-jaxb ---
[INFO] org.apache.camel:camel-jaxb:bundle:2.10-SNAPSHOT
[INFO] +- org.apache.camel:camel-core:jar:2.10-SNAPSHOT:compile
[INFO] |  \- org.slf4j:slf4j-api:jar:1.6.1:compile
[INFO] +- javax.xml.bind:jaxb-api:jar:2.1:compile
[INFO] |  +- javax.xml.stream:stax-api:jar:1.0-2:compile
[INFO] |  \- javax.activation:activation:jar:1.1:compile
[INFO] +- com.sun.xml.bind:jaxb-impl:jar:2.1.13:compile
[INFO] +- junit:junit:jar:4.8.1:test
[INFO] +- org.apache.camel:camel-test-spring:jar:2.10-SNAPSHOT:test
[INFO] |  +- org.apache.camel:camel-test:jar:2.10-SNAPSHOT:test
[INFO] |  \- org.apache.camel:camel-spring:jar:2.10-SNAPSHOT:test
[INFO] |     +- org.springframework:spring-context:jar:3.0.7.RELEASE:test
[INFO] |     |  +- org.springframework:spring-beans:jar:3.0.7.RELEASE:test
[INFO] |     |  +- org.springframework:spring-core:jar:3.0.7.RELEASE:test
[INFO] |     |  |  \- commons-logging:commons-logging:jar:1.1.1:test
[INFO] |     |  +- org.springframework:spring-expression:jar:3.0.7.RELEASE:test
[INFO] |     |  \- org.springframework:spring-asm:jar:3.0.7.RELEASE:test
[INFO] |     +- org.springframework:spring-aop:jar:3.0.7.RELEASE:test
[INFO] |     |  \- aopalliance:aopalliance:jar:1.0:test
[INFO] |     \- org.springframework:spring-tx:jar:3.0.7.RELEASE:test
[INFO] +- org.apache.camel:camel-spring-javaconfig:jar:2.10-SNAPSHOT:test
[INFO] |  \- 
org.apache.servicemix.bundles:org.apache.servicemix.bundles.cglib:jar:2.1_3_6:test
[INFO] +- org.slf4j:slf4j-log4j12:jar:1.6.1:test
[INFO] |  \- log4j:log4j:jar:1.2.16:test
[INFO] +- org.codehaus.woodstox:woodstox-core-asl:jar:4.1.1:test
[INFO] |  \- org.codehaus.woodstox:stax2-api:jar:3.1.1:test
[INFO] +- org.mockito:mockito-core:jar:1.8.5:test
[INFO] |  +- org.hamcrest:hamcrest-core:jar:1.1:test
[INFO] |  \- org.objenesis:objenesis:jar:1.0:test
[INFO] \- org.springframework:spring-test:jar:3.0.7.RELEASE:test
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1.377s
[INFO] Finished at: Wed Mar 07 09:03:22 CET 2012
[INFO] Final Memory: 10M/1011M
[INFO] ------------------------------------------------------------------------
{code} 
                  
> Camel should run in a Java 7 environment
> ----------------------------------------
>
>                 Key: CAMEL-4955
>                 URL: https://issues.apache.org/jira/browse/CAMEL-4955
>             Project: Camel
>          Issue Type: Improvement
>    Affects Versions: 2.9.0
>            Reporter: Christian Müller
>            Priority: Critical
>             Fix For: 2.10.0
>
>
> End of life for Java 6 is July 2012.
> If we release Camel 2.10.0 3 month after 2.9.0, it's round about March/April. 
> The next Camel release after 2.10.0 (may be 3.0.0) will be not available 
> before July 2012.
> Because of this, I think Camel 2.10.0 must be able to run in an Java 7 
> environment to give our users the possibility to migrate to Java 7 before it 
> reaches EOL.
> http://www.oracle.com/technetwork/java/eol-135779.html

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to