This is an automated email from the ASF dual-hosted git repository.
desruisseaux pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/maven-compiler-plugin.git
from 7d44073 Bump org.mockito:mockito-core from 5.16.1 to 5.17.0
add cb82437 Remove the `multirelease-patterns/packaging-plugin` test
because it was testing another plugin than this project. It was testing
`pw.krejci:multi-release-jar-maven-plugin`, which fails with
`NoSuchMethodError` with Maven 4-RC3. That external project may need to be
updated for Maven 4.
add f97b683 Upgrade to Maven 4.0.0-rc-3. Update for changes in the
`Project` interface. The fix leverages the new `SourceRoot` element.
add 9bf13d6 Move the formatting of options in the `Options` class.
add 5eaf0d1 Use relative paths when possible in the `javac.args` file
that is generated. https://github.com/apache/maven-compiler-plugin/pull/193
add 51947a8 Take in account more information provided by the `<source>`
elements of the `pom.xml` file. With this commit, the include/exclude filters
declared in `<source>` are combined with the include/exclude filters declared
in the puglin configuration. The module name and release information are also
stored, but only partially supported yet.
add 22f98b8 Refactor `AbstractCompilerMojo` by moving part of the
`compile` method body in a separated class. The intend is to reduce the
complexity by splitting one big method into smaller methods. It may also help
external applications (e.g. IDE) to use `ToolExecutor` in their environment.
add 5dae077 Initial support of multi-releases project by using the
`<source>` element. Initial support of modular project (JPMS) by using the
`<module>` element. Currently support only one or the other, not yet both in
same time.
add c484c82 Replace the convolved hack (which was moving files around) by
symbolic link. This hack is needed only when we want to simulate the Maven 3
behaviour for compatibility reasons.
add 195a809 Add two integration tests: - Multi-release project defined
with the new `<source>` element. - Modular project defined with the new
`<source>` element.
add 5d4d7f3 Documentation fixes and more straightforward policy for
showing or not the stack trace.
https://github.com/apache/maven-compiler-plugin/issues/323
add 1271c85 Use relative paths in compiler error and warning messages,
for shorter and more readable output. Give a hint when a change of directory is
needed for using the relative paths.
add 3a56f5a Use the native encoding for reading the file written by a
native process (forked compiler). Avoid too verbatim verifications of
locale-sensitive compiler messages.
https://github.com/apache/maven-compiler-plugin/issues/327
add 3f01ff8 More conservative default value of the
`<incrementalCompilation>` parameter if an annotation processor is present.
add 8ff2d3d Documentation fixes from feedbacks.
No new revisions were added by this update.
Summary of changes:
.github/workflows/maven-verify.yml | 2 +-
README.md | 20 +-
pom.xml | 4 +-
src/it/MCOMPILER-192/verify.groovy | 2 +-
src/it/MCOMPILER-346/verify.groovy | 13 +-
src/it/MCOMPILER-366/verify.groovy | 2 +-
src/it/automodules-application/verify.groovy | 2 +-
src/it/automodules-library/verify.groovy | 2 +-
src/it/automodules-manifest/verify.groovy | 2 +-
src/it/automodules-transitive-module/verify.groovy | 2 +-
.../src/main/java/module-info.java | 2 +-
.../src/test/java/bar/BarTests.java | 2 +-
.../src/test/java/module-info.java | 4 +-
.../src/main/java/module-info.java | 2 +-
.../src/test/java/foo/FooTests.java | 2 +-
.../src/test/java/module-info.java | 2 +-
src/it/mcompiler-120/verify.groovy | 7 +-
src/it/mcompiler-179/verify.groovy | 10 +-
src/it/mcompiler-21_methodname-change/pom.xml | 3 +
.../invoker.properties | 2 +-
.../pom.xml | 47 +-
.../src/java/org.bar/main/bar/App.java} | 9 +-
.../src/java/org.bar/main}/module-info.java | 4 +-
.../src/java/org.bar/test/bar}/AppTest.java | 11 +-
.../src/java/org.foo/main/foo/App.java} | 7 +-
.../src/java/org.foo/main}/module-info.java | 5 +-
.../src/java/org.foo/test/foo}/AppTest.java | 11 +-
.../verify.groovy | 15 +-
.../invoker.properties | 0
.../pom.xml | 40 +-
.../src/main/java/foo/MainFile.java} | 9 +-
.../src/main/java/foo/OtherFile.java} | 9 +-
.../src/main/java_16/foo/OtherFile.java} | 15 +-
.../src/main/java_17/foo/YetAnotherFile.java} | 18 +-
src/it/multirelease-on-classpath/verify.groovy | 45 ++
.../packaging-plugin/invoker.properties | 19 -
.../multirelease-patterns/packaging-plugin/pom.xml | 84 --
.../src/main/java-mr/9/module-info.java | 22 -
.../packaging-plugin/src/main/java-mr/9/mr/A.java | 34 -
.../packaging-plugin/src/main/java/base/Base.java | 26 -
.../packaging-plugin/src/main/java/mr/A.java | 32 -
.../packaging-plugin/src/main/java/mr/I.java | 23 -
.../packaging-plugin/src/test/java/mr/ATest.java | 48 --
.../packaging-plugin/verify.groovy | 73 --
.../plugin/compiler/AbstractCompilerMojo.java | 857 +++++++++------------
.../apache/maven/plugin/compiler/CompilerMojo.java | 76 +-
.../maven/plugin/compiler/DiagnosticLogger.java | 41 +-
.../apache/maven/plugin/compiler/ForkedTool.java | 8 +-
.../maven/plugin/compiler/ForkedToolSources.java | 10 +-
.../maven/plugin/compiler/IncrementalBuild.java | 208 +++--
.../org/apache/maven/plugin/compiler/Options.java | 91 +++
.../apache/maven/plugin/compiler/PathFilter.java | 114 +--
.../maven/plugin/compiler/SourceDirectory.java | 247 +++++-
.../apache/maven/plugin/compiler/SourceFile.java | 36 +-
.../maven/plugin/compiler/SourcePathType.java | 116 +++
.../maven/plugin/compiler/SourcesForRelease.java | 80 +-
.../maven/plugin/compiler/TestCompilerMojo.java | 288 ++-----
.../apache/maven/plugin/compiler/ToolExecutor.java | 637 +++++++++++++++
.../maven/plugin/compiler/ToolExecutorForTest.java | 400 ++++++++++
...ption.java => UnsupportedVersionException.java} | 14 +-
.../maven/plugin/compiler/package-info.java} | 33 +-
.../plugin/compiler/CompilerMojoTestCase.java | 8 +-
62 files changed, 2480 insertions(+), 1477 deletions(-)
copy src/it/{MCOMPILER-205 => modular-sources}/invoker.properties (94%)
copy src/it/{MCOMPILER-270_release => modular-sources}/pom.xml (58%)
copy src/it/{MCOMPILER-321_pathexceptions/src/test/java/test/MyTest.java =>
modular-sources/src/java/org.bar/main/bar/App.java} (90%)
copy src/it/{automodules-manifest/src/main/java =>
modular-sources/src/java/org.bar/main}/module-info.java (95%)
copy
src/it/{MCOMPILER-170/mcompiler-170-1/src/test/java/org/apache/maven/plugins/compiler/it
=> modular-sources/src/java/org.bar/test/bar}/AppTest.java (83%)
copy src/it/{MCOMPILER-542/src/main/java/org/maven/test/Main.java =>
modular-sources/src/java/org.foo/main/foo/App.java} (90%)
copy src/it/{jdk9-exportsto/bar/src/main/java =>
modular-sources/src/java/org.foo/main}/module-info.java (95%)
copy
src/it/{MCOMPILER-170/mcompiler-170-1/src/test/java/org/apache/maven/plugins/compiler/it
=> modular-sources/src/java/org.foo/test/foo}/AppTest.java (83%)
copy src/it/{default-fork-windows => modular-sources}/verify.groovy (62%)
copy src/it/{MCOMPILER-205 => multirelease-on-classpath}/invoker.properties
(100%)
copy src/it/{MCOMPILER-360 => multirelease-on-classpath}/pom.xml (65%)
copy
src/it/{MCOMPILER-349_dependencyChanged/dependent-module/src/main/java/Main.java
=> multirelease-on-classpath/src/main/java/foo/MainFile.java} (86%)
copy
src/it/{MCOMPILER-349_dependencyChanged/dependent-module/src/main/java/Main.java
=> multirelease-on-classpath/src/main/java/foo/OtherFile.java} (86%)
copy src/it/{non-english-warnings/src/test/java/MyTest.java =>
multirelease-on-classpath/src/main/java_16/foo/OtherFile.java} (72%)
copy src/it/{mcompiler-182/src/main/java/foo/BeanA.java =>
multirelease-on-classpath/src/main/java_17/foo/YetAnotherFile.java} (70%)
create mode 100644 src/it/multirelease-on-classpath/verify.groovy
delete mode 100644
src/it/multirelease-patterns/packaging-plugin/invoker.properties
delete mode 100644 src/it/multirelease-patterns/packaging-plugin/pom.xml
delete mode 100644
src/it/multirelease-patterns/packaging-plugin/src/main/java-mr/9/module-info.java
delete mode 100644
src/it/multirelease-patterns/packaging-plugin/src/main/java-mr/9/mr/A.java
delete mode 100644
src/it/multirelease-patterns/packaging-plugin/src/main/java/base/Base.java
delete mode 100644
src/it/multirelease-patterns/packaging-plugin/src/main/java/mr/A.java
delete mode 100644
src/it/multirelease-patterns/packaging-plugin/src/main/java/mr/I.java
delete mode 100644
src/it/multirelease-patterns/packaging-plugin/src/test/java/mr/ATest.java
delete mode 100644 src/it/multirelease-patterns/packaging-plugin/verify.groovy
create mode 100644
src/main/java/org/apache/maven/plugin/compiler/SourcePathType.java
create mode 100644
src/main/java/org/apache/maven/plugin/compiler/ToolExecutor.java
create mode 100644
src/main/java/org/apache/maven/plugin/compiler/ToolExecutorForTest.java
copy
src/main/java/org/apache/maven/plugin/compiler/{CompilationFailureException.java
=> UnsupportedVersionException.java} (76%)
copy
src/{it/MCOMPILER-170/mcompiler-170-1/src/main/java/org/apache/maven/plugins/compiler/it/App3.java
=> main/java/org/apache/maven/plugin/compiler/package-info.java} (52%)
