ForEachTag's class comment says "Implementation of <x:forEach>; tag using low-level Xalan API." Isn't there a way to emulate that with simple JAXP instead of re-introducing the huge & unmaintained dependency on Xalan ? As far as I remember, not only xalan.jar, but also serializer.jar had to be removed (maybe in our apps, I don't remember), and that was a bit chaotic. I'm scared at the idea of re-doing this error-prone process in the reverse order, and who know how many unsolved bugs Xalan fixed in vanilla JAXP will re-introduced on TomEE?
Alexandre Le mar. 8 déc. 2020 à 13:39, Jean-Louis Monteiro <[email protected]> a écrit : > > Quick update on this one ... > > Tomcat Taglib has a direct dependency on XParthContext > https://github.com/apache/tomcat-taglibs-standard/blob/master/impl/src/main/java/org/apache/taglibs/standard/tag/common/xml/ForEachTag.java > > This is available in Xalan > So at this point, we have an issue in TomEE with xml:transform not working. > TCK or not, does not matter, it's not working. > > Long term would probably be to get Tomcat Taglib implementation to use JAXP > APIs instead of straight Xalan. > Short term would be to patch Xalan ourselves. > > Thoughts? > > > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Tue, Dec 8, 2020 at 11:31 AM Jean-Louis Monteiro < > [email protected]> wrote: > > > Thanks. > > Alexandre also replied on the ticket so I'll revert and see if I can find > > a better solution. > > > > Thanks guys > > -- > > Jean-Louis Monteiro > > http://twitter.com/jlouismonteiro > > http://www.tomitribe.com > > > > > > On Tue, Dec 8, 2020 at 11:21 AM Jonathan Gallimore < > > [email protected]> wrote: > > > >> Yes - there's a bug with parsing Japanese characters with Xalan. It also > >> looks like its not maintained any more. > >> > >> I was under the impression that the Transformer functionality was part of > >> the JDK, so I'm curious as to what is missing. I'll have a look at that > >> failing test. Looks like we're missing org.apache.xpath - looks like this > >> is a little messy. > >> > >> Jon > >> > >> On Tue, Dec 8, 2020 at 9:58 AM Jean-Louis Monteiro < > >> [email protected]> > >> wrote: > >> > >> > Hi, > >> > > >> > Been looking at the JSTL failures > >> > > >> > > >> http://ec2-54-173-218-40.compute-1.amazonaws.com:17171/tests?path=com.sun.ts.tests.jstl&build=1607347669779&status=FAILED > >> > > >> > I realized that Xalan is missing from our distribution. > >> > Looks like it's been removed about a year ago. > >> > > >> > > >> > > >> https://github.com/apache/tomee/commit/5f1b57af1b4f11638c8c9540fcc83feb618980a4 > >> > > >> > Does anyone have any clue on why? > >> > Obviously we won't pass the transform TCK without an XSLT Processor like > >> > Xalan. Adding it back solved most of the issues. > >> > > >> > -- > >> > Jean-Louis Monteiro > >> > http://twitter.com/jlouismonteiro > >> > http://www.tomitribe.com > >> > > >> > >
