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

Update to guava 33.0.0.jre #782

Closed
wants to merge 1 commit into from
Closed

Conversation

merks
Copy link

@merks merks commented Jan 10, 2024

No description provided.

@merks
Copy link
Author

merks commented Jan 10, 2024

@cdietrich @jonahgraham

Could someone approve the build?

This change is analogous to the following:

523168f

@@ -6,8 +6,8 @@
<repository location="https://download.eclipse.org/tools/orbit/simrel/orbit-aggregation/2023-09"/>
<unit id="com.google.gson" version="2.10.1"/>
<unit id="com.google.gson.source" version="2.10.1"/>
<unit id="com.google.guava" version="32.1.2.jre"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

i assume you need to split this up into two orbit locations

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Why do you think that the orbit should be split up here? I want to make sure I don't mess things up in 0.21.2 that I am working on.

Copy link
Author

Choose a reason for hiding this comment

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

For sure the new version of guava needs 2024-03 to be added...

Copy link
Contributor

Choose a reason for hiding this comment

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

I think I answered the question. It is about the renaming of the jakarta.websocket bundles?

Copy link
Author

Choose a reason for hiding this comment

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

Yes, I think these

image

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks Ed. I think something is going wrong here as we don't use the older jakarta.websocket, but instead jakarta.websocket-api in the code, so I will look at that.

@jonahgraham
Copy link
Contributor

For 2024-03 M1 I think we should just widen the version and do a patch (0.21.2) release like #759 and we can plan on doing an full LSP4J release (0.22.0) in coordination with other projects (like LSP4E and xtext) otherwise we substitute duplicate guava for duplicate lsp4j.

I will start working on that now, let me know if you have objections.

@merks merks closed this Jan 10, 2024
@cdietrich
Copy link
Contributor

cdietrich commented Jan 10, 2024

plus one from me. then we can release Xtext with the 0.21.2 dependency in case there wont be a release in lsp4j for 2024-03 and lsp4j can then consume the Xtext 2.34 release

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