[Geoserver-devel] removing xerces ready to go

Hi all,

With the app-schema test issue resolved i believe the changes to remove xerces as a dependency are ready to go. All unit tests (both gs and gt) are passing and i just ran cite tests for gs with no problems.

So… shall we go ahead? Do we need a proposal for this one?

And then there is the question of where to push these changes? Shall we go with both 2.3.x and master? Or just let it sit on master for a while? I don’t have a strong preference on this one.

-Justin


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

On Thu, Jan 31, 2013 at 6:28 PM, Justin Deoliveira <jdeolive@anonymised.com> wrote:

Hi all,

With the app-schema test issue resolved i believe the changes to remove xerces as a dependency are ready to go. All unit tests (both gs and gt) are passing and i just ran cite tests for gs with no problems.

So… shall we go ahead? Do we need a proposal for this one?

And then there is the question of where to push these changes? Shall we go with both 2.3.x and master? Or just let it sit on master for a while? I don’t have a strong preference on this one.

Just my opinion, but:

  • don’t seem proposal worthy to me
  • let’s go on master, see how that works, and eventually backport to 2.2.x later?

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


On Thu, Jan 31, 2013 at 10:54 AM, Andrea Aime
<andrea.aime@anonymised.com>wrote:

On Thu, Jan 31, 2013 at 6:28 PM, Justin Deoliveira <jdeolive@anonymised.com>wrote:

Hi all,

With the app-schema test issue resolved i believe the changes to remove
xerces as a dependency are ready to go. All unit tests (both gs and gt) are
passing and i just ran cite tests for gs with no problems.

So... shall we go ahead? Do we need a proposal for this one?

And then there is the question of where to push these changes? Shall we
go with both 2.3.x and master? Or just let it sit on master for a while? I
don't have a strong preference on this one.

Just my opinion, but:
- don't seem proposal worthy to me
- let's go on master, see how that works, and eventually backport to 2.2.x
later?

Cool sounds good. And just to clarify do you mean 2.3.x?

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.

Woops yes sorry
/me has to reassign the variable “stable series” from 2.2.x to 2.3.x … may take some days :-p

Cheers
Andrea

···
  • don’t seem proposal worthy to me
  • let’s go on master, see how that works, and eventually backport to 2.2.x later?

Cool sounds good. And just to clarify do you mean 2.3.x?

Light blue touch paper and stand back!

This is a dependency adjustment, not a functional change in the API, so I do not believe a proposal is required.

Kind regards,
Ben.

On 01/02/13 01:28, Justin Deoliveira wrote:

Hi all,

With the app-schema test issue resolved i believe the changes to remove
xerces as a dependency are ready to go. All unit tests (both gs and gt)
are passing and i just ran cite tests for gs with no problems.

So... shall we go ahead? Do we need a proposal for this one?

And then there is the question of where to push these changes? Shall we
go with both 2.3.x and master? Or just let it sit on master for a while?
I don't have a strong preference on this one.

-Justin

--
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

Ok folks, changes pushed onto geotools and geoserver master.

···

On Thu, Jan 31, 2013 at 5:46 PM, Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com> wrote:

Light blue touch paper and stand back!

This is a dependency adjustment, not a functional change in the API, so I do not believe a proposal is required.

Kind regards,
Ben.

On 01/02/13 01:28, Justin Deoliveira wrote:

Hi all,

With the app-schema test issue resolved i believe the changes to remove
xerces as a dependency are ready to go. All unit tests (both gs and gt)
are passing and i just ran cite tests for gs with no problems.

So… shall we go ahead? Do we need a proposal for this one?

And then there is the question of where to push these changes? Shall we
go with both 2.3.x and master? Or just let it sit on master for a while?
I don’t have a strong preference on this one.

-Justin


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.

Justin,

my pull request to remove xerces from gt-app-schema-resolver that you merged into your repo appears to have gotten lost in the rebasing and merging into geotools master:
https://github.com/jdeolive/geotools/pull/1

I just cherry-picked it onto geotools master as 690678886e56cb77add77fd9f94cdbc1ddc3fa89; this should be included in any backport to stable for full dexercesification.

(This really belongs on geotools-devel)

Kind regards,
Ben.

On 01/02/13 22:45, Justin Deoliveira wrote:

Ok folks, changes pushed onto geotools and geoserver master.

On Thu, Jan 31, 2013 at 5:46 PM, Ben Caradoc-Davies
<Ben.Caradoc-Davies@anonymised.com <mailto:Ben.Caradoc-Davies@anonymised.com>> wrote:

    Light blue touch paper and stand back!

    This is a dependency adjustment, not a functional change in the API,
    so I do not believe a proposal is required.

    Kind regards,
    Ben.

    On 01/02/13 01:28, Justin Deoliveira wrote:

        Hi all,

        With the app-schema test issue resolved i believe the changes to
        remove
        xerces as a dependency are ready to go. All unit tests (both gs
        and gt)
        are passing and i just ran cite tests for gs with no problems.

        So... shall we go ahead? Do we need a proposal for this one?

        And then there is the question of where to push these changes?
        Shall we
        go with both 2.3.x and master? Or just let it sit on master for
        a while?
        I don't have a strong preference on this one.

        -Justin

        --
        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.

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

I concur.

2c/
Gabriel

On Thu, Jan 31, 2013 at 3:10 PM, Andrea Aime
<andrea.aime@anonymised.com> wrote:

On Thu, Jan 31, 2013 at 7:08 PM, Justin Deoliveira <jdeolive@anonymised.com>
wrote:

Just my opinion, but:

- don't seem proposal worthy to me
- let's go on master, see how that works, and eventually backport to
2.2.x later?

Cool sounds good. And just to clarify do you mean 2.3.x?

Woops yes sorry
/me has to reassign the variable "stable series" from 2.2.x to 2.3.x ... may
take some days :-p

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

-------------------------------------------------------

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_jan
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

--
Gabriel Roldan
OpenGeo - http://opengeo.org
Expert service straight from the developers.