Copilot commented on code in PR #128:
URL: https://github.com/apache/xerces-j/pull/128#discussion_r4494424421
##########
build.xml:
##########
@@ -808,6 +810,7 @@ Authors:
<path id="test.classpath">
<path refid="run.classpath"/>
<pathelement location="${build.dir}/tests"/>
+ <pathelement location="${build.tests.dir}"/>
</path>
Review Comment:
On a checkout that previously ran `tests`, `${build.dir}/classes` still
contains the old test bytecode. Because `run.classpath` puts that directory
ahead of `${build.tests.dir}`, `ant test` loads those stale classes instead of
the newly compiled tests, silently ignoring subsequent edits to existing tests.
Please put the new test output directory before `run.classpath`, or remove
legacy test outputs during migration.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]