You are viewing a plain text version of this content. The canonical link for it is here.
Posted to xml-commons-dev@xerces.apache.org by bu...@apache.org on 2010/02/14 13:15:19 UTC
DO NOT REPLY [Bug 48741] New: javax.xml.stream.XMLOutputFactory
should not depend on woodstox/WstxOutputFactory
https://issues.apache.org/bugzilla/show_bug.cgi?id=48741
Summary: javax.xml.stream.XMLOutputFactory should not depend on
woodstox/WstxOutputFactory
Product: XmlCommons
Version: 1.x
Platform: Macintosh
OS/Version: other
Status: NEW
Severity: critical
Priority: P2
Component: JAXP
AssignedTo: commons-dev@xml.apache.org
ReportedBy: dizzzz@exist-db.org
I did put xml-apis.jar (version 1.4) in my application endorsed directory, to
be sure that everyone runs/builds against the right XML-apis.
Unfortunately this introduces a dependency with the codehause / woodstox
implementation, in my case com.ctc.wstx.stax.WstxOutputFactory ; see line 30:
http://svn.apache.org/viewvc/xml/commons/trunk/java/external/src/javax/xml/stream/XMLOutputFactory.java?view=markup
In my project this dependency is 'undesired'. The bad thing here is that I
cannot override this default implementation. Since the application runs e.g. in
tomcat, setting a system property is difficult / impossible.
The second issue is that it is not possible to explicitly reference to the
com.sun... implementation from my code, since one can not rely on running in
the Sun JVM.
So my question:
- why the woodstox dependency?
- please could you remove this dependency?
It might be a good idea add some notes regarding this on the xml-commons web
pages.
Should xml-apis.jar be in the jvm/application endorsed directory?
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 48741] javax.xml.stream.XMLOutputFactory should
not depend on woodstox/WstxOutputFactory
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48741
--- Comment #3 from Dannes Wessels <di...@exist-db.org> 2010-02-20 11:09:21 UTC ---
Note: issue confirmed on Sun JVM on windows (32 bits) and the derived version
on MacOsX (64 bit). Both Java6u17 or u18
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 48741] javax.xml.stream.XMLOutputFactory should
not depend on woodstox/WstxOutputFactory
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48741
--- Comment #2 from Dannes Wessels <di...@exist-db.org> 2010-02-20 11:06:06 UTC ---
Since Java6 a default implementation is part of the JVM, e.g. check
http://kickjava.com/src/javax/xml/stream/XMLOutputFactory.java.htm : for the
*sun* jvm the default implementation is
com.sun.xml.internal.stream.XMLOutputFactoryImpl , for other vendors
(classpath?harmony?) this will be a different class.
You are right that via the 'service registry' the sun implementation should be
found, but this is -onfortunately- not the case. The factoryFinder scans all
jar files on the classpath, and reads for the "META-INF/services/"thing in
there. SUn does not set this service thing for its own jar files.
IMO it would be logical to choose for
com.sun.xml.internal.stream.XMLOutputFactoryImpl by default. Best alternative
is probably to use some reflection on the presence of some alternative
implementations as well.
Note that just by putting xml-api.jar on the classpath my application stopped
working. For our project there is no need to have an other StAX implementation
than a simple default one provided by the JVM.
I think closing this report is not a good thing. please reconsider :-)
regards
Dannes
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 48741] javax.xml.stream.XMLOutputFactory should
not depend on woodstox/WstxOutputFactory
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48741
--- Comment #5 from Dannes Wessels <di...@exist-db.org> 2010-06-19 17:58:30 EDT ---
Great to see 2.10 being released. super!
ABout this issue: why not fallback to the default sun implementation if this
implementation is available? For an enduser this is IMO much more transparent.
STAX is part of the language since Java6, why not use the default
implementation? All implementations are required to provide it?
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 48741] javax.xml.stream.XMLOutputFactory should
not depend on woodstox/WstxOutputFactory
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48741
Michael Glavassevich <mr...@ca.ibm.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |WONTFIX
--- Comment #4 from Michael Glavassevich <mr...@ca.ibm.com> 2010-03-07 04:44:47 UTC ---
This is no different than any of the other JAXP factories, each of which
nominate a default implementation when none could be found through its search
mechanism. If you merely place xml-apis.jar in the endorsed directory of a Sun
JDK you'll also get errors about not being able to find Xerces or Xalan.
The Apache distribution has its own set of defaults which may be different from
some other platform's. They were chosen intentionally, in this case to point to
the ASL licensed StAX implementation used by Apache projects which I could
argue is a far more appropriate value than Sun's proprietary implementation
(which only exists in their JDK).
If you put xml-apis.jar in the endorsed directory you also need a JAXP / StAX
implementation on your classpath. These will be picked up through the
META-INF/services files in their jars.
There is no bug here.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
DO NOT REPLY [Bug 48741] javax.xml.stream.XMLOutputFactory should
not depend on woodstox/WstxOutputFactory
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=48741
--- Comment #1 from Jacob Kjome <ho...@visi.com> 2010-02-19 23:41:43 UTC ---
There is no hard dependency on Woodstox here. What you need to do is place a
jar file containing your preferred Stax implementation in the classpath. That
jar will provide its own implementations and be registered via the service
registry. The default value in xml-apis.jar is a fallback. No need to set a
system property or anything like that. I think the value was chosen probably
because it was the first (or preferred) implementation chosen by the developer
of the Stax API. It is not relevant if you provide your own implementation
which is something you really have to do anyway.
I would suggest resolving this bug as there is no issue here that I can see.
Jake
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.