Yes I thought something similar when trashing 2 hours to try to get the JAXB 
generator running.
Just to figure it might be broken beyond repair...
Thanks that you at least confirm this view and spare me further investigation.

There are 2 options on the table

1.) Do a quick hack and add the <trim/> support directly in the code
2.) Invest more time and fix the JAXB source generator setup

I'm tempted to go 1.
But of course I'm happy if someone finds a bit time to fix the generator setup.
David, you did this initially. Is it worth it and can you free up some time for 
it?


LieGrue,
strub


> Am 31.01.2018 um 09:20 schrieb Romain Manni-Bucau <rmannibu...@gmail.com>:
> 
> Hi Mark,
> 
> For simple updates like that I tend to update the sax parser manually
> (that's what we did for JSF since most of the model is not handled by tomee
> anyway).
> For CDI we are supposed to still use JAXB (since there can be a lot of
> beans.xml):
> https://github.com/apache/tomee/blob/master/container/openejb-core/src/main/java/org/apache/openejb/config/ReadDescriptors.java#L681
> 
> 
> 
> Romain Manni-Bucau
> @rmannibucau <https://twitter.com/rmannibucau> |  Blog
> <https://rmannibucau.metawerx.net/> | Old Blog
> <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
> LinkedIn <https://www.linkedin.com/in/rmannibucau>
> 
> 2018-01-31 8:17 GMT+01:00 Mark Struberg <strub...@yahoo.de.invalid>:
> 
>> Hi folks!
>> 
>> I've come down to just 18 failing tests in tck/cdi-embedded.
>> Most of them are EE related (missing privileges exceptions etc)
>> 
>> There is one left with the CDI integration: <trim/> support.
>> This needs a change in the beans.xml schema.
>> I've added the changes and moved to container/openejb-jee/src/
>> main/resources/META-INF/schema/beans_2_0.xsd
>> 
>> But now I totally fail to generate the JAXB sources.
>> I've found the maven profile to start the generator, but it simply fails
>> with an Exception.
>> 
>> Anyone able to help out?
>> 
>> The work is done in fb_tomee8
>> 
>> Btw, I also tried the attached patch to move to valid urls at least...
>> 
>> 
>> LieGrue,
>> strub
>> 
>> 
>> 

Reply via email to