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