I have no need for it to be in the 10 release.
I did find a bug when building using jdk7 on a different matter altogether.
I have talked to Andrea about this.
Brett
Sent from my iPad
On 18/07/2013, at 9:52 AM, "Jody Garnett"
<jody.garn...@gmail.com<mailto:jody.garn...@gmail.com>> wrote:
Were you expecting this in the current release? If so please advise.
(And thanks for stepping up with all the pull request Brett)
--
Jody Garnett
On Thursday, 18 July 2013 at 2:58 AM, Simone Giannecchini wrote:
Ciao Bret,
I had a quick look and I am happy withe changes. I will create a jira
and backport to 9.x
Regards,
Simone Giannecchini
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for
more information.
==
Ing. Simone Giannecchini
@simogeo
Founder/Director
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 333 8128928
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
On Wed, Jul 17, 2013 at 3:28 AM, Brett Walker
<brett.wal...@geometryit.com<mailto:brett.wal...@geometryit.com>> wrote:
-----Original Message-----
From: Michael Bedward [mailto:michael.bedw...@gmail.com]
Sent: Wednesday, 17 July 2013 11:18 AM
To: Brett Walker
Cc:
geotools-devel@lists.sourceforge.net<mailto:geotools-devel@lists.sourceforge.net>;
jody.garn...@gmail.com<mailto:jody.garn...@gmail.com>
Subject: Re: Pull Request to upgrade JAITools
Hi Brett,
Thanks for this work.
Yes, there were substantial changes to RangeLookupTable and related classes to
fix thread safety, so it's no surprise that you had to hack things at the
GeoTools end. However, I don't know the CoverageUtilities code at all - it
would be best to ask Simone, Andrea or Daniele to review it.
Michael
On 16 July 2013 22:21, Brett Walker
<brett.wal...@geometryit.com<mailto:brett.wal...@geometryit.com>> wrote:
Hi,
I have made a pull request to upgrade JAITools to version 1.3.1.
To get the build to pass it seems that I had to brutalise a method in
CoverageUtilities.java
(https://github.com/brett-walker/geotools/commit/35ee8f8434f2dbb435622
671f725a08e295df848)
A question for Jody or Michael.
Do the changes look right? Have I changed the method's meaning too
drastically? Or is the change from version 1.3.0 to 1.3.1 to drastic
for CoverageUtilities do be altered that the upgrade is not possible?
The process-raster module, of which the CoverageUtilities is apart, is
in the unsupported part of GeoTools.
Brett
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net<mailto:GeoTools-Devel@lists.sourceforge.net>
https://lists.sourceforge.net/lists/listinfo/geotools-devel
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
