[Geoserver-devel] Xalan in app-schema

[Correct list this time]

More followup from Monday's meeting:

The JDK Xalan looks to be 2.6.0. It will not compile the GMLOutputFormat stylesheet used for app-schema WFS output:
https://jira.codehaus.org/browse/GEOS-5352

xalan-2.7.0 is shipped with the app-schema plugin.

On 29/10/12 22:15, Ben.Caradoc-Davies@anonymised.com wrote:

Dropping xerces (and friends)
---------------------------------------

Justin: cleaning up the explicit deps on Xerces using the one in the JDK
Last issue: app-schema resolver. Ben?
app-schema-resolver: move validation support to test and port to new non-xerces api - BCD/RA (and Justin to send draft patch to Ben)

Other? JDOM, streaming parsers, Xalan

--
Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com>
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

Justin,

please consider pull request:
https://github.com/geoserver/geoserver/pull/54

See also: https://jira.codehaus.org/browse/GEOS-5352

In addition to app-schema pom changes, this commit makes two tiny changes in wfs to the XSLTs used by GML3OutputFormat and GML32OutputFormat to workaround a problem in the the Xalan 2.6.0 shipped with the JDK.

The fix, which was suggested by Pavel Golodoniuc, is to add brackets "()" after processing-instruction.

xalan is still used in geoxacml (community) and in the test phase of ows and wms, so there is a remaining dependency version set in the top-level pom. Until this change, xalan was included in the app-schema plugin, but now we no longer need to ship xalan with GeoServer .

Kind regards,
Ben.

On 31/10/12 14:34, Ben Caradoc-Davies wrote:

[Correct list this time]

More followup from Monday's meeting:

The JDK Xalan looks to be 2.6.0. It will not compile the GMLOutputFormat
stylesheet used for app-schema WFS output:
https://jira.codehaus.org/browse/GEOS-5352

xalan-2.7.0 is shipped with the app-schema plugin.

On 29/10/12 22:15, Ben.Caradoc-Davies@anonymised.com wrote:

Dropping xerces (and friends)
---------------------------------------

Justin: cleaning up the explicit deps on Xerces using the one in the JDK
Last issue: app-schema resolver. Ben?
app-schema-resolver: move validation support to test and port to new non-xerces api - BCD/RA (and Justin to send draft patch to Ben)

Other? JDOM, streaming parsers, Xalan

--
Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com>
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

Nice work Ben! I will try merge/cleanup this branch later in the week / on the weekend.

On Tue, Nov 6, 2012 at 3:36 AM, Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com> wrote:

Justin,

please consider pull request:
https://github.com/geoserver/geoserver/pull/54

See also: https://jira.codehaus.org/browse/GEOS-5352

In addition to app-schema pom changes, this commit makes two tiny changes in wfs to the XSLTs used by GML3OutputFormat and GML32OutputFormat to workaround a problem in the the Xalan 2.6.0 shipped with the JDK.

The fix, which was suggested by Pavel Golodoniuc, is to add brackets “()” after processing-instruction.

xalan is still used in geoxacml (community) and in the test phase of ows and wms, so there is a remaining dependency version set in the top-level pom. Until this change, xalan was included in the app-schema plugin, but now we no longer need to ship xalan with GeoServer .

Kind regards,
Ben.

On 31/10/12 14:34, Ben Caradoc-Davies wrote:

[Correct list this time]

More followup from Monday’s meeting:

The JDK Xalan looks to be 2.6.0. It will not compile the GMLOutputFormat
stylesheet used for app-schema WFS output:
https://jira.codehaus.org/browse/GEOS-5352

xalan-2.7.0 is shipped with the app-schema plugin.

On 29/10/12 22:15, Ben.Caradoc-Davies@anonymised.com wrote:

Dropping xerces (and friends)

Justin: cleaning up the explicit deps on Xerces using the one in the JDK
Last issue: app-schema resolver. Ben?
app-schema-resolver: move validation support to test and port to new non-xerces api - BCD/RA (and Justin to send draft patch to Ben)

Other? JDOM, streaming parsers, Xalan


Ben Caradoc-Davies Ben.Caradoc-Davies@anonymised.com
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre


Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

Justin,

this pull request is for master and can be applied immediately. It has no dependency on the xerces cleanup. I can push it myself one you are happy with the four extra characters added to the wfs module.

Kind regards,
Ben.

On 06/11/12 22:57, Justin Deoliveira wrote:

Nice work Ben! I will try merge/cleanup this branch later in the week /
on the weekend.

On Tue, Nov 6, 2012 at 3:36 AM, Ben Caradoc-Davies
<Ben.Caradoc-Davies@anonymised.com <mailto:Ben.Caradoc-Davies@anonymised.com>> wrote:

    Justin,

    please consider pull request:
    https://github.com/geoserver/__geoserver/pull/54
    <https://github.com/geoserver/geoserver/pull/54&gt;

    See also: https://jira.codehaus.org/__browse/GEOS-5352
    <https://jira.codehaus.org/browse/GEOS-5352&gt;

    In addition to app-schema pom changes, this commit makes two tiny
    changes in wfs to the XSLTs used by GML3OutputFormat and
    GML32OutputFormat to workaround a problem in the the Xalan 2.6.0
    shipped with the JDK.

    The fix, which was suggested by Pavel Golodoniuc, is to add brackets
    "()" after processing-instruction.

    xalan is still used in geoxacml (community) and in the test phase of
    ows and wms, so there is a remaining dependency version set in the
    top-level pom. Until this change, xalan was included in the
    app-schema plugin, but now we no longer need to ship xalan with
    GeoServer .

    Kind regards,
    Ben.

    On 31/10/12 14:34, Ben Caradoc-Davies wrote:

        [Correct list this time]

        More followup from Monday's meeting:

        The JDK Xalan looks to be 2.6.0. It will not compile the
        GMLOutputFormat
        stylesheet used for app-schema WFS output:
        https://jira.codehaus.org/__browse/GEOS-5352
        <https://jira.codehaus.org/browse/GEOS-5352&gt;

        xalan-2.7.0 is shipped with the app-schema plugin.

        On 29/10/12 22:15, Ben.Caradoc-Davies@anonymised.com wrote:

            Dropping xerces (and friends)
            ------------------------------__---------

            Justin: cleaning up the explicit deps on Xerces using the
            one in the JDK
            Last issue: app-schema resolver. Ben?
            app-schema-resolver: move validation support to test and
            port to new non-xerces api - BCD/RA (and Justin to send
            draft patch to Ben)

            Other? JDOM, streaming parsers, Xalan

    --
    Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com>
    Software Engineer
    CSIRO Earth Science and Resource Engineering
    Australian Resources Research Centre

--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

--
Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com>
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

Ahh, sorry my bad :slight_smile: Sounds great to me. +1

On Tue, Nov 6, 2012 at 7:10 PM, Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com> wrote:

Justin,

this pull request is for master and can be applied immediately. It has no dependency on the xerces cleanup. I can push it myself one you are happy with the four extra characters added to the wfs module.

Kind regards,
Ben.

On 06/11/12 22:57, Justin Deoliveira wrote:

Nice work Ben! I will try merge/cleanup this branch later in the week /
on the weekend.

On Tue, Nov 6, 2012 at 3:36 AM, Ben Caradoc-Davies

<Ben.Caradoc-Davies@anonymised.com mailto:[Ben.Caradoc-Davies@csiro.au](mailto:Ben.Caradoc-Davies@anonymised.com)> wrote:

Justin,

please consider pull request:

https://github.com/geoserver/__geoserver/pull/54
<https://github.com/geoserver/geoserver/pull/54>

See also: https://jira.codehaus.org/__browse/GEOS-5352

<https://jira.codehaus.org/browse/GEOS-5352>

In addition to app-schema pom changes, this commit makes two tiny
changes in wfs to the XSLTs used by GML3OutputFormat and
GML32OutputFormat to workaround a problem in the the Xalan 2.6.0
shipped with the JDK.

The fix, which was suggested by Pavel Golodoniuc, is to add brackets
“()” after processing-instruction.

xalan is still used in geoxacml (community) and in the test phase of
ows and wms, so there is a remaining dependency version set in the
top-level pom. Until this change, xalan was included in the
app-schema plugin, but now we no longer need to ship xalan with
GeoServer .

Kind regards,
Ben.

On 31/10/12 14:34, Ben Caradoc-Davies wrote:

[Correct list this time]

More followup from Monday’s meeting:

The JDK Xalan looks to be 2.6.0. It will not compile the
GMLOutputFormat
stylesheet used for app-schema WFS output:

https://jira.codehaus.org/__browse/GEOS-5352

<https://jira.codehaus.org/browse/GEOS-5352>

xalan-2.7.0 is shipped with the app-schema plugin.

On 29/10/12 22:15, Ben.Caradoc-Davies@anonymised.com wrote:

Dropping xerces (and friends)

------------------------------__---------

Justin: cleaning up the explicit deps on Xerces using the
one in the JDK
Last issue: app-schema resolver. Ben?
app-schema-resolver: move validation support to test and
port to new non-xerces api - BCD/RA (and Justin to send
draft patch to Ben)

Other? JDOM, streaming parsers, Xalan


Ben Caradoc-Davies Ben.Caradoc-Davies@anonymised.com
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre


Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.


Ben Caradoc-Davies Ben.Caradoc-Davies@anonymised.com
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre


Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.