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

jdk(next) OpenJ9 builds fail as built with Hotspot JDK15 BootJDK #1890

Closed
andrew-m-leonard opened this issue Jun 16, 2020 · 3 comments · Fixed by #1899
Closed

jdk(next) OpenJ9 builds fail as built with Hotspot JDK15 BootJDK #1890

andrew-m-leonard opened this issue Jun 16, 2020 · 3 comments · Fixed by #1899
Assignees
Labels
bug Issues that are problems in the code as reported by the community openj9 Issues that are enhancements or bugs raised against the OpenJ9 group
Milestone

Comments

@andrew-m-leonard
Copy link
Contributor

Platform:
All OpenJ9 builds

05:20:11  * For target buildtools_interim_langtools_modules_java.compiler.interim__the.BUILD_java.compiler.interim_batch:
05:20:11  warning: unknown enum constant Feature.SEALED_CLASSES
05:20:11  warning: unknown enum constant Feature.SEALED_CLASSES
05:20:11  error: warnings found and -Werror specified
05:20:11  1 error
05:20:11  2 warnings

SEALED_CLASSES is not defined in openj9 extensions jdk(next) yet as it has not been merged.
Whereas, Hotspot jdk15 builds already have this, hence as this is used as boot JDK for openj9 the compiler uses the constant in the interim code, hence the error...

Q: Should we use a OpenJ9 boot jdk for OpenJ9 builds...?

@karianna karianna added bug Issues that are problems in the code as reported by the community openj9 Issues that are enhancements or bugs raised against the OpenJ9 group labels Jun 16, 2020
@M-Davies
Copy link
Contributor

@andrew-m-leonard What timescale are we looking at until OpenJ9 implement the missing classes? Could we wait until they implement rather than adding in a specific use case?

@M-Davies M-Davies self-assigned this Jun 17, 2020
@andrew-m-leonard
Copy link
Contributor Author

I can't say for sure, but I would estimate within a week

@karianna karianna added this to the June 2020 milestone Jun 17, 2020
@M-Davies
Copy link
Contributor

I've got a PR ready so we can put that in for now and then revert when they implement the classes 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues that are problems in the code as reported by the community openj9 Issues that are enhancements or bugs raised against the OpenJ9 group
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants