Hi,

After answering

   << I never tried JDK 17 yet, but that should easy to upgrade when ready.>>

to Michael at https://lists.apache.org/thread/8cpz4dh7zoznp4mx2xxv35hns4j8mvf0

I thought it would be time to go for it. So far, I was wrong.

I just tried to use the last JDK 17 (Adoptium) version without changing the 
Gradle version (still 6.5). The compilation fails with this error message:

Execution failed for task ':compileJava'.
> java.lang.IllegalAccessError: class org.gradle.internal.compiler.java.ClassNameCollector (in unnamed module @0xe5354e) cannot access class com.sun.tools.javac.code.Symbol$TypeSymbol (in module jdk.compiler) because module jdk.compiler
does not export com.sun.tools.javac.code to unnamed module @0xe5354e

So I tried with Gradle 7.5 and 7.6* to no avail. The compilation fails too. It 
seems related to https://github.com/gradle/gradle/issues/20837

* loading Gradle only with

|gradlew wrapper --gradle-version=7.6 --distribution-type=bin or ||gradlew 
wrapper --gradle-version=7.5 --distribution-type=bin Has someone got better 
results? |
Jacques
||

Le 15/07/2022 à 15:09, Nicolas Malin a écrit :
Hi Michael,

I totally agree with that. However, the JDK 17 LTS failed to compile on my 
local.

I propose to move 8 to 11 as first step and homogenize all server, CI and 
documentation next move to the 17 as soon as it's safe.

Nicolas

On 15/07/2022 14:50, Michael Brohl wrote:
Hi Nicolas,

I suggested to think about switching to JDK 17 LTS a while ago: 
https://lists.apache.org/thread/8cpz4dh7zoznp4mx2xxv35hns4j8mvf0

Time went by, so I think it would be better to skip 11 and go with 17 LTS, 
because we will have longer support.

What do you think?

Regards,

Michael Brohl

ecomify GmbH - www.ecomify.de


Am 15.07.22 um 10:14 schrieb Nicolas Malin:
Hello,

I saw that break the CI github because I commited a List.of on release22.01 [1] 
not supported by java 8

After a quick check, the CI work on java 8 and the documentation quote also the java 8 (release22.01 and trunk) but for the demo server we move to java 11 [2]

I propose to update the CI and documentation but to be sure, we move the 
release22.01 on java 11 or the LTS 17 ?

Reading you
Nicolas


[1] https://gitbox.apache.org/repos/asf/ofbiz-framework.git
[2] https://lists.apache.org/thread/fq6q58lv52751t4xlkb17mmwkj4kk8q4


Reply via email to