Antoine Levy-Lambert schreef:


http://issues.apache.org/bugzilla/show_bug.cgi?id=39708

This patch renders the classes XalanExecutor, Xalan1Executor, Xalan2Executor in 
the package org.apache.tools.ant.taskdefs.optional.junit useless.

So I have 2 questions :

1) do you agree that I submit the patch containing in the bug report 39708, 
rendering the XalanExecutor, Xalan1Executor, Xalan2Executor classes useless and 
never called by our APIs.

Yes [X]
Could you please consider to not create tasks to create the tempfile and to delete it in that case? Also can we be sure that project.createTask("xslt") always gives us an XSLTProcess?

No []

2) if you answered yes to the first question, do you think that we should

[a] leave the 3 classes XalanExecutor, Xalan1Executor, Xalan2Executor unchanged

[b] mark them as deprecated
At least this one

[c] remove them physically, as they are not needed any more
Maybe mark them now and remove them at ant 1.8? (Still I would like to start a vote to get the Xalan1 stuff out, as it is not available anymore).

Let me start with +1 for the first question.
For the second, my preference would be for the option c which is completely 
clear, but somewhat breaks BC.

Regards,

Antoine


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to