Re: [Geotools-devel] Why gt-main cannot be split: CommonFactoryFinder

2018-10-28 Thread Jody Garnett
Somewhat related question how much do we like the precompiled error message bundles? The main justification for this complexity (wasting strings in permgen space) looks like it has been somewhat addressed in Java 7 with string literals and string intern adding to a string pool - which is no longer

Re: [Geotools-devel] [Geoserver-devel] OSGeo Java Code Sprint 2018 (Stage 3)

2018-10-28 Thread Jody Garnett
Thanks James it was quite a week. On Sat, Oct 27, 2018 at 12:58 AM Jim Hughes wrote: > Hi all, > > As part of the GeoTools/GeoWebCache/GeoServer code sprint, we addressed > a number of concerns. The immediate goal was to do enough so that users > could run GeoServer on Java 9/10/11. Jody Garne

Re: [Geotools-devel] Why gt-main cannot be split: CommonFactoryFinder

2018-10-28 Thread Jody Garnett
Okay I will have a look :) On Sun, Oct 28, 2018 at 1:41 PM Andrea Aime wrote: > Hi Jody, > I stopped after realizing the issues with common factory finder and > reverted the changes. So nope, no branch, sorry. > > Cheers > Andrea > > Il Dom 28 Ott 2018, 21:26 Jody Garnett ha > scritto: > >> And

Re: [Geotools-devel] Why gt-main cannot be split: CommonFactoryFinder

2018-10-28 Thread Andrea Aime
Hi Jody, I stopped after realizing the issues with common factory finder and reverted the changes. So nope, no branch, sorry. Cheers Andrea Il Dom 28 Ott 2018, 21:26 Jody Garnett ha scritto: > Andrea do you have a branch started for this activity? > > I will also note that FileDataStoreFactory

Re: [Geotools-devel] Why gt-main cannot be split: CommonFactoryFinder

2018-10-28 Thread Jody Garnett
Andrea do you have a branch started for this activity? I will also note that FileDataStoreFactory lookup by common factory results in duplication, should delegate to DataStoreFactoryFinder (which delegates to DataAccessFactory). On Sun, Oct 28, 2018 at 1:57 AM Andrea Aime wrote: > Hi, > I was l

Re: [Geotools-devel] EMF Update - Inadvertent WFSConfiguration rename?

2018-10-28 Thread Jody Garnett
I think we should rename that back as I do not want to disrupt client code (those configuration objects are one of the few points of contact client code uses). That said I like the explicit naming - is it worth having the explicit naming and extending a deprecated Configuration class for exsisting

Re: [Geotools-devel] Travis run restart request

2018-10-28 Thread Andrea Aime
Restarted. You can also force Travis to rebuild by closing and reopening the pull request Cheers Andrea On Sun, Oct 28, 2018 at 2:48 PM Kurt Schwehr wrote: > Can someone with travis-ci permissions restart this test run? > > https://travis-ci.org/geotools/geotools/jobs/447301305 > > This is for

[Geotools-devel] Travis run restart request

2018-10-28 Thread Kurt Schwehr
Can someone with travis-ci permissions restart this test run? https://travis-ci.org/geotools/geotools/jobs/447301305 This is for https://github.com/geotools/geotools/pull/2129 that now has a covering test case. Thanks! -kurt ___ GeoTools-Devel mailing

[Geotools-devel] Jenkins build is back to normal : geotools-master #2705

2018-10-28 Thread monitor
See ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel

[Geotools-devel] Build failed in Jenkins: geotools-master #2704

2018-10-28 Thread monitor
See Changes: [Andrea Aime] Removing unused dependencies from modules, help parallelize the build [Andrea Aime] Untangle tile-client from unsupported gt-swing -- [...tru

Re: [Geotools-devel] EMF Update - Inadvertent WFSConfiguration rename?

2018-10-28 Thread Ian Turton
Yes, I did - it was causing a compile error in eclipse, but I suspect that was a corrupt or out of date jar thing. So it could be reverted. I'm running the 2.15.0v20180723 version of the EMF base runtime, and apparently 2.14.0.v20180706 of the EMF code generation - both are what update considers t

[Geotools-devel] Why gt-main cannot be split: CommonFactoryFinder

2018-10-28 Thread Andrea Aime
Hi, I was looking at splitting main into smaller bits to avoid some split package issues, ideally trying to to towards a gt-data (everything vector), gt-style (styling) and and almost empty gt-main to collect the residuals, with gt-main depending on the two above... and realized it's not possible.

Re: [Geotools-devel] EMF Update - Inadvertent WFSConfiguration rename?

2018-10-28 Thread Jody Garnett
So just to confirm you did rename WFSConfiguration_1_0.java by hand :) Any idea about my inability to open the genmodel files? Can I confirm what version we are using or something. I did install 18.9 (4.9.0) eclipse modeling distribution, what are you using? On Sat, Oct 27, 2018 at 1:28 AM Ian Tu

Re: [Geotools-devel] Dropping the spatialite store?

2018-10-28 Thread Andrea Aime
HI Brad, well it's a good start, thank you for that, we'd just need someone else to pitch in some maintenance effort. Come on guys and gals, don't be shy :-p Cheers Andrea On Sat, Oct 27, 2018 at 11:59 PM wrote: > I have some legacy with the main spatialite project (sometime committer, > good