Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Romain Manni-Bucau
depends the perfs. If javax gives the same no need of xalan, otherwise we would need some flag. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn 2017-11-21 11:52 GMT+01:00 Matthew Broadhead : > should xalan be available through some route or

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Matthew Broadhead
should xalan be available through some route or does everything need switching to some generic xml parser like javax.xml.xpath.XPath On 21/11/2017 11:28, Romain Manni-Bucau wrote: yes, that's the ones to fix - make it compiling + validate perfs are still ok. Romain Manni-Bucau @rmannibucau |

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Romain Manni-Bucau
yes, that's the ones to fix - make it compiling + validate perfs are still ok. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn 2017-11-21 11:25 GMT+01:00 Matthew Broadhead : > it is imported in these ones in main as well as loads in test >

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Matthew Broadhead
it is imported in these ones in main as well as loads in test ./impl/src/main/java/org/apache/taglibs/standard/tag/common/xml/ExprSupport.java:28:import org.apache.xpath.XPath; ./impl/src/main/java/org/apache/taglibs/standard/tag/common/xml/ExprSupport.java:29:import

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Romain Manni-Bucau
No, you see this stack cause the test cant be loaded but it is in several classes of taglib project, XalanUtil and a few others. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn 2017-11-21 11:14 GMT+01:00 Matthew Broadhead : > so do i need

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Matthew Broadhead
so do i need to replace it with something else?  is that inside ReflectionUtils? On 21/11/2017 11:08, Romain Manni-Bucau wrote: That's the one, org.apache.xpath is provided by xalan (don't ask me why xalan is not in the name ;)) Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github |

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Romain Manni-Bucau
That's the one, org.apache.xpath is provided by xalan (don't ask me why xalan is not in the name ;)) Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn 2017-11-21 10:57 GMT+01:00 Matthew Broadhead : > mvn install outputs this: > >

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Matthew Broadhead
mvn install outputs this: org.apache.maven.surefire.util.SurefireReflectionException: java.lang.reflect.InvocationTargetException; nested exception is java.lang.reflect.InvocationTargetException: null java.lang.reflect.InvocationTargetException     at

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Matthew Broadhead
Hi Romain, the dependency is listed in each pom.xml in - compat - impl - jstlel - standard-test it is listed as provided in all of them except standard-test. would i just remove all references and then do a mvn install to check the output? Matthew On 21/11/2017 10:38, Romain Manni-Bucau

Re: [VOTE] Apache TomEE 7.0.4 - Roll 2

2017-11-21 Thread Romain Manni-Bucau
Hi Matthew, if you checkout https://svn.apache.org/repos/asf/tomcat/taglibs/standard/trunk and drop xalan dependency you should be on track. You can have a look to XalanUtil class but the xalan dependency is mainly here for XPath usage IIRC. There are some replacements in the JRE so it should be