Martin Desruisseaux ha scritto:
> Andrea Aime a écrit :
>> This is a bug in the dependencies of the renderer module that has to be
>> fixed, but suggesting the removal of the gdal based module from GeoTools
>> as a cure is an exageration. The deps must be fixed instead.
> 
> Yes I agree. If the suggested cure was the complete removal of GDAL 
> modules, I didn't realized that. As far as I known, Johan was only 
> concerned by the dependencies that appears in "render".

Ok, then I misunderstood Johan proposal, sorry :)
I'm of course very pro removing any unnecessary dependency from
the renderer module.

However, since we're talking, I did run mvn dependency:tree on my
checkout of gt2-trunk and got the following:

[INFO] 
------------------------------------------------------------------------
[INFO] Building Render
[INFO]    task-segment: [dependency:tree]
[INFO] 
------------------------------------------------------------------------
[INFO] snapshot org.geotools:gt-sample-data:2.5-SNAPSHOT: checking for 
updates from refractions
[INFO] [dependency:tree]
[INFO] org.geotools:gt-render:jar:2.5-SNAPSHOT
[INFO] +- org.geotools:gt-main:jar:2.5-SNAPSHOT:compile
[INFO] |  +- org.geotools:gt-api:jar:2.5-SNAPSHOT:compile
[INFO] |  +- com.vividsolutions:jts:jar:1.9.1-SNAPSHOT:compile
[INFO] |  +- jdom:jdom:jar:1.0:compile
[INFO] |  \- commons-beanutils:commons-beanutils:jar:1.7.0:compile
[INFO] |     \- commons-logging:commons-logging:jar:1.1.1:compile 
(version managed from 1.0.3)
[INFO] +- org.geotools:gt-coverage:jar:2.5-SNAPSHOT:compile
[INFO] |  \- org.geotools:gt-referencing:jar:2.5-SNAPSHOT:compile
[INFO] |     +- java3d:vecmath:jar:1.3.1:compile
[INFO] |     +- commons-pool:commons-pool:jar:1.3:compile
[INFO] |     \- org.geotools:gt-metadata:jar:2.5-SNAPSHOT:compile
[INFO] |        +- org.opengis:geoapi:jar:2.2-SNAPSHOT:compile
[INFO] |        \- javax.units:jsr108:jar:0.01:compile
[INFO] +- org.geotools:gt-cql:jar:2.5-SNAPSHOT:compile
[INFO] +- 
it.geosolutions.imageio-ext:imageio-ext-arcgrid:jar:1.0-SNAPSHOT:compile
[INFO] |  \- 
it.geosolutions.imageio-ext:imageio-ext-customstreams:jar:1.0-SNAPSHOT:compile
[INFO] |     \- commons-httpclient:commons-httpclient:jar:3.0.1:compile
[INFO] |        \- commons-codec:commons-codec:jar:1.2:compile
[INFO] +- org.geotools:gt-epsg-hsql:jar:2.5-SNAPSHOT:test
[INFO] |  \- hsqldb:hsqldb:jar:1.8.0.7:test
[INFO] +- org.geotools:gt-sample-data:jar:2.5-SNAPSHOT:test
[INFO] +- javax.media:jai_codec:jar:1.1.3:test
[INFO] +- javax.media:jai_imageio:jar:1.1:test
[INFO] +- batik:batik-transcoder:jar:1.6:compile
[INFO] +- batik:batik-util:jar:1.6:compile
[INFO] +- xerces:xercesImpl:jar:2.7.1:compile
[INFO] +- xml-apis:xml-apis-xerces:jar:2.7.1:compile
[INFO] +- batik:batik-svg-dom:jar:1.6:compile
[INFO] +- batik:batik-dom:jar:1.6:compile
[INFO] +- batik:batik-css:jar:1.6:compile
[INFO] +- batik:batik-bridge:jar:1.6:compile
[INFO] +- batik:batik-gvt:jar:1.6:compile
[INFO] +- batik:batik-ext:jar:1.6:compile
[INFO] +- batik:batik-xml:jar:1.6:compile
[INFO] +- batik:batik-script:jar:1.6:compile
[INFO] +- batik:batik-awt-util:jar:1.6:compile
[INFO] +- batik:batik-parser:jar:1.6:compile

I believe the problem is with:
  it.geosolutions.imageio-ext:imageio-ext-arcgrid:jar:1.0-SNAPSHOT:compile

Hmm... this should be a test depedendency, not a compile one...
Looks like a very easy fix.

> 
>>> "imageio-ext" is not a dependency like the other ones to us. Some 
>>> libraries like Apache, JTS and Sun Microsystems are widely trusted 
>>> (even if they still have bugs). "imageio-ext" is a work in progress 
>>> by what looks like a subset of the GeoTools community (maybe I'm 
>>> wrong - I don't know the list of contributors). Being isolated on an 
>>> external SVN, I don't think it has received the reviewing it would 
>>> have got on GeoTools SVN (at least not from me - I don't know for 
>>> other).
>>
>> It's an unsupported module, that level of review is simply not required.
>> If you believe it should, you should make up a proposal to change the 
>> rules of entry into gt2 and raise the bar for entry and have the
>> PMC vote on it.
> 
> I was talking about the "imageio-ext" project on dev.java.net, not the 
> unsupported modules in GeoTools SVN.

Oh, I see... but then again, it's not like the entire GeoTools depends
on it, there is only one unsupported module and the above dependency
which, afaik, it's only used for tests.

Cheers
Andrea

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to