> Here we go. Hum, logging should not be configured with
> GeoServer.configureGeoServerLogging, after the changes for
> http://jira.codehaus.org/browse/GEOS-1671 nothing is using that method 
> anymore (at least on my checkout), and afaik it wasn't used by startup
> code even before that change (only test setup used it).
Right, forgot about that. This patch is a couple of months old so it was 
done before that change. I will update it to reflect the new way of 
things. Also do need to keep the method on GEoServer around? Should we 
kill it. Maybe just mark it with a big nasty comment for now.
> 
> The code that actually does something is in LoggingInitializer, the
> method is basically the same.
> 
> Besides this the patch looks good to me.
> Cheers
> Andrea
> 


-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to