[Geoserver-devel] GeoTools / GeoServer Meeting 2014-04-29

GeoTools / GeoServer Meeting 2014-04-29

Attending
---------

Ben Caradoc-Davies
Jukka Rahkonen
Jody Garnett
Andrea Aime

Agenda
------

- previous meeting
- fonts and testing
- cite tests and build server
- Java 7
- Next Release
- filter/expression cleanup
- pull request round up / nagging

Actions:
- JG: Test Fonts on OSX
- JG: Pester GeoSever list to vote on Java 7 proposal
- BCD: Raise CITE tests on geoserver-devel mailing list
- AA: Email lists asking for a release volunteer (May 18th)

Previous Meeting
----------------

ACTIONS from last meeting:
- (done) Ben to follow up reported rendering failure on openjdk with Justin
- (done) All: Submit your foss4g presentation!

Fonts and Testing
-----------------

Both java image diff and consistent font (LabelShieldTest) have arrived on master.
Jody will need to test on Mac (ie font metrics differences?)

Java 7
------

Ares has Oracle JDK7 - we can update configuration for master.. when proposals accepted?
CSIRO build box is using Oracle JDK6 and can be switched to OpenJDK 7.
https://cgsrv8.arrc.csiro.au/jenkins/view/geoserver-master/
https://www.seegrid.csiro.au/wiki/Infosrvices/JenkinsGeoserverMaster

Ideally CSIRO could set up OpenJDK 7 so we can get a second opinon. And OSGeo Live would like that as well.

1) GeoTools "Upgrade master to Java 7" is approved

http://docs.codehaus.org/display/GEOTOOLS/Upgrade+master+to+Java+7

2) GeoServer proposal is not approved yet ...

http://geoserver.org/display/GEOS/GSIP+112+-+Upgrade+master+to+Java+7

Cite tests and build server
---------------------------

Justin/Andrea email discussion about CITE tests not working consistently/all the time. They are stating to be ignored ...

Is multithreaded builds being an issue -T2C ? Andrea builds this way all the time so that is probably not it.

Justin is the guy, Andrea has offered to help. Stay tunned?

Next Release (May 18th)
-----------------------

We need a volunteer GeoTools 11.1 / GeoServer 2.5.1 :slight_smile:

http://geoserver.org/display/GEOS/GeoTools+and+GeoServer+release+schedule

Filter/expression cleanup
-------------------------

It is finally happening ... technical debt since GeoTools 2.3

https://github.com/geotools/geotools/pull/434

GeoServer so far works with no change :slight_smile: Well one change to StreamingSVGMap that has already been applied.

Cumulative work, easy to review as it does one thing at a time.
However it may be easier to squash and review?

Andrea says leave it as it is.

Justin or Andrea should do this review?

Q: Is Kevin okay? This cannot be backported?
A: Kevin applied just the "equals" fix (ie use of getExpression1() and getExpression2()) and that was enough for his customer

Pull request round up / nagging
-------------------------------

Q: A zillion pull requests are happening, anything that needs attention?

1) CQL fix - jody needs to apply feedback, and then merge to master.

2) Security configuration files showing up in wrong place / login configuration

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

Jody says to check out the namespace.xml / workspace.xml file handling logic in GeoServerPersister.
- Code using new File( dir, path ) ended up in the wrong spot when dir was was not found (ie dir was null)
- Fix to FileUtils to allow it to copy directories was important.

Andrea is working on this.

3) fix jetty log versions dependency

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

Caution with this on stable (as removing sf4l jar?)
Ben will comment on and close the 2.4.x pull request.
https://github.com/geoserver/geoserver/pull/563

4) Still waiting on contribution agreement for legend graphics :slight_smile:

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