>> Hi Jody,
>>
>> If you get the chance I'd be grateful if you can review some changes
>> that I've just made in the swing module.
>>
>> The reason that the Quickstart app needed JAI at runtime was because
>> grid coverage classes were referenced directly by the InfoTool class.
>> I've think I've fixed this by hiding the grid coverage classes in a
>> helper class that is loaded by reflection if required.
>
> Btw, the pure java version of JAI is in our repo, you can make it
> a dependency in your module if you need so.
>
> The reason we have JAI as a non transitive dependency in the GT2
> module boils down to avoid conflicts with the one installed in
> the JRE, should the user have one.
>
> In GS we do have a transitive dependency instead, so far we did not
> have much issues (some with users having an outdated JAI in their
> JRE, but very few and far apart afaik).
>
> I would be ok to just make it simpler and make JAI a transitive
> dependency. If someone really does not want it they can use
> Maven excludes to get rid of it.

I wonder if we can optimize for the 90% use case on both these dependencies:
- set up batik as an optional dependency for renderer (ie a compile
scope dependency?) - thus avoiding the exclude for batik
- set up JAI as a formal dependency; and the projects that have JAI
installed into their JRE can taken on the burden of providing an
exclude?

This way the demo examples would be light; and the use of exclude /
include would be confined to the rare cases (and not come up in our
demo code).

What do you think?
Jody

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to