Skip to content

Upgrade to Spring 7.0.8#2222

Closed
cshannon wants to merge 1 commit into
apache:mainfrom
cshannon:spring7
Closed

Upgrade to Spring 7.0.8#2222
cshannon wants to merge 1 commit into
apache:mainfrom
cshannon:spring7

Conversation

@cshannon

Copy link
Copy Markdown
Contributor

Spring 6.2.x is now EOL so this upgrades to Spring 7.0.x going forward. Spring 7 requires JakartaEE 11, which was upgraded in #1734

I looked this over and I think the only thing that needs to be done is updating the version and version range (which is used for OSGi). I ran through some of the spring tests and things seem to work without any issue but the full test suite needs to run.

Spring 6.2.x is now EOL so this upgrades to Spring 7.0.x going forward.
Spring 7 requires JakartaEE 11, which was upgraded in apache#1734
@cshannon

Copy link
Copy Markdown
Contributor Author

@mattrpav and @jbonofre - It would be good if you both can look over the OSGi stuff to make sure it's ok with this change (plus all the changes in #1734). I think the version range for spring is enough for OSGi but i don't use it so i'm not the best person to test it out.

@jbonofre

Copy link
Copy Markdown
Member

@cshannon we would need the spring features in Karaf. In the meantime, I will implement a workaround.

By the way, we have a dependabot PR about this update already (I commented on it while ago).

@cshannon

Copy link
Copy Markdown
Contributor Author

@cshannon we would need the spring features in Karaf. In the meantime, I will implement a workaround.

By the way, we have a dependabot PR about this update already (I commented on it while ago).

Yes I am aware of the dependabot, I commented on the same PR. I opened this PR because I'm assuming there is more work to do here than a simple simple version change with the major version update

@jbonofre

Copy link
Copy Markdown
Member

@cshannon no worries, I will add commits in this one.

@cshannon

Copy link
Copy Markdown
Contributor Author

I realized we could just push to the dependabot branch if we need to make more changes, which I wasn't think about. I can close this and we can use that one instead which his #2102

@cshannon cshannon closed this Jul 16, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Apache ActiveMQ v6.3.0 Jul 16, 2026
@mattrpav mattrpav moved this from Done to In review in Apache ActiveMQ v6.3.0 Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants