Permalink
Comparing changes
Open a pull request
- 2 commits
- 8 files changed
- 0 commit comments
- 2 contributors
Commits on Apr 12, 2019
This update brings in the following fixes: * [MODULES-387] Add JMX operation to get a class location * [MODULES-386] Fixing Javadoc build issues on JDK11 * [MODULES-375] Check if the artifact was successfully resolved before attempting to create the loader. * [MODULES-372] provide our own "java.se" module on JDK 9+
Commits on Apr 14, 2019
[WFCORE-4415] Upgrading JBoss Modules from 1.9.0.Final to 1.9.1.Final
Unified
Split
Showing
with
2 additions
and 9 deletions.
- +0 −1 core-feature-pack/src/main/resources/content/bin/common.bat
- +0 −1 core-feature-pack/src/main/resources/content/bin/common.ps1
- +0 −1 core-feature-pack/src/main/resources/content/bin/common.sh
- +0 −1 host-controller/src/main/java/org/jboss/as/host/controller/jvm/JvmType.java
- +1 −1 host-controller/src/test/java/org/jboss/as/host/controller/ManagedServerBootCmdFactoryTestCase.java
- +0 −1 launcher/src/main/java/org/wildfly/core/launcher/AbstractCommandBuilder.java
- +0 −2 launcher/src/test/java/org/wildfly/core/launcher/CommandBuilderTest.java
- +1 −1 pom.xml
| @@ -15,7 +15,6 @@ goto :eof | ||
| set "DEFAULT_MODULAR_JVM_OPTIONS=!DEFAULT_MODULAR_JVM_OPTIONS! --add-exports=java.base/sun.nio.ch=ALL-UNNAMED" | ||
| set "DEFAULT_MODULAR_JVM_OPTIONS=!DEFAULT_MODULAR_JVM_OPTIONS! --add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED" | ||
| set "DEFAULT_MODULAR_JVM_OPTIONS=!DEFAULT_MODULAR_JVM_OPTIONS! --add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED" | ||
| set "DEFAULT_MODULAR_JVM_OPTIONS=!DEFAULT_MODULAR_JVM_OPTIONS! --add-modules=java.se" | ||
| ) else ( | ||
| set "DEFAULT_MODULAR_JVM_OPTIONS=" | ||
| ) | ||
| @@ -132,7 +132,6 @@ Param( | ||
| $DEFAULT_MODULAR_JVM_OPTIONS += "--add-exports=java.base/sun.nio.ch=ALL-UNNAMED" | ||
| $DEFAULT_MODULAR_JVM_OPTIONS += "--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED" | ||
| $DEFAULT_MODULAR_JVM_OPTIONS += "--add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED" | ||
| $DEFAULT_MODULAR_JVM_OPTIONS += "--add-modules=java.se" | ||
| } | ||
| return $DEFAULT_MODULAR_JVM_OPTIONS | ||
| } | ||
| @@ -13,7 +13,6 @@ setDefaultModularJvmOptions() { | ||
| DEFAULT_MODULAR_JVM_OPTIONS="$DEFAULT_MODULAR_JVM_OPTIONS --add-exports=java.base/sun.nio.ch=ALL-UNNAMED" | ||
| DEFAULT_MODULAR_JVM_OPTIONS="$DEFAULT_MODULAR_JVM_OPTIONS --add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED" | ||
| DEFAULT_MODULAR_JVM_OPTIONS="$DEFAULT_MODULAR_JVM_OPTIONS --add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED" | ||
| DEFAULT_MODULAR_JVM_OPTIONS="$DEFAULT_MODULAR_JVM_OPTIONS --add-modules=java.se" | ||
| else | ||
| DEFAULT_MODULAR_JVM_OPTIONS="" | ||
| fi | ||
| @@ -59,7 +59,6 @@ | ||
| modularJavaOpts.add("--add-exports=java.base/sun.nio.ch=ALL-UNNAMED"); | ||
| modularJavaOpts.add("--add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED"); | ||
| modularJavaOpts.add("--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED"); | ||
| modularJavaOpts.add("--add-modules=java.se"); | ||
| DEFAULT_MODULAR_JVM_ARGUMENTS = Collections.unmodifiableList(modularJavaOpts); | ||
| } | ||
|
|
||
| @@ -124,7 +124,7 @@ public void testGetServerLaunchCommand() throws UnknownHostException { | ||
| List<String> result = instance.getServerLaunchCommand(); | ||
| Assert.assertThat(result.size(), is(notNullValue())); | ||
| if (result.size() > 16) { | ||
| Assert.assertThat(result.size(), is(20)); | ||
| Assert.assertThat(result.size(), is(19)); | ||
| } else { | ||
| Assert.assertThat(result.size(), is(16)); | ||
| } | ||
| @@ -60,7 +60,6 @@ | ||
| modularJavaOpts.add("--add-exports=java.base/sun.nio.ch=ALL-UNNAMED"); | ||
| modularJavaOpts.add("--add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED"); | ||
| modularJavaOpts.add("--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED"); | ||
| modularJavaOpts.add("--add-modules=java.se"); | ||
| DEFAULT_MODULAR_VM_ARGUMENTS = Collections.unmodifiableList(modularJavaOpts); | ||
| } | ||
|
|
||
| @@ -203,15 +203,13 @@ private void testModularJvmArguments(final Collection<String> command, final int | ||
| assertArgumentExists(command, "--add-exports=java.base/sun.nio.ch=ALL-UNNAMED", expectedCount); | ||
| assertArgumentExists(command, "--add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED", expectedCount); | ||
| assertArgumentExists(command, "--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED", expectedCount); | ||
| assertArgumentExists(command, "--add-modules=java.se", expectedCount); | ||
| } else { | ||
| Assert.assertFalse("Did not expect \"--add-exports=java.base/sun.nio.ch=ALL-UNNAMED\" to be in the command list", | ||
| command.contains("--add-exports=java.base/sun.nio.ch=ALL-UNNAMED")); | ||
| Assert.assertFalse("Did not expect \"--add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED\" to be in the command list", | ||
| command.contains("--add-exports=jdk.unsupported/sun.reflect=ALL-UNNAMED")); | ||
| Assert.assertFalse("Did not expect \"--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED\" to be in the command list", | ||
| command.contains("--add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED")); | ||
| Assert.assertFalse("Did not expect \"--add-modules=java.se\" to be in the command list", command.contains("--add-modules=java.se")); | ||
| } | ||
| } | ||
|
|
||
| @@ -185,7 +185,7 @@ | ||
| <version.org.jboss.logmanager.jboss-logmanager>2.1.8.Final</version.org.jboss.logmanager.jboss-logmanager> | ||
| <version.org.jboss.logmanager.log4j-jboss-logmanager>1.1.6.Final</version.org.jboss.logmanager.log4j-jboss-logmanager> | ||
| <version.org.jboss.marshalling.jboss-marshalling>2.0.6.Final</version.org.jboss.marshalling.jboss-marshalling> | ||
| <version.org.jboss.modules.jboss-modules>1.9.0.Final</version.org.jboss.modules.jboss-modules> | ||
| <version.org.jboss.modules.jboss-modules>1.9.1.Final</version.org.jboss.modules.jboss-modules> | ||
| <version.org.jboss.msc.jboss-msc>1.4.5.Final</version.org.jboss.msc.jboss-msc> | ||
| <version.org.jboss.remoting>5.0.8.Final</version.org.jboss.remoting> | ||
| <version.org.jboss.remotingjmx.remoting-jmx>3.0.1.Final</version.org.jboss.remotingjmx.remoting-jmx> | ||