Skip to content

Commit

Permalink
HHH-14371 Correctly set JVM args in the JVM running tests
Browse files Browse the repository at this point in the history
Turns out getJvmArgs() returns a copy, so modifying that copy won't
change the JVM args.

Signed-off-by: Yoann Rodière <yoann@hibernate.org>
  • Loading branch information
yrodiere committed Jan 7, 2021
1 parent ab44830 commit 6845edd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion gradle/java-module.gradle
Expand Up @@ -229,7 +229,7 @@ if ( gradle.ext.javaToolchainEnabled ) {
}

// Configure JVM Options
jvmArgs.addAll( getProperty( 'toolchain.launcher.jvmargs' ).toString().split( ' ' ) )
jvmArgs( getProperty( 'toolchain.launcher.jvmargs' ).toString().split( ' ' ) )

// Display version of Java tools
doFirst {
Expand Down
2 changes: 1 addition & 1 deletion hibernate-core/hibernate-core.gradle
Expand Up @@ -282,7 +282,7 @@ task testJavassist(type: Test) {
}

// Configure JVM Options
jvmArgs.addAll( getProperty( 'toolchain.launcher.jvmargs' ).toString().split( ' ' ) )
jvmArgs( getProperty( 'toolchain.launcher.jvmargs' ).toString().split( ' ' ) )

// Display version of Java tools
doFirst {
Expand Down

0 comments on commit 6845edd

Please sign in to comment.