From: Justin D. <jde...@op...> - 2012-10-23 12:59:37
|
On Tue, Oct 23, 2012 at 3:02 AM, Andrea Aime <and...@ge...>wrote: > On Tue, Oct 23, 2012 at 2:29 AM, Justin Deoliveira <jde...@op...>wrote: > >> Hi all, >> >> So I recently I took a crack at removing the xerces dependency as we >> discussed in our last skype meeting. For the most part this was relatively >> straight forward. Unfortunately one thing i wasn't quite able to figure out >> was some stuff in app-schema-resolver module. The AppSchemaValidator class >> uses an implementation of what was XMLEntityResolver directly in xerces. I >> tried porting this to the jaxp EntityResolver interface which is included >> in the JDK, but couldn't make it work. So I think i will need some help >> there. Or we can simply keep xerces as a dependency of just app-schema. >> >> Here is the branch I am currently working on. >> >> https://github.com/jdeolive/geotools/tree/noxmldeps >> >> And the corresponding geoserver branch: >> >> https://github.com/jdeolive/geoserver/tree/noxmldeps >> >> WIth these changes all unit tests pass in both code bases, and so do the >> GeoServer cite tests. >> > > Well done! Hopefully Ben & co will chime in for the last push on app-schema > Indeed. And also while we are at it i will pose the question. There is also the opportunity to prune out some other xml dependencies as well. Namely jdom, xpp and stax from what i can tell. All seem to have equivalents built into the jdk. Is it worth pruning them out? > > Cheers > Andrea > > -- > == > Our support, Your Success! Visit http://opensdi.geo-solutions.it for more > information. > == > > Ing. Andrea Aime > @geowolf > Technical Lead > > GeoSolutions S.A.S. > Via Poggio alle Viti 1187 > 55054 Massarosa (LU) > Italy > phone: +39 0584 962313 > fax: +39 0584 1660272 > mob: +39 339 8844549 > > http://www.geo-solutions.it > http://twitter.com/geosolutions_it > > ------------------------------------------------------- > > -- Justin Deoliveira OpenGeo - http://opengeo.org Enterprise support for open source geospatial. |