RFR: 8338675: javac shouldn't silently change .jar files on the classpath [v4]

[Joe Darcy]

On Thu, 6 Mar 2025 15:43:21 GMT, David Beaumont <duke at openjdk.org> wrote:

>> Modifying `JavacFileManager` to skip creating sibling output class files for source files found in JARs.
>> 
>> This should match older (JDK 8) behavior whereby the JAR was not writable, and results in any newly generated class files being written to the current working directory (the output of class files into current directory isn't good, but it should match the old behavior).
>
> David Beaumont has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Adding a test for the annotation processing Filer.

test/langtools/tools/javac/modules/AnnotationFilerTest.java line 122:

> 120: 
> 121:     @SupportedAnnotationTypes("*")
> 122:     static class TestAnnotationProcessor extends AbstractProcessor {

Please see the test library file

langtools/toosl/javac/lib/JavacTestingAbstractProcessor.java

and consider extending that class instead.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23699#discussion_r1983754190