You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by David Blevins <da...@gmail.com> on 2012/02/29 18:17:22 UTC
JAXB Upgrade (was: svn commit: r1295012)
Thanks for the two JIRAs!! :)
On Feb 29, 2012, at 12:43 AM, rmannibucau@apache.org wrote:
> ==============================================================================
> --- openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java (original)
> +++ openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java Wed Feb 29 08:43:53 2012
> @@ -114,10 +114,9 @@ public class Installer {
> File endorsed = new File(paths.getCatalinaHomeDir(), "endorsed");
> endorsed.mkdir();
>
> - File sourceJar = paths.getJavaEEAPIJAr();
>
> - copyClasses(sourceJar, new File(endorsed, "annotation-api.jar"), "javax/annotation/.*");
> -// copyClasses(sourceJar, new File(endorsed, "jaxb-api.jar"), "javax/xml/bind/.*");
> + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed, "annotation-api.jar"), "javax/annotation/.*");
> + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed, "jaxb-api.jar"), "javax/xml/bind/.*");
> }
I think we need to move the JAXB 2.2. RI in there too as the JAXB 2.1 RI is part of the JVM.
-David
Re: JAXB Upgrade (was: svn commit: r1295012)
Posted by Romain Manni-Bucau <rm...@gmail.com>.
some more info on it:
i added jaxb-api and jaxb-impl in endorsed folder.
jaxb-impl is a copy of the RI jar (on repo1).
jaxb-api is extracted from javaee-api (like annotation-api). The weird
thing is i excluded ContextFinder class because the geronimo jaxb api jar
was depending on OSGi because of this class (geronimo locator). Since the
Context of the jre 6 is fine it is fine for us but that's a bit odd when
you don't know the cause.
- Romain
2012/2/29 David Blevins <da...@gmail.com>
>
> On Feb 29, 2012, at 9:38 AM, Romain Manni-Bucau wrote:
>
> > that's ok for me,
> >
> > but in that case we remove geronimo jaxb from javaee-api
>
> We need both the api and impl jars in the endorsed dir.
>
>
> -David
>
> > 2012/2/29 David Blevins <da...@gmail.com>
> >
> >> Thanks for the two JIRAs!! :)
> >>
> >> On Feb 29, 2012, at 12:43 AM, rmannibucau@apache.org wrote:
> >>>
> >>
> ==============================================================================
> >>> ---
> >>
> openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java
> >> (original)
> >>> +++
> >>
> openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java
> >> Wed Feb 29 08:43:53 2012
> >>> @@ -114,10 +114,9 @@ public class Installer {
> >>> File endorsed = new File(paths.getCatalinaHomeDir(),
> "endorsed");
> >>> endorsed.mkdir();
> >>>
> >>> - File sourceJar = paths.getJavaEEAPIJAr();
> >>>
> >>> - copyClasses(sourceJar, new File(endorsed,
> >> "annotation-api.jar"), "javax/annotation/.*");
> >>> -// copyClasses(sourceJar, new File(endorsed, "jaxb-api.jar"),
> >> "javax/xml/bind/.*");
> >>> + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed,
> >> "annotation-api.jar"), "javax/annotation/.*");
> >>> + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed,
> >> "jaxb-api.jar"), "javax/xml/bind/.*");
> >>> }
> >>
> >> I think we need to move the JAXB 2.2. RI in there too as the JAXB 2.1 RI
> >> is part of the JVM.
> >>
> >>
> >> -David
> >>
> >>
>
>
Re: JAXB Upgrade (was: svn commit: r1295012)
Posted by David Blevins <da...@gmail.com>.
On Feb 29, 2012, at 9:38 AM, Romain Manni-Bucau wrote:
> that's ok for me,
>
> but in that case we remove geronimo jaxb from javaee-api
We need both the api and impl jars in the endorsed dir.
-David
> 2012/2/29 David Blevins <da...@gmail.com>
>
>> Thanks for the two JIRAs!! :)
>>
>> On Feb 29, 2012, at 12:43 AM, rmannibucau@apache.org wrote:
>>>
>> ==============================================================================
>>> ---
>> openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java
>> (original)
>>> +++
>> openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java
>> Wed Feb 29 08:43:53 2012
>>> @@ -114,10 +114,9 @@ public class Installer {
>>> File endorsed = new File(paths.getCatalinaHomeDir(), "endorsed");
>>> endorsed.mkdir();
>>>
>>> - File sourceJar = paths.getJavaEEAPIJAr();
>>>
>>> - copyClasses(sourceJar, new File(endorsed,
>> "annotation-api.jar"), "javax/annotation/.*");
>>> -// copyClasses(sourceJar, new File(endorsed, "jaxb-api.jar"),
>> "javax/xml/bind/.*");
>>> + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed,
>> "annotation-api.jar"), "javax/annotation/.*");
>>> + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed,
>> "jaxb-api.jar"), "javax/xml/bind/.*");
>>> }
>>
>> I think we need to move the JAXB 2.2. RI in there too as the JAXB 2.1 RI
>> is part of the JVM.
>>
>>
>> -David
>>
>>
Re: JAXB Upgrade (was: svn commit: r1295012)
Posted by Romain Manni-Bucau <rm...@gmail.com>.
that's ok for me,
but in that case we remove geronimo jaxb from javaee-api
- Romain
2012/2/29 David Blevins <da...@gmail.com>
> Thanks for the two JIRAs!! :)
>
> On Feb 29, 2012, at 12:43 AM, rmannibucau@apache.org wrote:
> >
> ==============================================================================
> > ---
> openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java
> (original)
> > +++
> openejb/trunk/openejb/tomee/tomee-common/src/main/java/org/apache/tomee/installer/Installer.java
> Wed Feb 29 08:43:53 2012
> > @@ -114,10 +114,9 @@ public class Installer {
> > File endorsed = new File(paths.getCatalinaHomeDir(), "endorsed");
> > endorsed.mkdir();
> >
> > - File sourceJar = paths.getJavaEEAPIJAr();
> >
> > - copyClasses(sourceJar, new File(endorsed,
> "annotation-api.jar"), "javax/annotation/.*");
> > -// copyClasses(sourceJar, new File(endorsed, "jaxb-api.jar"),
> "javax/xml/bind/.*");
> > + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed,
> "annotation-api.jar"), "javax/annotation/.*");
> > + copyClasses(paths.getJavaEEAPIJar(), new File(endorsed,
> "jaxb-api.jar"), "javax/xml/bind/.*");
> > }
>
> I think we need to move the JAXB 2.2. RI in there too as the JAXB 2.1 RI
> is part of the JVM.
>
>
> -David
>
>