Hi Eugen,
As you know I'm still on Win7 and was able to manage all related issues so far
(like the need to use npm 13.14.0 locally in build.gradle)
To simplify things I just committed the non functional Java changes unrelated
to Gradle and JDK upgrades, not the same than your for CsrfUtil class.
I tried several mixed things w/o success so far. I'll continue and inform you
here later.
Thanks
Jacques
Le 29/11/2022 à 12:12, Ioan Eugen Stan a écrit :
An update to this:
I've rebased the PR for gradle upgrade
https://issues.apache.org/jira/browse/OFBIZ-12400
https://github.com/apache/ofbiz-framework/pull/354
I've bumped gradle to 7.6 .
I started ofbiz with temurin jdk17 and it works (with warnings and some
errors).
See screenshot in PR.
The CI build passes.
Can you please review @Jacques ?
I do hope this will help move things forward.
Eugen
On 2022/11/29 09:55:02 eugen.s...@netdava.com wrote:
Hi,
There are some open issues about this:
https://issues.apache.org/jira/browse/OFBIZ-10757 - jdk 11
https://issues.apache.org/jira/browse/OFBIZ-12399 - jdk 17
https://issues.apache.org/jira/browse/OFBIZ-12400 - gradle 7.x for jdk17
There is also a PR for gradle upgrade
https://github.com/apache/ofbiz-framework/pull/354
Gradle upgrade is a pre-requisite for JDK upgrade.
I remember some issues with project not generating some docs as part of build
on some system.
I think this should not be considered an upgrade blocker IMO.
Since we are using third party libraries that might not support all platforms.
I really hope this gets merged.
Regads,
Eugen
