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

[Debian] Switching to OpenJDK 17: issues #185

Open
pgrt opened this issue May 28, 2022 · 3 comments
Open

[Debian] Switching to OpenJDK 17: issues #185

pgrt opened this issue May 28, 2022 · 3 comments

Comments

@pgrt
Copy link

pgrt commented May 28, 2022

Hello,

In Debian we are getting prepared to adopt OpenJDK 17 as the new standard soon (?).

So I built barclay against it and I met issues as the com.sun.javadoc package will disappear -- it is already deprecated in sooner JDKs.

com_sun_javadoc_removal_in_openjdk17.txt

With the attached patch, one may discard the classes that use it, but perhaps you will want to rework the code in the classes instead of removing them. I know for instance that picard is using part of them -- anyway, at the moment I will also remove the 2 concerned classes in picard for the same reason.

Best wishes,
Pierre

@cmnbroad
Copy link
Collaborator

@pgrt I've actually just started looking into what we need to do to migrate these obsolete doclet(s) and remove the dependencies on deprecated APIs. It may take some time though, and we can't remove the classes in the interim because we have downstream projects that rely on these for both documentation and code generation.

@pgrt
Copy link
Author

pgrt commented May 31, 2022

Hello,

Thanks so much for your answer to my concern. I plainly understand the constraint linked to downstream projects.
My sole intent was to help by investigating the issue from my side, but of course you know far more about barclay than I do!

Please feel free to close the issue is it more convenient for your workflows.

Best wishes,
Pierre

@cmnbroad
Copy link
Collaborator

cmnbroad commented Nov 7, 2022

Update: there is now a Java 17 PR with updated implementations of all of the old doclets, and fixes for all Java 17 issues at #188.

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

No branches or pull requests

2 participants