Hi Andrea,
Good catch on the gt-process dependency. This seems like it is something
that can be resolved relatively easily, as the only actual dependency is in
the gt-ysld test cases. It looks like the core ylsd code is designed in a
way that can use gt-process without explicitly depending upon it.
If we can graduate the process module to supported that would be even
better.
There are a few of other things that need to be done before gt-ysld is
ready for graduation (such as documentation), so this is really just one
more to add to that list.
I'm aiming to get this in before the 16.2 release next month, so I should
have sufficient time to make the necessary changes to get the module up to
spec.
With respect to the build failure - I was kind of surprised to see gt-ysld
as part of the default build. Given that gt-process is also used by
downstream projects, it probably makes the most sense to add that to the
default build as well. (And as a slight tangent, while looking at the
geotools build, I ran into build issues with the gt-wfs-ng module - it
seems to be taking an excessive amount of time (around an hour) to build
and run tests).
Torben
On Fri, Jan 20, 2017 at 9:47 AM, Andrea Aime <andrea.a...@geo-solutions.it>
wrote:
> Hey Torben,
> I just had my location geotools 15.4 build fail on gt-ysld because it
> depends on a process module,
> which is not in the build unless one adds -Dall.
>
> So... you cannot graduate the gt-ysld module in that state, as it would be
> a supported module
> depending on an unsupported one (and one not normally in the build):
>
> https://github.com/geotools/geotools/blob/master/modules/
> unsupported/ysld/pom.xml#L82
>
> As part of the graduation I would consider checking if you can do without
> that dependency (or graduate
> the processing modules along with it, Jody had still something he wanted
> to do before graduating them
> I believe).
> And before the graduation, it probably makes sense to fix the community
> pom so that a build without -Dall works.
> Not sure if we want to consider this a 15.4 blocker... but without -Dall
> it would not build.
>
> Cheers
> Andrea
>
>
> On Fri, Jan 20, 2017 at 12:38 AM, Torben Barsballe <
> tbarsba...@boundlessgeo.com> wrote:
>
>> Hello All,
>>
>> I am working towards moving gt-ysld towards being a supported module, and
>> ran into a bunch of minor outdated procedures in the dev guide.
>>
>> Pull request with fixes here
>> <https://github.com/geotools/geotools/pull/1435>, feedback welcome.
>>
>> Torben
>>
>> ------------------------------------------------------------
>> ------------------
>> Check out the vibrant tech community on one of the world's most
>> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
>> _______________________________________________
>> GeoTools-Devel mailing list
>> GeoTools-Devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>
>>
>
>
> --
> ==
> GeoServer Professional Services from the experts! Visit
> http://goo.gl/it488V for more information.
> ==
>
> Ing. Andrea Aime
> @geowolf
> Technical Lead
>
> GeoSolutions S.A.S.
> Via di Montramito 3/A
> 55054 Massarosa (LU)
> phone: +39 0584 962313 <+39%200584%20962313>
> fax: +39 0584 1660272 <+39%200584%20166%200272>
> mob: +39 339 8844549 <+39%20339%20884%204549>
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*
>
> Le informazioni contenute in questo messaggio di posta elettronica e/o
> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
> loro utilizzo è consentito esclusivamente al destinatario del messaggio,
> per le finalità indicate nel messaggio stesso. Qualora riceviate questo
> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
> darcene notizia via e-mail e di procedere alla distruzione del messaggio
> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
> utilizzarlo per finalità diverse, costituisce comportamento contrario ai
> principi dettati dal D.Lgs. 196/2003.
>
>
>
> The information in this message and/or attachments, is intended solely for
> the attention and use of the named addressee(s) and may be confidential or
> proprietary in nature or covered by the provisions of privacy act
> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
> Code).Any use not in accord with its purpose, any disclosure, reproduction,
> copying, distribution, or either dissemination, either whole or partial, is
> strictly forbidden except previous formal approval of the named
> addressee(s). If you are not the intended recipient, please contact
> immediately the sender by telephone, fax or e-mail and delete the
> information in this message that has been received in error. The sender
> does not give any warranty or accept liability as the content, accuracy or
> completeness of sent messages and accepts no responsibility for changes
> made after they were sent or for other risks which arise as a result of
> e-mail transmission, viruses, etc.
>
> -------------------------------------------------------
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
