infra: update jdk for eclipse static analysis#13201
Conversation
|
Link to semaphore execution: https://checkstyle.semaphoreci.com/workflows/4815f09c-12df-48d6-b0f3-40ebcf838836?pipeline_id=df083745-bd5d-42f0-87c6-ad29e6df5241
|
|
https://checkstyle.semaphoreci.com/jobs/03067066-250a-4b0a-9518-ee586b8b65d5 |
|
@nrmancuso Same error at sevntu-checkstyle/sevntu.checkstyle#1019 . |
|
Hmm, I wasn't expecting a merge commit to be added. |
|
I will fix on my local, you have to manually select rebase in GitHub |
b949be8 to
4f9f606
Compare
|
Do we know why it becomes a problem unexpectedly? |
|
@romani The jar used for this job was compiled with jdk17, so I assume that eclipse migrated from 11 to 17 |
|
Better for us to stay on jdk 11 version and migrate to jdk 17 after summer . Can we set hardcoded version at checkstyle/.ci/eclipse-compiler-javac.sh Line 15 in 3f77dd1 Eclipse is on jdk17 https://wiki.eclipse.org/Eclipse/Installation We should stay on jdk 11. http://ftp-stud.fht-esslingen.de/pub/Mirrors/eclipse/eclipse/downloads/drops4/ We should use We should comment out line I pointed to and make new line with hardcoded value. |
|
@romani done |
Noticed at https://checkstyle.semaphoreci.com/jobs/a2f1d5ba-6547-45a8-a9b1-fc00c7f7c49c, we need to update jdk for this job to 17 (major ver. 61)