[jira] [Commented] (MCOMPILER-294) test-compile broken due to -Xmodule removal in jdk-ea167

["Hudson (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MCOMPILER-294?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16011176#comment-16011176 ] 

Hudson commented on MCOMPILER-294:
----------------------------------

FAILURE: Integrated in Jenkins build maven-plugins #8959 (See [https://builds.apache.org/job/maven-plugins/8959/])
[MCOMPILER-294] replace -Xmodule with --patch-module
[MCOMPILER-293] Add advice message in case automodules are used
[MCOMPILER-286] Detect if dependencies should be added to classpath or modulepath (rfscholte: [http://svn.apache.org/viewvc/?view=rev&rev=1795232])
* (edit) maven-compiler-plugin/pom.xml
* (edit) maven-compiler-plugin/src/it/automodules-application/invoker.properties
* (edit) maven-compiler-plugin/src/it/automodules-application/src/main/java/org/maven/test/Main.java
* (edit) maven-compiler-plugin/src/it/automodules-library/invoker.properties
* (edit) maven-compiler-plugin/src/it/automodules-library/src/main/java/org/maven/test/Main.java
* (add) maven-compiler-plugin/src/it/automodules-manifest
* (add) maven-compiler-plugin/src/it/automodules-manifest/invoker.properties
* (add) maven-compiler-plugin/src/it/automodules-manifest/pom.xml
* (add) maven-compiler-plugin/src/it/automodules-manifest/src
* (add) maven-compiler-plugin/src/it/automodules-manifest/src/main
* (add) maven-compiler-plugin/src/it/automodules-manifest/src/main/java
* (add) maven-compiler-plugin/src/it/automodules-manifest/src/main/java/module-info.java
* (add) maven-compiler-plugin/src/it/automodules-manifest/src/main/java/org
* (add) maven-compiler-plugin/src/it/automodules-manifest/src/main/java/org/maven
* (add) maven-compiler-plugin/src/it/automodules-manifest/src/main/java/org/maven/test
* (add) maven-compiler-plugin/src/it/automodules-manifest/src/main/java/org/maven/test/Main.java
* (add) maven-compiler-plugin/src/it/automodules-manifest/verify.groovy
* (add) maven-compiler-plugin/src/it/automodules-transitive-module
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/invoker.properties
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/pom.xml
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src/main
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src/main/java
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src/main/java/module-info.java
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src/main/java/org
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src/main/java/org/maven
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src/main/java/org/maven/test
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/src/main/java/org/maven/test/Main.java
* (add) maven-compiler-plugin/src/it/automodules-transitive-module/verify.groovy
* (add) maven-compiler-plugin/src/it/setup_jar_automodule
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/invoker.properties
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/pom.xml
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/src
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/src/main
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/src/main/java
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/src/main/java/com
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/src/main/java/com/ta3
* (add) maven-compiler-plugin/src/it/setup_jar_automodule/src/main/java/com/ta3/MyClass.java
* (add) maven-compiler-plugin/src/it/setup_jar_classic
* (add) maven-compiler-plugin/src/it/setup_jar_classic/invoker.properties
* (add) maven-compiler-plugin/src/it/setup_jar_classic/pom.xml
* (add) maven-compiler-plugin/src/it/setup_jar_classic/src
* (add) maven-compiler-plugin/src/it/setup_jar_classic/src/main
* (add) maven-compiler-plugin/src/it/setup_jar_classic/src/main/java
* (add) maven-compiler-plugin/src/it/setup_jar_classic/src/main/java/com
* (add) maven-compiler-plugin/src/it/setup_jar_classic/src/main/java/com/ta2
* (add) maven-compiler-plugin/src/it/setup_jar_classic/src/main/java/com/ta2/MyClass.java
* (add) maven-compiler-plugin/src/it/setup_jar_module
* (add) maven-compiler-plugin/src/it/setup_jar_module/invoker.properties
* (add) maven-compiler-plugin/src/it/setup_jar_module/pom.xml
* (add) maven-compiler-plugin/src/it/setup_jar_module/src
* (add) maven-compiler-plugin/src/it/setup_jar_module/src/main
* (add) maven-compiler-plugin/src/it/setup_jar_module/src/main/java
* (add) maven-compiler-plugin/src/it/setup_jar_module/src/main/java/com
* (add) maven-compiler-plugin/src/it/setup_jar_module/src/main/java/com/ta2
* (add) maven-compiler-plugin/src/it/setup_jar_module/src/main/java/com/ta2/MyClass.java
* (add) maven-compiler-plugin/src/it/setup_jar_module/src/main/java/module-info.java
* (add) maven-compiler-plugin/src/it/setup_module-transitive
* (add) maven-compiler-plugin/src/it/setup_module-transitive/invoker.properties
* (add) maven-compiler-plugin/src/it/setup_module-transitive/pom.xml
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src/main
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src/main/java
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src/main/java/com
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src/main/java/com/ta2
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src/main/java/com/ta2/plus
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src/main/java/com/ta2/plus/MyClass.java
* (add) maven-compiler-plugin/src/it/setup_module-transitive/src/main/java/module-info.java
* (edit) maven-compiler-plugin/src/main/java/org/apache/maven/plugin/compiler/CompilerMojo.java
* (edit) maven-compiler-plugin/src/main/java/org/apache/maven/plugin/compiler/TestCompilerMojo.java
* (edit) maven-compiler-plugin/src/main/java/org/apache/maven/plugin/compiler/module/AsmModuleInfoParser.java
* (edit) maven-compiler-plugin/src/main/java/org/apache/maven/plugin/compiler/module/ProjectAnalyzer.java
* (edit) maven-compiler-plugin/src/main/java/org/apache/maven/plugin/compiler/module/ProjectAnalyzerResult.java


> test-compile broken due to -Xmodule removal in jdk-ea167
> --------------------------------------------------------
>
>                 Key: MCOMPILER-294
>                 URL: https://issues.apache.org/jira/browse/MCOMPILER-294
>             Project: Maven Compiler Plugin
>          Issue Type: Bug
>    Affects Versions: 3.6.1
>            Reporter: Peter Parson
>              Labels: jigsaw
>
> [{{- Xmodule}} has been merged into {{--patch-module}}|https://bugs.openjdk.java.net/browse/JDK-8173777], and has obviously been [removed in JDK9 build 167|http://bugs.java.com/bugdatabase/view_bug.do?bug_id=8178012].
> This change breaks {{test-compile}}, as [{{TestCompilerMojo}} uses {{-XModule}} to add reads|http://svn.apache.org/viewvc/maven/plugins/trunk/maven-compiler-plugin/src/main/java/org/apache/maven/plugin/compiler/TestCompilerMojo.java?view=markup#l266].



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)