ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jaikiran Pai <>
Subject Re: Need a Second Pair of Eyes
Date Wed, 14 Feb 2018 04:29:27 GMT
Hi Stefan,

You are right. The change introduces a different behaviour than what was 
there before. Just to be extra sure that we indeed are reading it right, 
I added a (dummy) test case in my personal repo, which compares the 
previous logic and the new logic and it does shows that the change 
indeed introduces a different behaviour.



On 13/02/18 11:20 PM, Stefan Bodewig wrote:
> Hi all
> I'm currently reviewing the big change that introduced the regression in
> 1.10.2 as we may have overlooked more than this issue. It is a *big*
> change and so it is taking time.
> While looking at the hunk starting at
> I think the buildup of the path is now backwards.
> The original code would add all elements of dependClasspath to the path
> that are not members of destPath to the new Path p. Unless I am mistaken
> the Difference created in the new code will in addition retain all
> elements of destPath that are not in dependClasspath. So we get all
> elements that are in exactly one of the two - which is more than we
> would have with the old code unless destPath is a subset of
> dependClasspath.
> Does anybody else read it the same way? Or can tell me I'm wrong?
> Unfortunately our test coverage for the depend task is so close to
> non-existent that we can't tell from the tests whether we've broken
> anything.
> Stefan
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message