On Sat, Sep 19, 2009 at 4:43 PM, Andreas Veithen <andreas.veit...@gmail.com>wrote:
> The situation in Axis2 trunk is now as follows: > - I added an integration test for code generation from WSDL 2.0. This > tests indeed fails (with the current Woden 1.0-SNAPSHOT available from > the snapshot repository) if xercesImpl is not available. > - The dependencies are such that xercesImpl only comes in as a > transitive dependency of woden-impl-dom. > - There is a comment in the dependencyManagement entry for > woden-impl-dom that refers to WODEN-152. > is this means I can pass all the tests if I build a woden SNAPSHOT locally in may machine? if so lets go with this change. But for next Axis2 release we need to add a statement to release note about this change. thanks, Amila. > > Andreas > > On Fri, Sep 18, 2009 at 23:04, Andreas Veithen > <andreas.veit...@gmail.com> wrote: > > Interesting. It looks like we don't have test cases for WSDL 2.0. We > > have a couple of WSDL 2.0 files in > > modules/integration/test-resources/wsdl20, but they are never used > > during the tests... > > > > Andreas > > > > On Fri, Sep 18, 2009 at 22:12, Sagara Gunathunga > > <sagara.gunathu...@gmail.com> wrote: > >> Hi Folks, > >> > >> I think Axis2 use Woden as a dependency. At the moment current Woden > >> 1.0-SNAPSHOTs available on [1] depends on xercesImpl and > >> xmlParserAPI ,because some classes directly call DomParser . So i > >> guess this will make run time issues for Axis2 WSDL 2.0 features, > >> but the good news is recently i have ported Woden code based to use > >> JAXP 1.2 [2] , still i need little time to port few test cases after > >> that i will remove Xerces dependencies from woden too. > >> > >> > >> [1] - > http://people.apache.org/maven-snapshot-repository/org/apache/woden/woden/1.0-SNAPSHOT/ > >> > >> [2] - https://issues.apache.org/jira/browse/WODEN-152 > >> > >> Thanks , > >> > >> > >> On Sat, Sep 19, 2009 at 12:19 AM, Andreas Veithen > >> <andreas.veit...@gmail.com> wrote: > >>> I fixed the dependencies of axis2-jaxws, and as a side effect, > >>> xercesImpl is no longer packaged in the distribution. If somebody > >>> comes up with evidence that there is an issue with this, then we > >>> should add it back as an explicit dependency of the module that really > >>> depends on it. > >>> > >>> Andreas > >>> > >>> On Fri, Sep 18, 2009 at 15:00, Deepal jayasinghe <deep...@gmail.com> > wrote: > >>>> Amila and Andreas, > >>>> > >>>> I think Andreas's argument is valid so let's go with this changes. I > >>>> also like to remove all the unwanted dependencies, because it is so > hard > >>>> to set up the project (w.o using maven), and we have so many > dependencies. > >>>> > >>>> Thanks, > >>>> Deepal > >>>>> Since Axis2 1.5 depends on Java 1.5, there is no need to include > >>>>> xercesImpl anymore and it works fine without it. Xerces still got > >>>>> packaged into Axis2 1.5 because of an incorrect dependency in > >>>>> axis2-jaxws: this module has a direct dependency on jaxen (which in > >>>>> turn depends on xercesImpl), while it doesn't use Jaxen. This also > >>>>> causes axis2-jaxws to have a transitive dependency on jdom, dom4j, > XOM > >>>>> and some other unnecessary stuff. If we fix the dependencies of > >>>>> axis2-jaxws, xercesImpl will no longer be included in the dist and > the > >>>>> users can add the version they want if they have a specific need for > >>>>> this. > >>>>> > >>>>> Note that up to now, there is no evidence that the xercesImpl version > >>>>> causes any issues. The post by Wim Goossens probably means that in > his > >>>>> own code, he is using some feature that only exists in recent Xerces > >>>>> versions. Unfortunately he didn't reply to my request to provide > >>>>> additional information. > >>>>> > >>>>> Andreas > >>>>> > >>>>> On Fri, Sep 18, 2009 at 12:14, Amila Suriarachchi > >>>>> <amilasuriarach...@gmail.com> wrote: > >>>>> > >>>>>> hi Andreas, > >>>>>> > >>>>>> was there any discussion on dev list regarding this change? > >>>>>> was there any problem with the XercesImpl-2.8.1.jar? > >>>>>> would there be any advantage if this fixed worked correctly? > >>>>>> > >>>>>> I think if we don't have clear answer for latter two cases better to > revert > >>>>>> this change since this seems to be causing problems for some people. > >>>>>> > >>>>>> thanks, > >>>>>> Amila. > >>>>>> > >>>>>> On Sat, Mar 28, 2009 at 9:21 PM, <veit...@apache.org> wrote: > >>>>>> > >>>>>>> Author: veithens > >>>>>>> Date: Sat Mar 28 15:51:08 2009 > >>>>>>> New Revision: 759488 > >>>>>>> > >>>>>>> URL: http://svn.apache.org/viewvc?rev=759488&view=rev > >>>>>>> Log: > >>>>>>> Removed dependencies on xml-apis and xercesImpl since the target > platform > >>>>>>> is now Java 5. > >>>>>>> > >>>>>>> Modified: > >>>>>>> > >>>>>>> > > webservices/axis2/trunk/java/modules/jaxws-integration/test/org/apache/axis2/jaxws/proxy/GorillaDLWProxyTests.java > >>>>>>> webservices/axis2/trunk/java/modules/parent/pom.xml > >>>>>>> webservices/axis2/trunk/java/modules/saaj/pom.xml > >>>>>>> > >>>>>>> Modified: > >>>>>>> > webservices/axis2/trunk/java/modules/jaxws-integration/test/org/apache/axis2/jaxws/proxy/GorillaDLWProxyTests.java > >>>>>>> URL: > >>>>>>> > http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/jaxws-integration/test/org/apache/axis2/jaxws/proxy/GorillaDLWProxyTests.java?rev=759488&r1=759487&r2=759488&view=diff > >>>>>>> > >>>>>>> > ============================================================================== > >>>>>>> --- > >>>>>>> > webservices/axis2/trunk/java/modules/jaxws-integration/test/org/apache/axis2/jaxws/proxy/GorillaDLWProxyTests.java > >>>>>>> (original) > >>>>>>> +++ > >>>>>>> > webservices/axis2/trunk/java/modules/jaxws-integration/test/org/apache/axis2/jaxws/proxy/GorillaDLWProxyTests.java > >>>>>>> Sat Mar 28 15:51:08 2009 > >>>>>>> @@ -25,7 +25,6 @@ > >>>>>>> import org.apache.axis2.jaxws.framework.AbstractTestCase; > >>>>>>> import > org.apache.axis2.jaxws.message.databinding.JAXBUtilsMonitor; > >>>>>>> import > org.apache.axis2.jaxws.proxy.gorilla_dlw.sei.GorillaInterface; > >>>>>>> -import org.apache.xerces.jaxp.datatype.DatatypeFactoryImpl; > >>>>>>> > >>>>>>> import javax.xml.datatype.DatatypeFactory; > >>>>>>> import javax.xml.datatype.Duration; > >>>>>>> @@ -482,7 +481,6 @@ > >>>>>>> GorillaInterface proxy = getProxy(); > >>>>>>> GregorianCalendar cal = new > >>>>>>> GregorianCalendar(Calendar.YEAR, Calendar.MONTH, > Calendar.DAY_OF_MONTH); > >>>>>>> DatatypeFactory javaxtypeFactory = > >>>>>>> DatatypeFactory.newInstance(); > >>>>>>> - DatatypeFactory xercesfactoryImpl = > >>>>>>> DatatypeFactoryImpl.newInstance(); > >>>>>>> XMLGregorianCalendar request= > >>>>>>> javaxtypeFactory.newXMLGregorianCalendar(cal); > >>>>>>> System.out.println("Javax Factory Clazz Name = > >>>>>>> "+request.getClass().getName()); > >>>>>>> Duration d = > >>>>>>> javaxtypeFactory.newDuration(System.currentTimeMillis()); > >>>>>>> > >>>>>>> Modified: webservices/axis2/trunk/java/modules/parent/pom.xml > >>>>>>> URL: > >>>>>>> > http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/parent/pom.xml?rev=759488&r1=759487&r2=759488&view=diff > >>>>>>> > >>>>>>> > ============================================================================== > >>>>>>> --- webservices/axis2/trunk/java/modules/parent/pom.xml (original) > >>>>>>> +++ webservices/axis2/trunk/java/modules/parent/pom.xml Sat Mar 28 > >>>>>>> 15:51:08 2009 > >>>>>>> @@ -142,8 +142,6 @@ > >>>>>>> <woden.version>1.0-SNAPSHOT</woden.version> > >>>>>>> <wsdl4j.version>1.6.2</wsdl4j.version> > >>>>>>> <xalan.version>2.7.0</xalan.version> > >>>>>>> - <xerces.version>2.8.1</xerces.version> > >>>>>>> - <xml_apis.version>1.3.04</xml_apis.version> > >>>>>>> <xmlbeans.version>2.3.0</xmlbeans.version> > >>>>>>> <xml_resolver.version>1.2</xml_resolver.version> > >>>>>>> <xmlschema.version>1.4.3</xmlschema.version> > >>>>>>> @@ -569,11 +567,6 @@ > >>>>>>> <version>${tomcat.version}</version> > >>>>>>> </dependency> > >>>>>>> <dependency> > >>>>>>> - <groupId>xml-apis</groupId> > >>>>>>> - <artifactId>xml-apis</artifactId> > >>>>>>> - <version>${xml_apis.version}</version> > >>>>>>> - </dependency> > >>>>>>> - <dependency> > >>>>>>> <groupId>xml-resolver</groupId> > >>>>>>> <artifactId>xml-resolver</artifactId> > >>>>>>> <version>${xml_resolver.version}</version> > >>>>>>> @@ -582,6 +575,12 @@ > >>>>>>> <groupId>xalan</groupId> > >>>>>>> <artifactId>xalan</artifactId> > >>>>>>> <version>${xalan.version}</version> > >>>>>>> + <exclusions> > >>>>>>> + <exclusion> > >>>>>>> + <groupId>xml-apis</groupId> > >>>>>>> + <artifactId>xml-apis</artifactId> > >>>>>>> + </exclusion> > >>>>>>> + </exclusions> > >>>>>>> </dependency> > >>>>>>> <dependency> > >>>>>>> <groupId>com.sun.xml.bind</groupId> > >>>>>>> @@ -702,6 +701,16 @@ > >>>>>>> <groupId>org.apache.ws.commons.axiom</groupId> > >>>>>>> <artifactId>axiom-api</artifactId> > >>>>>>> <version>${axiom.version}</version> > >>>>>>> + <exclusions> > >>>>>>> + <exclusion> > >>>>>>> + <groupId>xml-apis</groupId> > >>>>>>> + <artifactId>xml-apis</artifactId> > >>>>>>> + </exclusion> > >>>>>>> + <exclusion> > >>>>>>> + <groupId>xerces</groupId> > >>>>>>> + <artifactId>xercesImpl</artifactId> > >>>>>>> + </exclusion> > >>>>>>> + </exclusions> > >>>>>>> </dependency> > >>>>>>> <dependency> > >>>>>>> <groupId>org.apache.ws.commons.axiom</groupId> > >>>>>>> @@ -830,6 +839,10 @@ > >>>>>>> <groupId>org.codehaus.woodstox</groupId> > >>>>>>> <artifactId>wstx-asl</artifactId> > >>>>>>> </exclusion> > >>>>>>> + <exclusion> > >>>>>>> + <groupId>xerces</groupId> > >>>>>>> + <artifactId>xmlParserAPIs</artifactId> > >>>>>>> + </exclusion> > >>>>>>> </exclusions> > >>>>>>> </dependency> > >>>>>>> <dependency> > >>>>>>> @@ -841,6 +854,10 @@ > >>>>>>> <groupId>org.codehaus.woodstox</groupId> > >>>>>>> <artifactId>wstx-asl</artifactId> > >>>>>>> </exclusion> > >>>>>>> + <exclusion> > >>>>>>> + <groupId>xerces</groupId> > >>>>>>> + <artifactId>xmlParserAPIs</artifactId> > >>>>>>> + </exclusion> > >>>>>>> </exclusions> > >>>>>>> </dependency> > >>>>>>> <dependency> > >>>>>>> @@ -999,11 +1016,6 @@ > >>>>>>> <version>${jalopy.version}</version> > >>>>>>> </dependency> > >>>>>>> <dependency> > >>>>>>> - <groupId>xerces</groupId> > >>>>>>> - <artifactId>xercesImpl</artifactId> > >>>>>>> - <version>${xerces.version}</version> > >>>>>>> - </dependency> > >>>>>>> - <dependency> > >>>>>>> <groupId>org.igniterealtime</groupId> > >>>>>>> <artifactId>smack</artifactId> > >>>>>>> <version>${smack.version}</version> > >>>>>>> @@ -1072,10 +1084,6 @@ > >>>>>>> <artifactId>geronimo-stax-api_1.0_spec</artifactId> > >>>>>>> </dependency> > >>>>>>> <dependency> > >>>>>>> - <groupId>xerces</groupId> > >>>>>>> - <artifactId>xercesImpl</artifactId> > >>>>>>> - </dependency> > >>>>>>> - <dependency> > >>>>>>> <groupId>org.apache.httpcomponents</groupId> > >>>>>>> <artifactId>httpcore</artifactId> > >>>>>>> <scope>test</scope> > >>>>>>> > >>>>>>> Modified: webservices/axis2/trunk/java/modules/saaj/pom.xml > >>>>>>> URL: > >>>>>>> > http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/saaj/pom.xml?rev=759488&r1=759487&r2=759488&view=diff > >>>>>>> > >>>>>>> > ============================================================================== > >>>>>>> --- webservices/axis2/trunk/java/modules/saaj/pom.xml (original) > >>>>>>> +++ webservices/axis2/trunk/java/modules/saaj/pom.xml Sat Mar 28 > 15:51:08 > >>>>>>> 2009 > >>>>>>> @@ -163,6 +163,8 @@ > >>>>>>> <skip>false</skip> > >>>>>>> <forkMode>once</forkMode> > >>>>>>> > >>>>>>> > <argLine>-Djava.endorsed.dirs=${m2Repository}/org/apache/geronimo/specs/geronimo-saaj_1.3_spec/${geronimo.spec.saaj.version}/</argLine> > >>>>>>> + <!-- This fixes an issue on Sun JDKs caused by > the > >>>>>>> presence of jaxp-ri on the classpath --> > >>>>>>> + > >>>>>>> > > <argLine>-Dcom.sun.org.apache.xerces.internal.xni.parser.XMLParserConfiguration=com.sun.org.apache.xerces.internal.parsers.XIncludeParserConfiguration</argLine> > >>>>>>> </configuration> > >>>>>>> </plugin> > >>>>>>> </plugins> > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>> > >>>>>> -- > >>>>>> Amila Suriarachchi > >>>>>> WSO2 Inc. > >>>>>> blog: http://amilachinthaka.blogspot.com/ > >>>>>> > >>>>>> > >>>>> > >>>>> > >>>> > >>>> > >>>> -- > >>>> Thank you! > >>>> > >>>> > >>>> http://blogs.deepal.org > >>>> http://deepal.org > >>>> > >>>> > >>> > >> > >> > >> > >> -- > >> Sagara Gunathunga > >> > >> Blog - http://ssagara.blogspot.com > >> Web - > >> http://people.apache.org/~sagara/<http://people.apache.org/%7Esagara/> > >> > > > -- Amila Suriarachchi WSO2 Inc. blog: http://amilachinthaka.blogspot.com/
