j2cl icon indicating copy to clipboard operation
j2cl copied to clipboard

bump jdt to 4.22

Open treblereel opened this issue 1 year ago • 3 comments

It seems safe (at least, I hope so) to upgrade the JDT to version 4.22. The updated version supports Java 17 syntax.

treblereel avatar May 31 '24 03:05 treblereel

Let me check if it breaks anything internally...

rluble avatar May 31 '24 06:05 rluble

For GWT, we found that 4.27 was the highest we could go without changing how the build works, though some visitor changes were still necessary. Past that point required a minimum of Java 17 to run, and new jdt-core dependencies, plus something changed about how the compiler tooling is loaded. We didn't work that last piece out, but just updated to 4.27 to add Java 17 support while still keeping support to run on Java 11.

niloc132 avatar May 31 '24 12:05 niloc132

For GWT, we found that 4.27 was the highest we could go without changing how the build works, though some visitor changes were still necessary. Past that point required a minimum of Java 17 to run, and new jdt-core dependencies, plus something changed about how the compiler tooling is loaded. We didn't work that last piece out, but just updated to 4.27 to add Java 17 support while still keeping support to run on Java 11.

Well, 4.22 requires no additional changes (at least from what I see), so it could be a good starting point for now. If it works, I'll try to push 4.27 later. Ideally it would be great to jump to 4.32 (with java21), but it drops java11, as far as I remember, it requires java17 min

treblereel avatar May 31 '24 15:05 treblereel