Hi,
well I don't want to block this, so -0 (uncomfortable, but if you want to
go ahead do).
Cheers
Andrea
On Wed, Feb 18, 2015 at 5:56 PM, Torben Barsballe <
tbarsba...@boundlessgeo.com> wrote:
> I aggree that this big a change (at least it is a non-functional change)
> is probably a bad idea right before a release, but Jody asked if I could
> get it done in time, so here we are.
>
> Andrea:
>
> - According to our documentation
>
> <http://docs.geotools.org/latest/userguide/build/maven/testing.html#online-testing>,
> any tests requiring an online resurce should end in OnlineTest.java, and
> such tests will only run under the "-P online" maven profile; currently
> this is not followed, hence the rename.
> - If we want to have online tests running on a build box (eg. ares),
> then controling them with a flag is necessary, since just having/not having
> the properties files could affect other builds that were not intended to
> run online tests (Particularily in cases of concurrent builds, or builds
> not cleaning up after themselves quickly enough. The primary reason for
> this change is to support doing automated online tests.
>
> Torben
>
> On Wed, Feb 18, 2015 at 1:40 AM, Christian Mueller <
> christian.muel...@os-solutions.at> wrote:
>
>> Hi
>>
>> The drivers for DB2 have to be installed manually too.
>>
>> Cheers
>> Christian
>>
>> On Wed, Feb 18, 2015 at 5:54 AM, Andrea Aime <
>> andrea.a...@geo-solutions.it> wrote:
>>
>>> I'm in the "last minute change hater's club" too. What's the damage in
>>> doing it on trunk only after the cut, and backport later, or just do not
>>> backport at all if those are considered API? (arguably, the base classes
>>> are API as any JDBC store depends on them)
>>>
>>> Cheers
>>> Andrea
>>> Il 18/feb/2015 05:29 "Ben Caradoc-Davies" <b...@transient.nz> ha scritto:
>>>
>>> Torben,
>>>>
>>>> I am not convinced that it is a good idea to rename 291 classes on the
>>>> eve of an RC1 release. We are in a freeze, after all. This proposal
>>>> certainly merits discussion.
>>>>
>>>> Kind regards,
>>>> Ben.
>>>>
>>>> On 18/02/15 14:29, Torben Barsballe wrote:
>>>> > Here is a pull request that updates *all* classes inheriting from
>>>> > OnlineTest to end in OnlineTest.java:
>>>> > https://github.com/geotools/geotools/pull/742
>>>> > Hopefully this is still in time for RC1.
>>>> >
>>>> > Torben
>>>> >
>>>> > On Tue, Feb 17, 2015 at 2:47 PM, Torben Barsballe <
>>>> > tbarsba...@boundlessgeo.com> wrote:
>>>> >
>>>> >> Renaming all database tests to end in OnlineTest is easy, and can
>>>> get done
>>>> >> in time for RC1.
>>>> >>
>>>> >> We should probably also look and see if there are other "Online"
>>>> tests
>>>> >> that should be renamed, but this will likely require a bit of
>>>> digging and
>>>> >> probably won't be ready for RC1.
>>>> >>
>>>> >> Torben
>>>> >>
>>>> >> On Tue, Feb 17, 2015 at 2:24 PM, Jody Garnett <
>>>> jody.garn...@gmail.com>
>>>> >> wrote:
>>>> >>
>>>> >>> Good research, that will be quite the pull request :) Can we get it
>>>> done
>>>> >>> in time for RC1?
>>>> >>>
>>>> >>> --
>>>> >>> Jody Garnett
>>>> >>>
>>>> >>> On 17 February 2015 at 14:07, Torben Barsballe <
>>>> >>> tbarsba...@boundlessgeo.com> wrote:
>>>> >>>
>>>> >>>> I am working on getting online database tests set up and runing on
>>>> >>>> ares/Jenkins.
>>>> >>>> According to our documentation
>>>> >>>> <
>>>> http://docs.geotools.org/latest/userguide/build/maven/testing.html#online-testing
>>>> >,
>>>> >>>> any tests requiring an online resurce should end in
>>>> OnlineTest.java, and
>>>> >>>> will only run under the "-P online" maven profile.
>>>> >>>>
>>>> >>>> It turns out that all of our database tests do not follow this
>>>> naming
>>>> >>>> convention, and instead derive their name from the jdbc test class
>>>> which
>>>> >>>> they extend.
>>>> >>>>
>>>> >>>> This means that the database tests will run as long as you have the
>>>> >>>> appropriate database fixture your ~/.geotools directory, completely
>>>> >>>> independant of the maven "online" profile.
>>>> >>>>
>>>> >>>> In order to properly support enabling/disabling online tests, it
>>>> seems
>>>> >>>> like it will be necessary to append OnlineTest to the name of all
>>>> tests
>>>> >>>> which require online resources...
>>>> >>>>
>>>> >>>> Torben
>>>> >>>>
>>>> >>>>
>>>> >>>>
>>>> ------------------------------------------------------------------------------
>>>> >>>> Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
>>>> >>>> from Actuate! Instantly Supercharge Your Business Reports and
>>>> Dashboards
>>>> >>>> with Interactivity, Sharing, Native Excel Exports, App Integration
>>>> & more
>>>> >>>> Get technology previously reserved for billion-dollar
>>>> corporations, FREE
>>>> >>>>
>>>> >>>>
>>>> http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
>>>> >>>> _______________________________________________
>>>> >>>> GeoTools-Devel mailing list
>>>> >>>> GeoTools-Devel@lists.sourceforge.net
>>>> >>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>> >>>>
>>>> >>>>
>>>> >>>
>>>> >>
>>>> >
>>>> >
>>>> >
>>>> >
>>>> ------------------------------------------------------------------------------
>>>> > Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
>>>> > from Actuate! Instantly Supercharge Your Business Reports and
>>>> Dashboards
>>>> > with Interactivity, Sharing, Native Excel Exports, App Integration &
>>>> more
>>>> > Get technology previously reserved for billion-dollar corporations,
>>>> FREE
>>>> >
>>>> http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
>>>> >
>>>> >
>>>> >
>>>> > _______________________________________________
>>>> > GeoTools-Devel mailing list
>>>> > GeoTools-Devel@lists.sourceforge.net
>>>> > https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>> >
>>>>
>>>> --
>>>> Ben Caradoc-Davies <b...@transient.nz>
>>>> Software Engineer
>>>> Transient Software <http://transient.nz>
>>>> New Zealand
>>>>
>>>>
>>>> ------------------------------------------------------------------------------
>>>> Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
>>>> from Actuate! Instantly Supercharge Your Business Reports and Dashboards
>>>> with Interactivity, Sharing, Native Excel Exports, App Integration &
>>>> more
>>>> Get technology previously reserved for billion-dollar corporations, FREE
>>>>
>>>> http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
>>>> _______________________________________________
>>>> GeoTools-Devel mailing list
>>>> GeoTools-Devel@lists.sourceforge.net
>>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>>
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
>>> from Actuate! Instantly Supercharge Your Business Reports and Dashboards
>>> with Interactivity, Sharing, Native Excel Exports, App Integration & more
>>> Get technology previously reserved for billion-dollar corporations, FREE
>>>
>>> http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
>>> _______________________________________________
>>> GeoTools-Devel mailing list
>>> GeoTools-Devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>
>>>
>>
>>
>> --
>> DI Christian Mueller MSc (GIS), MSc (IT-Security)
>> OSS Open Source Solutions GmbH
>>
>>
>>
>> ------------------------------------------------------------------------------
>> Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
>> from Actuate! Instantly Supercharge Your Business Reports and Dashboards
>> with Interactivity, Sharing, Native Excel Exports, App Integration & more
>> Get technology previously reserved for billion-dollar corporations, FREE
>>
>> http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
>> _______________________________________________
>> 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/NWWaa2 for more information.
==
Ing. Andrea Aime
@geowolf
Technical Lead
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549
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.
-------------------------------------------------------
------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
