-
Notifications
You must be signed in to change notification settings - Fork 565
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
Helidon 3.x build fails with Java 21 #9346
base: helidon-3.x
Are you sure you want to change the base?
Conversation
Signed-off-by: Jorge Bescos Gascon <[email protected]>
Signed-off-by: Jorge Bescos Gascon <[email protected]>
Signed-off-by: Jorge Bescos Gascon <[email protected]>
Signed-off-by: Jorge Bescos Gascon <[email protected]>
@tjquinno I saw in the history that you have experience in open-api. Do you happen to know why this TCK is failing?:
Is it maybe related to the |
I added a commit that backports a scanning enhancement from 4.x that fixes the TCK failure. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should be clear that we have no requirement to build the helidon-3.x branch using Java 21. We are only doing this for the convenience of people working on Helidon. Therefore we need to be very careful that this change does not introduce any incompatibilities. I'm concerned about the upgrading of jandex as we've had issues in this area before.
Please make sure you do some backwards compatibility testing. For example please try the example https://github.com/helidon-io/helidon-examples/tree/3.2.10/examples/quickstarts/helidon-standalone-quickstart-mp (with no modification!) against your changes and run the built project with java -jar
. I want to make sure that the indexes generated by that example work at runtime with the changes you are making here.
You might want to extend that example to use openapi to exercise that code as well.
Signed-off-by: Jorge Bescos Gascon <[email protected]>
I executed it with this manifest (note it refers to version 3.2.11-SNAPSHOT, which is the version that contains my changes):
This is the output:
And these are the commands I executed:
|
Description
#9024
The index is correctly generated now
Documentation
NA