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
