Hmm, ok.  I'll buy it.  That makes sense to me.

Can we (as you suggested, martin) roll this move into the whole "remove
the gt2- prefix" change that we're doing later?

--saul


On Thu, 2008-03-13 at 20:44 +0100, Martin Desruisseaux wrote:
> Saul Farber a écrit :
> > I'm a little hesitant to move the sde-dummy code away from the sde
> > module.  It is true that it's only used (currently) for building, but
> > why does that mean that we should move it away from its current home?
> > 
> > I guess I just think that the sde-dummy module isn't really of any
> > general use, except to the arcsde-datastore module.  the jaxb-dummy
> > module, however, *may* be of use to many plugins, so perhaps it could
> > have a more general location?
> 
> Actually the rational is not general use. The way I saw the directory 
> structure is:
> 
>    - build:   Everything needed for building, no matter if of general use of 
> not.
>    - modules: All GeoTools code to be included in releases (minus 
> unsupported).
> 
> The "build" directory already contains other stuff that are very specific to 
> a 
> particular module, for example "modules/extension/openoffice".
> 
> By putting "arcsde-dummy" in "modules", you inherit the Maven definitions 
> from 
> the parent pom.xml (assuming we don't trink Maven config, which is possible). 
> Inherited definition includes javadoc settings, assembly and the like, which 
> may 
> not be what we intented (we don't want arcsde-dummy to appears in javadoc or 
> to 
> be included in assemblies).
> 
> The Maven directory layout is not to be understood in the sense of 
> "package-privated" like Java, but rather in the sense of class hierarchy 
> (subdirectories inherits the behavior from their parent directory).
> 
> If "arcsde-dummy" was defined in "build" instead, it would inherit from a 
> parent 
> pom.xml better suited for dummy modules, without javadoc or assembly 
> instructions. It would clearly appears in a "dummy" directory in the Maven 
> repository, instead of looking like a real GeoTools module as it currently is 
> ("gt2-sde-dummy" in http://maven.geotools.fr/repository/org/geotools/).
> 
> 
>       Martin



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to