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

Use version ranges to include versions jakarta.inject.jakarta.inject-api #1439

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Oct 13, 2023

@akurtakov
Copy link
Member

I've already tried locally with even hardcoded 1.x and 2.x versoins and it didn't work as only the latest version was picked. I blame

. I've even tried using full 1.x and 2.x versions there but no luck.
If one wants to test it locally, just run mvn clean verify -Declipse.javadoc=/usr/lib/jvm/java/bin/javadoc -Pbuild-individual-bundles in eclipse.platform.releng.aggregator/eclipse.platform.common/bundles/org.eclipse.platform.doc.isv (you may have to adjust the path to javadoc executable to use), after that check that target/doc.bin.log is empty - if not it will contain the errors.

@laeubi
Copy link
Contributor Author

laeubi commented Oct 13, 2023

Alright I'll try to see if I find a solution here...

@akurtakov
Copy link
Member

akurtakov commented Oct 13, 2023

The way javadoc is produced is so error prone that it really costs multiple workdays per release to handhold it. It would be nice to get rid of the ant part and make it pure maven. Can this be covered by the IDE WG releng work ticket, @jonahgraham ?

@akurtakov
Copy link
Member

So the issue seems to come from maven-dependency-plugin first as running mvn clean dependency:copy-dependencies old versions are not copied in target/dependencies. This is due to Maven resolving to single version.

@jonahgraham
Copy link
Contributor

Can this be covered by the IDE WG releng work ticket, @jonahgraham ?

I think so - @laeubi should talk to Paul Buck at EF if in doubt. My hope is that if it fits within the existing budget on the SoW then Paul says it is fine. Please feel free to cc in any email to Paul if I can help.

@akurtakov
Copy link
Member

I'm closing this one in favor of #1449 which hardcodes versions.
@laeubi If you can work on this one to make it pure maven build without all these hacks please take it in another issue.

@akurtakov akurtakov closed this Oct 17, 2023
@laeubi
Copy link
Contributor Author

laeubi commented Oct 31, 2023

@akurtakov @jonahgraham I have now started with an initiative to get rid of more ant-magic here, of course it is only a first step and likely need enhancements and more investigations:

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