Then we should at least add a README? Or at least document that profile in the pom.
LieGrue, strub > Am 31.01.2018 um 09:59 schrieb Romain Manni-Bucau <rmannibu...@gmail.com>: > > 2018-01-31 9:34 GMT+01:00 Mark Struberg <strub...@yahoo.de.invalid>: > >> 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 >> > +1 > >> 2.) Invest more time and fix the JAXB source generator setup >> > No point today to do it. Can be worse keeping it as a comment somewhere for > future specs if needed but not for existing one IMHO. > >> >> 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 >>>> >>>> >>>> >> >>