-
Notifications
You must be signed in to change notification settings - Fork 71
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
MYFACES-4708: Quarkus 3.8.2 #832
base: 4.0.x
Are you sure you want to change the base?
Conversation
Still failing for me:
Looks like Java 17 was introduced in 3.7: https://quarkus.io/blog/java-17/ Alternatively, I wonder if you can set a higher java level for the quarkus sub-projects? However, that means Faces standalone needs JDK 11, but with Quarkus will require Java 17. |
ahhh damn. |
9b8732c
to
8702817
Compare
I bumped it to 17 but i have no idea if thta will work just for the extension? |
facepalm You're right -- my suggestion was bad.... since JDK 11 doesn't know about 17. Any chance you can use an older version? I verified that 3.6.9 still uses Java 11:
Can we recommend any users who need quarkus 3.7 or higher use Faces 4.1 instead? It's a minor release, so it shouldn't pose any migration issues. |
Or we change the required JDK to Java 17 for 4.0 -- target Java 11 for the API / IMPL but quarkus for 17. I think this would need some community discussion. Alternatively, we just keep 3.15.1, if possible? I don't know too much about these different quarkus libraries, but we don't change the compilation level for MyFaces. |
yep let me back up to 3.6.9 no problem. 4.0 should stay on 11 that is my bad! |
8702817
to
2c067c9
Compare
I reverted back to 3.2.9 which is the last LTS release before 3.8 |
No description provided.