Skip to content

Upgrade Ehcache to 3.10.x when the Jakarta migration adds the jakarta classifier - #1262

Merged
jkschneider merged 1 commit into
boot4/maven-wrapper-3-9from
boot4/ehcache-jakarta-version
Oct 3, 2026
Merged

jkschneider merged 1 commit into
boot4/maven-wrapper-3-9from
boot4/ehcache-jakarta-version

Conversation

@jkschneider

Copy link
Copy Markdown
Member

Adding the jakarta classifier to org.ehcache:ehcache materializes a version that was managed for the unclassified artifact (3.2.3), and that version has no jakarta classifier. The Jakarta EE 9 migration now upgrades org.ehcache:* to 3.10.x, the line that ships it.

Its test importedBomManagedDependencyGetsExplicitJakartaVersion relies on ChangeDependencyClassifier writing the previously managed version, openrewrite/rewrite openrewrite/rewrite#9021; CI here stays red until that is merged and published.

Found by a team of coding agents reviewing a Moderne run of org.openrewrite.java.spring.boot4.UpgradeSpringBoot_4_0 over the 31 JHipster repositories (run 20261003115037-NUeOz). The issue is ehcache-jakarta-managed-version in the run's io.moderne.RecipeIssues data table (7 rows), and the fix was discussed and reviewed on the run's message board. A row from the table, as a generalized example:

Adding the Jakarta classifier loses the version managed for the unclassified Ehcache artifact. (jhipster/jhipster-sample-app-ng1, pom.xml, recipe org.openrewrite.maven.ChangeDependencyClassifier)

Before:

<dependencyManagement><dependencies><dependency><groupId>org.ehcache</groupId><artifactId>ehcache</artifactId><version>3.2.3</version></dependency></dependencies></dependencyManagement><dependencies><dependency><groupId>org.ehcache</groupId><artifactId>ehcache</artifactId></dependency></dependencies>

What the recipe produced:

<dependencyManagement><dependencies><dependency><groupId>org.ehcache</groupId><artifactId>ehcache</artifactId><version>3.2.3</version></dependency></dependencies></dependencyManagement><dependencies><dependency><groupId>org.ehcache</groupId><artifactId>ehcache</artifactId><classifier>jakarta</classifier></dependency></dependencies>

Expected:

<dependencyManagement><dependencies><dependency><groupId>org.ehcache</groupId><artifactId>ehcache</artifactId><version>3.2.3</version></dependency></dependencies></dependencyManagement><dependencies><dependency><groupId>org.ehcache</groupId><artifactId>ehcache</artifactId><version>3.10.8</version><classifier>jakarta</classifier></dependency></dependencies>

Stacked on #1261 (3 of 3); merge that one first.

@jkschneider
jkschneider added this pull request to stack #1263 October 3, 2026 21:29
@jkschneider
jkschneider merged commit 77b41cc into main Oct 3, 2026
1 check passed
@jkschneider
jkschneider deleted the boot4/ehcache-jakarta-version branch October 3, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant