git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Mass changes to various projects under Ant umbrella - should we be doing it?


On 2018-04-12, Gintautas Grigelionis wrote:

> 2018-04-12 15:18 GMT+00:00 Stefan Bodewig <bodewig@xxxxxxxxxx>:

>> On 2018-04-12, Gintautas Grigelionis wrote:

>>> 2018-04-12 14:39 GMT+00:00 Stefan Bodewig <bodewig@xxxxxxxxxx>:

>>>> On 2018-04-11, Gintautas Grigelionis wrote:

>>>>> XMLCatalogTest is an example of unit tests complicated by root
>> property.

>>>> Actually, the test even passes if you remove the line using the property
>>>> as long as your current working directory is Ant's basedir :-)

>>> What is the reason of having and/or keeping the property, anyway?

>> Running the test when your current working directory is not the root of
>> the Ant source tree. I don't know who does so, but it has been something
>> somebody needed some time ago (and took the time to make it
>> possible). Right now I don't see any reason to stop supporting that.

> I'd say it's the other way around: unless I'm mistaken, I have to
> remember to set the property in the IDEs that I use if I want to use a
> test case for debugging. So IMHO it's an unnecessary complication for
> no good reason.

Ant test cases are not designed to be run from an IDE, this has never
been a goal. I'm surprised this system property is the only thing you
need to remember :-)

Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxx
For additional commands, e-mail: dev-help@xxxxxxxxxxxxxx