Justin Deoliveira ha scritto: > Hi Andrea, > > Looking over the recent changes to GeoServerExtensions i am starting to > have second thoughts about adding a direct dependency on the servlet api > from platform. > > I know I am not making a strong case for it but i am starting to think > perhaps we should try to avoid the dependency if we can. I think it > would be easy enough to just add it to a class in main where the > dependency already exists. Something like: > > GeoServerUtil.getProperty( String, ApplicationContext ) > GeoServerUtil.getProperty( String, ServletContext ) > > > What do you think? What do others think?
Yeah, I had the same thought when I noticed I had to add the dependency on servlet-api to platform... then I realized I had already spent more than one day on that bug fix and swallowed the doubt to get unstuck. On the downside, having a proliferation of utility classes around does not look too good either. Does it make sense to include that functionality into one of the many existing xxxUtil classes? Generally speaking, that dependency is an issue not from the point of view of what GeoServer is (it's a webapp, servlets are the defining element of a java webapp), but from the pov of trying to make GeoServer a framework. Which is something I do share, I'd love to see GS one day being as scriptable and adaptable as MapServer with MapScript. The question is, do we want to make GeoServer a framework usable outside of a web application? That seems a little more far fetched, thought it's not the first time I think of making a GeoClient out of the code of GeoServer ;) Cheers Andrea -- Andrea Aime OpenGeo - http://opengeo.org Expert service straight from the developers. ------------------------------------------------------------------------- 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 Geoserver-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geoserver-devel
