Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-10 Thread Claus Ibsen
Morning guys Christian I noticed that you import a com.sun package. +import com.sun.xml.bind.marshaller.NamespacePrefixMapper; Will this not be problems for people using IBM JVMs etc? I would normally discourage against using com.sun packages, especially in camel-core. Maybe there is an interfac

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-10 Thread Christian Müller
Good morning Claus! I had the same thoughts, but because this class comes from jaxb-impl-2.1.13, I assume it will not be a problem with an IBM JDK. I use the Apple JDK (Java version: 1.6.0_31, vendor: Apple Inc.) and it works well. NamespacePrefixMapper is an abstract class which doesn't implemen

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-10 Thread Christian Müller
And the import +import com.sun.xml.bind.marshaller.NamespacePrefixMapper; is in camel-jaxb, not camel-core... Best, Christian On Fri, May 11, 2012 at 5:57 AM, Claus Ibsen wrote: > Morning guys > > Christian I noticed that you import a com.sun package. > +import com.sun.xml.bind.marshaller.Names

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-11 Thread Babak Vahdat
Good morning Not sure if I'm missing anything here about the requested feature but I think there's no extra feature/functionality we should provide for this as that's already given out of the box through the JAXB's own *public* API: See the Javadoc examples here: http://docs.oracle.com/javase/6/

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-11 Thread Babak Vahdat
Good morning Not sure if I'm missing anything here about the requested feature but I think there's no extra feature/functionality we should provide for this as that's already given out of the box through the JAXB's own *public* API: See the Javadoc examples here: http://docs.oracle.com/javase/6/

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-11 Thread Babak Vahdat
Hi again, Just found that we've got exactly the same thing here as well (since ages): https://svn.apache.org/repos/asf/camel/trunk/components/camel-web/src/main/java/org/apache/camel/web/util/JAXBMarshallerResolver.java Which came through: https://issues.apache.org/jira/browse/CAMEL-2280 Babak

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-11 Thread Christian Müller
Good catch! I will write a test which proves this. If it works, I will revert my changes for this issue. Best, Christian On Fri, May 11, 2012 at 9:48 AM, Babak Vahdat wrote: > Hi again, > > Just found that we've got exactly the same thing here as well (since ages): > > > https://svn.apache.org/

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-11 Thread Daniel Kulp
We ran into the same thing with CXF a long time ago. It's also a bit more complex than just referencing the NamespaceMapper class. The "key" that is used to set it into the context is also different with the built in version of JAXB as even the "key" gets the ".internal" into it. For CXF,

Re: svn commit: r1336836 - in /camel/trunk/components/camel-jaxb/src: main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java test/java/org/apache/camel/converter/jaxb/JaxbDataFormatMultipleName

2012-05-11 Thread Christian Müller
Done. Thanks again Babak for the hint. Best, Christian On Fri, May 11, 2012 at 5:58 PM, Christian Müller < christian.muel...@gmail.com> wrote: > Good catch! > > I will write a test which proves this. If it works, I will revert my > changes for this issue. > > Best, > Christian > > > On Fri, May