[Geoserver-devel] CSW did not make it into extension by RC1

Hi,
I was looking at the release notes and noticed that CSW is not yet in extension despite
the positive vote for the associated GSIP

Do you think it will make it by RC2?

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


Yes, a pull request has just been made for this:

https://github.com/geoserver/geoserver/pull/299

Kind Regards
Niels

On 18/08/13 14:52, Andrea Aime wrote:

Hi,
I was looking at the release notes and noticed that CSW is not yet in extension despite
the positive vote for the associated GSIP

Do you think it will make it by RC2?

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

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

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk

_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

On Mon, Aug 19, 2013 at 4:08 PM, Niels Charlier <niels@anonymised.com> wrote:

Yes, a pull request has just been made for this:

https://github.com/geoserver/geoserver/pull/299

I believe you'll have to make one for 2.4.x too, which was branched off
yesterday.
At this point it should just be a cherry-pick away

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

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

Done https://github.com/geoserver/geoserver/pull/300

On 19/08/13 16:15, Andrea Aime wrote:

On Mon, Aug 19, 2013 at 4:08 PM, Niels Charlier <niels@anonymised.com <mailto:niels@anonymised.com>> wrote:

    Yes, a pull request has just been made for this:

    https://github.com/geoserver/geoserver/pull/299

I believe you'll have to make one for 2.4.x too, which was branched off yesterday.
At this point it should just be a cherry-pick away

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 Mon, Aug 19, 2013 at 4:08 PM, Niels Charlier <niels@anonymised.com> wrote:

Yes, a pull request has just been made for this:

https://github.com/geoserver/geoserver/pull/299

Hi Niels,
this pull request was merged on master yesterday and today we have the
results of
the nightly build.
As far as I can see there are issues with the zip contents:
http://gridlock.opengeo.org/geoserver/trunk/ext-latest/geoserver-2.5-SNAPSHOT-csw-plugin.zip

It contains a test jar that probably should not be there, and lacks the
jars for the csw extension
altogether

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 Tue, Aug 20, 2013 at 2:35 PM, Andrea Aime
<andrea.aime@anonymised.com>wrote:

Hi Niels,
this pull request was merged on master yesterday and today we have the
results of
the nightly build.
As far as I can see there are issues with the zip contents:

http://gridlock.opengeo.org/geoserver/trunk/ext-latest/geoserver-2.5-SNAPSHOT-csw-plugin.zip

It contains a test jar that probably should not be there, and lacks the
jars for the csw extension
altogether

Btw, if you want to make changes and try locally, you can do the following:
* cd release
* mvn clean install -nsu
* cd ..
* mvn assembly:attached

and then check the results in target/release.

From a cursory look, you did not make the release module depend on the csw

ones,
so their jars are not in the right place when assembly:attached builds the
zip files.

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

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

Yes you were right, sorry about that, a fix has been pull requested. 2.4.x request updated. Cheers Niels

···

On 20/08/13 14:37, Andrea Aime wrote:

On Tue, Aug 20, 2013 at 2:35 PM, Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi Niels,

this pull request was merged on master yesterday and today we have the results of
the nightly build.
As far as I can see there are issues with the zip contents:
http://gridlock.opengeo.org/geoserver/trunk/ext-latest/geoserver-2.5-SNAPSHOT-csw-plugin.zip

It contains a test jar that probably should not be there, and lacks the jars for the csw extension
altogether

Btw, if you want to make changes and try locally, you can do the following:

  • cd release
  • mvn clean install -nsu
  • cd …
  • mvn assembly:attached

and then check the results in target/release.

From a cursory look, you did not make the release module depend on the csw ones,
so their jars are not in the right place when assembly:attached builds the zip files.

On Tue, Aug 20, 2013 at 10:48 PM, Niels Charlier <niels@anonymised.com> wrote:

Yes you were right, sorry about that, a fix has been pull requested.
2.4.x request updated.

Hi,
had a quick look at the extension, now it contains a number of packaged
schemas normally
used for app-schema. I guess they are needed for the ISO mappings?

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, Aug 22, 2013 at 6:16 PM, Andrea Aime
<andrea.aime@anonymised.com>wrote:

On Tue, Aug 20, 2013 at 10:48 PM, Niels Charlier <niels@anonymised.com> wrote:

Yes you were right, sorry about that, a fix has been pull requested.
2.4.x request updated.

Hi,
had a quick look at the extension, now it contains a number of packaged
schemas normally
used for app-schema. I guess they are needed for the ISO mappings?

Ah, just noticed that CSW was not getting build with a -Prelease build,
just fixed it on trunk:
https://github.com/geoserver/geoserver/commit/c25da2ebca987688fb52d0228fa52be8ab4e4154

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

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

Yes, the feature types are automatically generated from the xsd files so they all need to be present. Cheers Niels

···

On 22/08/13 18:16, Andrea Aime wrote:

On Tue, Aug 20, 2013 at 10:48 PM, Niels Charlier <niels@anonymised.com> wrote:

Yes you were right, sorry about that, a fix has been pull requested. 2.4.x request updated.

Hi,
had a quick look at the extension, now it contains a number of packaged schemas normally
used for app-schema. I guess they are needed for the ISO mappings?

On Thu, Aug 22, 2013 at 6:42 PM, Niels Charlier <niels@anonymised.com> wrote:

On 22/08/13 18:16, Andrea Aime wrote:

On Tue, Aug 20, 2013 at 10:48 PM, Niels Charlier <niels@anonymised.com> wrote:

Yes you were right, sorry about that, a fix has been pull requested.
2.4.x request updated.

Hi,
had a quick look at the extension, now it contains a number of packaged
schemas normally
used for app-schema. I guess they are needed for the ISO mappings?

Yes, the feature types are automatically generated from the xsd files so
they all need to be present.

Very good then, just merged on 2.4.x as well.
I'm going to make a little bug fix in the DescribeRecordResponse schema
location (lacks a /),
so I guess I can backport the other commit I did as well

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

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

Thank you, Andrea

On 22/08/13 18:47, Andrea Aime wrote:

On Thu, Aug 22, 2013 at 6:42 PM, Niels Charlier <niels@anonymised.com <mailto:niels@anonymised.com>> wrote:

    On 22/08/13 18:16, Andrea Aime wrote:

    On Tue, Aug 20, 2013 at 10:48 PM, Niels Charlier <niels@anonymised.com
    <mailto:niels@anonymised.com>> wrote:

        Yes you were right, sorry about that, a fix has been pull
        requested. 2.4.x request updated.

    Hi,
    had a quick look at the extension, now it contains a number of
    packaged schemas normally
    used for app-schema. I guess they are needed for the ISO mappings?

    Yes, the feature types are automatically generated from the xsd
    files so they all need to be present.

Very good then, just merged on 2.4.x as well.
I'm going to make a little bug fix in the DescribeRecordResponse schema location (lacks a /),
so I guess I can backport the other commit I did as well

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

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