Skip to content
This repository was archived by the owner on Oct 11, 2023. It is now read-only.

Import javax.annotation with more restrictive version range#11

Merged
guw merged 1 commit intoeclipse-orbit:masterfrom
eclipseguru:master
May 4, 2023
Merged

Import javax.annotation with more restrictive version range#11
guw merged 1 commit intoeclipse-orbit:masterfrom
eclipseguru:master

Conversation

@guw
Copy link
Copy Markdown
Contributor

@guw guw commented May 3, 2023

Required for proper resolution when jsr305 is present in same target platform.

Fixes eclipse-platform/eclipse.platform.ui#727
eclipse-platform/eclipse.platform.ui#738

Required for proper resolution when jsr305 is present in same target platform.

eclipse-platform/eclipse.platform.ui#738
@guw
Copy link
Copy Markdown
Contributor Author

guw commented May 4, 2023

@jonahgraham Is Orbit officially on GitHub now? Is it ok to merge the PR and publish a new version?

@merks
Copy link
Copy Markdown
Contributor

merks commented May 4, 2023

Yes, it's on GitHub. I'm not so familiar with the details but I assume it will modify what's current generated to specify upper bounds on these 3 requirements:

image

@guw
Copy link
Copy Markdown
Contributor Author

guw commented May 4, 2023

I assume it will modify what's current generated to specify upper bounds on these 3 requirements

Correct. This will prevent importing (and re-exporting) the bogus JSR 305 version of the packages.

@merks
Copy link
Copy Markdown
Contributor

merks commented May 4, 2023

Yes, that seems good!

@guw guw merged commit 30d99d5 into eclipse-orbit:master May 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bundle with Export-Package javax.annotation prevents Eclipse from starting

2 participants