[Issue Comment Deleted] (SUREFIRE-1403) [Jigsaw] [Java 9] add "--add-modules ALL-SYSTEM" to forked CLI argument

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Issue Comment Deleted] (SUREFIRE-1403) [Jigsaw] [Java 9] add "--add-modules ALL-SYSTEM" to forked CLI argument

JIRA jira@apache.org

     [ https://issues.apache.org/jira/browse/SUREFIRE-1403?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tibor Digana updated SUREFIRE-1403:
-----------------------------------
    Comment: was deleted

(was: SUCCESS: Integrated in Jenkins build maven-surefire-dev #39 (See [https://builds.apache.org/job/maven-surefire-dev/39/])
[SUREFIRE-1403] [Jigsaw] [Java 9] add "--add-modules ALL-SYSTEM" to (tibor17: [http://git-wip-us.apache.org/repos/asf/?p=maven-surefire.git&a=commit&h=fcfe5daef5704360d5b9bd009647758e63a287dd])
* (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/ForkConfiguration.java
* (add) surefire-booter/src/test/resources/jdk8-IBM/release
* (add) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/Java9FullApiIT.java
* (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
* (edit) surefire-booter/src/test/java/org/apache/maven/surefire/booter/SystemUtilsTest.java
* (add) surefire-integration-tests/src/test/resources/java9-full-api/src/test/java/J9Test.java
* (add) surefire-booter/src/test/resources/jdk/bin/java
* (edit) maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/booterclient/BooterDeserializerStartupConfigurationTest.java
* (edit) surefire-booter/pom.xml
* (add) surefire-booter/src/test/resources/jdk8-oracle/release
* (add) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/AbstractJigsawIT.java
* (edit) maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/booterclient/BooterDeserializerProviderConfigurationTest.java
* (edit) pom.xml
* (add) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/JdkAttributes.java
* (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java
* (edit) maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/booterclient/ForkConfigurationTest.java
* (edit) surefire-api/pom.xml
* (edit) surefire-booter/src/main/java/org/apache/maven/surefire/booter/SystemUtils.java
* (edit) surefire-integration-tests/pom.xml
* (add) surefire-booter/src/test/resources/jdk/jre/bin/java
* (add) surefire-integration-tests/src/test/resources/java9-full-api/pom.xml
* (add) surefire-booter/src/test/resources/jdk9-oracle/release
* (edit) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1265Java9IT.java
* (edit) maven-surefire-common/pom.xml
)

> [Jigsaw] [Java 9] add "--add-modules ALL-SYSTEM" to forked CLI argument
> -----------------------------------------------------------------------
>
>                 Key: SUREFIRE-1403
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-1403
>             Project: Maven Surefire
>          Issue Type: Bug
>          Components: Maven Failsafe Plugin, Maven Surefire Plugin
>            Reporter: Tibor Digana
>            Assignee: Tibor Digana
>             Fix For: 2.20.1
>
>
> Calling *findClass( cls, "java.se.ee")* in *IsolatedClassLoader* does not help and does not do anything because the module is ignored in Java 9.
> In-plugin provider does not have any problem to load classes from entire JDK.
> Forked JVM would work only after added
> {{<argLine>--add-modules ALL-SYSTEM</argLine>}}
> The fix would be to add "--add-modules ALL-SYSTEM" if {{--add-modules}} is not specified by user at Java 9+.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
Loading...