Increase gradle 9 compatibility by peterkir · Pull Request #7297 · bndtools/bnd

peterkir · Jun 27, 2026

updates the bnd Gradle plugins so they work cleanly on Gradle 9. It replaces deprecated/internal Gradle API usage, switches build-scan integration to the Develocity plugin, adjusts tests and compatibility checks, and updates the README to document Gradle 9 support and the configuration-cache limitation.

Copilot

Pull request overview

This PR updates the bnd Gradle build and Gradle plugins to run cleanly on Gradle 9 by migrating from Gradle Enterprise to Develocity, removing deprecated/internal Gradle API usage, and tightening the Gradle compatibility test matrix and documentation.

Changes:

Migrate build scan integration from com.gradle.enterprise to com.gradle.develocity (settings + plugin build).
Replace/avoid internal and removed Gradle APIs (manifest handling, dynamic property access, version comparisons, task action mutation).
Update/readjust Gradle TestKit-based tests and documentation to validate Gradle 9.x behavior and document configuration-cache limitations.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
settings.gradle	Switch root settings build scan integration to Develocity.
build.gradle	Update CI test retry configuration to Develocity-based configuration (needs follow-up for correctness).
gradle-plugins/settings.gradle.kts	Update plugin-publish + migrate plugin build to Develocity.
gradle-plugins/README.md	Document Gradle 9 support, config-cache limitation, and local development workflows.
gradle-plugins/biz.aQute.bnd.gradle/src/test/groovy/aQute/bnd/gradle/TestHelper.groovy	Add compatibility runner/version helpers for Gradle 9 testing.
gradle-plugins/biz.aQute.bnd.gradle/src/test/groovy/aQute/bnd/gradle/TestGradleCompatibility.groovy	Add targeted Gradle version compatibility tests for builder/workspace plugins.
gradle-plugins/biz.aQute.bnd.gradle/src/test/groovy/aQute/bnd/gradle/TestBundlePlugin.groovy	Make outcomes tolerant of `UP_TO_DATE` under Gradle 9/incremental behavior.
gradle-plugins/biz.aQute.bnd.gradle/src/test/groovy/aQute/bnd/gradle/TestBndPlugin.groovy	Make outcomes tolerant of `UP_TO_DATE` and simplify TaskOutcome imports.
gradle-plugins/biz.aQute.bnd.gradle/src/main/java/aQute/bnd/gradle/BundleTaskExtension.java	Remove internal manifest implementation usage; update manifest mutation strategy.
gradle-plugins/biz.aQute.bnd.gradle/src/main/java/aQute/bnd/gradle/BndWorkspacePlugin.java	Remove internal dynamic-object usage; replace with extra-properties/provider/reflection helpers.
gradle-plugins/biz.aQute.bnd.gradle/src/main/java/aQute/bnd/gradle/BndUtils.java	Avoid removed `GradleVersion`; implement version detection/comparison without that API.
gradle-plugins/biz.aQute.bnd.gradle/src/main/java/aQute/bnd/gradle/BndPlugin.java	Declare config-cache incompatibility and avoid unsupported `getActions().clear()` on Gradle 9.
gradle-plugins/biz.aQute.bnd.gradle/src/main/java/aQute/bnd/gradle/BeanProperties.java	Replace use of Gradle internal dynamic APIs with Groovy property access + reflection fallback.
gradle-plugins/biz.aQute.bnd.gradle/build.gradle.kts	Remove internal version parsing (`VersionNumber`) and simplify Groovy-major detection.

 import org.gradle.StartParameter;
+import groovy.lang.Closure;
 import org.gradle.api.Action;


+	public static List<String> compatibilityGradleVersions() {
+		if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_25)) {
+			return ["9.2.0"]
+		}
+		return ["8.14", "9.2.0"]


bjhargrave

I am not clear what all of this is necessary. I can see making a config cache warning in the workspace plugin's apply method to fail fast for users. But the rest does not seem necessary or useful.

Fail fast in the Bnd Workspace plugin when the Gradle configuration cache is requested. The Bnd Workspace model objects are not serializable, so such builds fail late with hard to understand serialization errors. The check uses the public BuildFeatures service (Gradle 8.5+) and is skipped on older Gradle versions. A TestKit test asserts the fail fast message and that builds without the configuration cache keep working. Avoid Gradle internal API which can change without notice: - BundleTaskExtension updates the task manifest directly instead of instantiating the internal DefaultManifest class. - The plugin build parses the Groovy version instead of using the internal VersionNumber class. Update com.gradle.plugin-publish to 2.1.1 for Gradle 9 support. Remove the proprietary Gradle Enterprise build scan and test retry integration so the build only depends on open source plugins. Move the developer notes about using locally built Gradle plugins to DEV_README.md. Signed-off-by: Peter Kirschner <peter@klib.io>

peterkir · Jul 24, 2026

I am not clear what all of this is necessary. I can see making a config cache warning in the workspace plugin's apply method to fail fast for users. But the rest does not seem necessary or useful.

@bjhargrave Thx for the review. Blunt as always, but I asked for ;-) You are right.
I reduced the PR to the minimal set of changes. All unnecessary
API replacements are reverted, and the config cache concern is now handled by a
fail-fast check in the workspace plugin's apply path as you suggested. What
remains is the fail-fast (+ test) and the removal of internal API usages
(DefaultManifest, VersionNumber), and the com.gradle.plugin-publish update.

Created a new clean commit, without the noise.

bjhargrave · Jul 24, 2026

-			org.gradle.api.java.archives.Manifest mergeManifest = new DefaultManifest(null);
-			mergeManifest.attributes(new AttributesMap(builtManifest.getMainAttributes()));
+		private void updateTaskManifest(org.gradle.api.java.archives.Manifest taskManifest, Manifest builtManifest) {
+			taskManifest.getAttributes()


I am afraid of this mutation of the existing manifest. Perhaps you can create a new Manifest by using taskManifest.getEffectiveManifest() which you can then mutate. The purpose of mergeManifest was to create a new Manifest object rather the mutate the existing Manifest.

bjhargrave · Jul 24, 2026

-		id("com.gradle.plugin-publish") version("1.2.0")
+		id("com.gradle.plugin-publish") version("2.1.1")
 		id("dev.hargrave.addmavendescriptor") version("1.1.0")
-		id("com.gradle.enterprise") version("3.13.4")


Why is this removed? It enables build scan.

bjhargrave · Jul 24, 2026

 		}
 		tasks.named("test") {
 			useJUnitPlatform()
-			if (isCI) {


Why is this removed? It attempt to work around flakiness in testing.

bjhargrave · Jul 24, 2026


 import aQute.bnd.osgi.Constants

-pluginManagement {


Why removed?

bjhargrave · Jul 24, 2026

+
+		when: "the configuration cache is requested"
+		// The BuildFeatures service used for the fail fast check requires Gradle 8.5
+		def result = TestHelper.getGradleRunner("8.5")


Asking for version 8.5 seems wrong since 8.5 will always return false. The floor version for this test should be 9.0, no?

bjhargrave · Jul 24, 2026

+		result.output.contains("The Bnd Workspace plugin does not support the Gradle configuration cache")
+
+		when: "the configuration cache is not requested"
+		result = TestHelper.getGradleRunner("8.5")


peterkir force-pushed the gradle-9-compatibility branch from 863400f to 50ae667 Compare June 27, 2026 17:37

peterkir changed the title ~~Gradle 9 compatibility~~ Increase gradle 9 compatibility Jun 27, 2026

peterkir requested review from bjhargrave, Copilot and maho7791 and removed request for Copilot June 27, 2026 18:26

Copilot started reviewing on behalf of peterkir June 27, 2026 18:27 View session

peterkir requested a review from Copilot June 28, 2026 11:22

Copilot started reviewing on behalf of peterkir June 28, 2026 11:23 View session

Copilot AI reviewed Jun 28, 2026

View reviewed changes

bjhargrave requested changes Jun 29, 2026

View reviewed changes

peterkir force-pushed the gradle-9-compatibility branch from 50ae667 to 787c315 Compare July 24, 2026 13:21

peterkir requested a review from bjhargrave July 24, 2026 13:24

bjhargrave requested changes Jul 24, 2026

View reviewed changes

Search code, repositories, users, issues, pull requests...

Uh oh!

Uh oh!

Conversation

peterkir commented Jun 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

bjhargrave left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

peterkir commented Jul 24, 2026

Uh oh!

bjhargrave Jul 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bjhargrave Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

bjhargrave Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

bjhargrave Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

bjhargrave Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

bjhargrave Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

peterkir commented Jun 27, 2026 •

edited

Loading

bjhargrave Jul 24, 2026 •

edited

Loading