[
https://issues.apache.org/jira/browse/SUREFIRE-1403?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124913#comment-16124913
]
Tibor Digana commented on SUREFIRE-1403:
----------------------------------------
Robert, so failsafe:verify needs to use javax.xml.
This is not nice on Java 9 that they broke backwards compatibility and
force the world to use the switch to use --add-modules ALL-SYSTEM instead
of providing all modules installed in JRE. For instance, small JRE having
{{java.base}} has advantage on embedded systems and the only should be
propagated. Big scope JRE should propagate all installed modules.
But for me it does not make security sense and common sense to force JRE to
provide modules. It should be opposite and the admin/Jenkins should
configure big scope JRE with selected modules propagated to Java runtime
applications.
If this admin does not do that then all modules should be available by
default which is backwards compatibility for me and we do not have to to
implement these stupid tricks.
On Sun, Aug 13, 2017 at 1:36 PM, Robert Scholte (JIRA) <j...@apache.org>
> [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
>
> Attachments: surefire-1403.zip
>
>
> 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)
