Skip to content

Upgrade maven-ejb-plugin alongside jakarta.ejb-api#1155

Open
evie-lau wants to merge 5 commits into
openrewrite:mainfrom
evie-lau:maven-ejb-plugin
Open

Upgrade maven-ejb-plugin alongside jakarta.ejb-api#1155
evie-lau wants to merge 5 commits into
openrewrite:mainfrom
evie-lau:maven-ejb-plugin

Conversation

@evie-lau

@evie-lau evie-lau commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Also handles decoupling old javax.ejb-api version from maven-ejb-plugin's <ejbVersion> configuration

@evie-lau evie-lau self-assigned this Jun 26, 2026
@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Jun 26, 2026
- org.openrewrite.maven.UpgradePluginVersion:
groupId: org.apache.maven.plugins
artifactId: maven-ejb-plugin
newVersion: 3.2.x

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think 3.3.0 should work as well, but 3.2.1 is the minimum required.

Thoughts?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine to use this; should resolve to whatever version is available in 3.2 range; if you're confident 3.x would work for the foreseeable future we could use that as well to cover both, although it could lead to some unnecessary changes.

Comment thread src/main/resources/META-INF/rewrite/jakarta-ee-9.yml Outdated
@evie-lau

Copy link
Copy Markdown
Contributor Author

Added a case where some versions need decoupling. This may have been more common in older apps using javax.ejb-api. There's a version 3.2 that could be used as a shared variable for both javax.ejb-api version and maven-ejb-plugin's <ejbVersion> configuration.

After the jakarta version releases, the version syntax no longer matches.
ie.

  • 4.0.1 in jakarta.ejb-api
  • 4.0 in <ejbVersion> in maven-ejb-plugin

@evie-lau evie-lau moved this from In Progress to Ready to Review in OpenRewrite Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready to Review

Development

Successfully merging this pull request may close these issues.

Upgrade maven-ejb-plugin when upgrading jakarta.ejb-api in EE9

2 participants