https://bz.apache.org/bugzilla/show_bug.cgi?id=69586

--- Comment #1 from Jaikiran Pai <jaiki...@apache.org> ---
Hello SeB, I haven't had a chance to run this locally, but what you note
appears to be a genuine issue. As for the patch, the patch looks reasonable.
However, I see that at one place in the patch, you use:

if (!classpath.toString().isEmpty())

I think it should just be:

if (!classpath.isEmpty())

i.e. instead of checking the string representation of the classpath, we should
just use the org.apache.tools.ant.types.Path.isEmpty() call to match some other
places where such checks are done. Would you be able to see if your test
project works fine if you update the patch to this?

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to