]> source.dussan.org Git - aspectj.git/commit
Fix: WeaveSpec no longer ignores the '-Xlintfile' parameter 34/head
authorAlexander Kriegisch <Alexander@Kriegisch.name>
Fri, 12 Mar 2021 03:16:36 +0000 (10:16 +0700)
committerAlexander Kriegisch <Alexander@Kriegisch.name>
Fri, 12 Mar 2021 05:54:16 +0000 (12:54 +0700)
commita78abecee0a116336cc9ae604c2a7fbe9d11d621
treecb417e1163926772d20cae78284db06a653b8921
parent42b1e6f09e00194ba2ec32e0972b91e19b9161f9
Fix: WeaveSpec no longer ignores the '-Xlintfile' parameter

Some tests in ajc150.xml and ajc190_from150.xml contain '<weave ...>'
build steps with 'xlintfile="..."' parameters. Those parameters were
passed through to Java and aspect code building steps, but not to the
final weaving step, sometimes leading to spurious "type not exposed to
weaver" warnings which occurred for some local and CI builds, but not
always. Very strange indeed. Anyway, by making method
WeaveSpec.buildWeaveArgs() pass on this parameter, the tests seem to run
reliably now.

TODO: Why does Ajc report that warning if the application JAR is on the
inpath and the aspect JAR is on the aspectpath? Is it because a marker
annotation is defined within the aspect JAR? But actually, that should
not matter, especially not work once and fail at other times. I guess
there is a class loading order problem or similar involved.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
testing/src/test/java/org/aspectj/testing/WeaveSpec.java