Thanks! See latest comment for pull request. Its still a pull request against 14.x because I'd like to get it in for uDig. Once it's accepted I'm going to merge it into master follwing the procedure http://docs.geotools.org/latest/developer/procedures/stable.html
@Ben, the build at travis has been started and I don't expect any broken code. Have you started releace procedure already? Sorry for the late incomming, it would be great if the fix can make it either - Frank 2015-11-17 3:53 GMT+01:00 Jody Garnett <[email protected]>: > Commented on the issue, like the idea of being able to remove an > EnvVariable (it will be a bit of race if you need this in for 14.1) > > -- > Jody Garnett > > On 16 November 2015 at 13:05, Frank Gasdorf <[email protected]> > wrote: > >> Hello folks, >> >> recently I filed GEOT-5302 >> <https://osgeo-org.atlassian.net/browse/GEOT-5302> for EnvFunction >> behavior. At the moment its not possible to remove an entry. >> >> I used this in an OSGi environment where I'd inject dynamically set >> values from an OSGi Service. Its possible that the service can be stopped >> at runtime and now I need an option to "unset" the value. >> >> In a SLD I used env to get the value and dependent of its value images >> are rendered. >> >> At the moment I'm tending to provide two additional functions, for global >> and local entries to remove these. Then I guess its clear how to use the >> static function. Independenly I would delegate to to remove if the value >> for setGlobalValue or setLocalValue is null. >> >> Please have a look to issue comments. >> >> Does this makes sense to you? What's your opinion? >> >> Thanks for sharing your thoughts >> >> - Frank >> >> >> ------------------------------------------------------------------------------ >> Presto, an open source distributed SQL query engine for big data, >> initially >> developed by Facebook, enables you to easily query your data on Hadoop in >> a >> more interactive manner. Teradata is also now providing full enterprise >> support for Presto. Download a free open source copy now. >> http://pubads.g.doubleclick.net/gampad/clk?id=250295911&iu=/4140 >> _______________________________________________ >> GeoTools-Devel mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/geotools-devel >> >> >
------------------------------------------------------------------------------
_______________________________________________ GeoTools-Devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
