Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing Dependendcy jakarta.xml.bind:jakarta.xml.bind-api after Spring 2.7 to 3.0 migration #284

Closed
MBoegers opened this issue Sep 1, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@MBoegers
Copy link
Contributor

MBoegers commented Sep 1, 2023

What version of OpenRewrite are you using?

rewrite-maven-plugin:5.4.2

How are you running OpenRewrite?

From terminal via maven on a single module maven project.
I run open Rewrite against Spring Petclinic https://github.com/spring-projects/spring-petclinic

What is the smallest, simplest way to reproduce the problem?

  1. Clone https://github.com/spring-projects/spring-petclinic
  2. Reset to Commit 276880edef4c3d1029865d19d6d28e982b9d4d01 it is the commit with the update to Spring Boot 2.7.3
  3. run ./mvnw -U org.openrewrite.maven:rewrite-maven-plugin:run -Drewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-spring:RELEASE -Drewrite.activeRecipes=org.openrewrite.java.spring.boot3.UpgradeSpringBoot_3_0
  4. run ./mvnw clean compile

What did you expect to see?

Build succeeds and Application is runable.

What did you see instead?

Missing dependency jakarta.xml.bind:jakarta.xml.bind-api

Are you interested in contributing a fix to OpenRewrite?

not yet

@MBoegers MBoegers added the bug Something isn't working label Sep 1, 2023
@timtebeek
Copy link
Contributor

Thanks for the detailed report @MBoegers ! We did a fresh batch of releases yesterday, and I just reran through the steps above with that; the problem is still there unfortunately. After the recipe run, an analysis of the dependencies shows me this:
Image

Seems like there's a transitive runtime dependency; not sure if that's picked up as a reason not to add an explicit dependency. 🤔

@timtebeek timtebeek moved this to Backlog in OpenRewrite Sep 13, 2023
@timtebeek
Copy link
Contributor

I feel this might have been another case as we saw here, where incorrect scope information lead to dependencies (not) added.

I'd lean towards closing this one and reopening a new one if necessary, given that time since.

@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenRewrite Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

2 participants