Only use jfr-polyfill for OpenJ9

marcphilipp · marcphilipp · commit e62a01cb3d8c · 2024-04-12T20:55:40.000+02:00
diff --git a/documentation/documentation.gradle.kts b/documentation/documentation.gradle.kts
@@ -64,8 +64,10 @@ dependencies {
 		because("Jimfs is used in src/test/java")
 	}
 
-	testRuntimeOnly(libs.jfrPolyfill) {
-		because("OpenJ9 does not include JFR")
+	if (java.toolchain.implementation.orNull == JvmImplementation.J9) {
+		testRuntimeOnly(libs.jfrPolyfill) {
+			because("OpenJ9 does not include JFR")
+		}
 	}
 
 	standaloneConsoleLauncher(projects.junitPlatformConsoleStandalone)
diff --git a/junit-platform-jfr/junit-platform-jfr.gradle.kts b/junit-platform-jfr/junit-platform-jfr.gradle.kts
@@ -9,7 +9,12 @@ dependencies {
 	api(projects.junitPlatformLauncher)
 
 	compileOnlyApi(libs.apiguardian)
-	compileOnly(libs.jfrPolyfill)
+
+	if (java.toolchain.implementation.orNull == JvmImplementation.J9) {
+		compileOnly(libs.jfrPolyfill) {
+			because("OpenJ9 does not include JFR")
+		}
+	}
 
 	osgiVerification(projects.junitJupiterEngine)
 	osgiVerification(projects.junitPlatformLauncher)

Original file line number	Diff line number	Diff line change
`@@ -64,8 +64,10 @@ dependencies {`
`64`	`64`	`because("Jimfs is used in src/test/java")`
`65`	`65`	`}`
`66`	`66`
`67`		`- testRuntimeOnly(libs.jfrPolyfill) {`
`68`		`- because("OpenJ9 does not include JFR")`
	`67`	`+ if (java.toolchain.implementation.orNull == JvmImplementation.J9) {`
	`68`	`+ testRuntimeOnly(libs.jfrPolyfill) {`
	`69`	`+ because("OpenJ9 does not include JFR")`
	`70`	`+ }`
`69`	`71`	`}`
`70`	`72`
`71`	`73`	`standaloneConsoleLauncher(projects.junitPlatformConsoleStandalone)`