Broken on Java 16 about palantir-java-format HOT 6 CLOSED

To also confirm, this worked on jdk 15:

openjdk version "15.0.4" 2021-07-20
OpenJDK Runtime Environment Zulu15.34+17-CA (build 15.0.4+5-MTS)
OpenJDK 64-Bit Server VM Zulu15.34+17-CA (build 15.0.4+5-MTS, mixed mode, sharing)

from palantir-java-format.

This is due to JEP 396: Strongly Encapsulate JDK Internals by Default
We haven't adopted java 16 internally since the supported lifespan is too short for us to roll off in time, however we're beginning to work on Java 17 support.

from palantir-java-format.

That JEP dangles some tantalising options:

It will still be possible to use the --add-opens command-line option, or the Add-Opens JAR-file manifest attribute, to open specific packages.

https://openjdk.java.net/jeps/261#Packaging:-Modular-JAR-files - could this be as easy as just embedding a module-info.class file that specifies all the crazy --add-opens things this project needs?

EDIT seems like I confused --add-exports with --add-opens - maybe this won't be so easy

from palantir-java-format.

Some relevant context from how google-java-format is tackling this: issue, readme.

Basically their solution for running with Java16+ is that you have to run the formatter with --add-exports flags added. Bit unfortunate and we might have to do some hacky stuff to get this working as Gradle integration or with the intellij plugin. One option might be to do something like bootstrapping our own jre when trying to format some files so we can pass through our flags (as mentioned here).

from palantir-java-format.

The latest version should work with Java 16+. However depending on how you invoke the formatter you need to add --add-exports flags due to JEP 396: Strongly Encapsulate JDK Internals by Default.

When running the formatter through Gradle, you can for example add the args to your gradle.properties file:

org.gradle.jvmargs=--add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED \
  --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED \
  --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED \
  --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED \
  --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED

The intellij plugin and integration with palantir/goethe will automatically add these flags for you.

Closing this issue for now. Please re-open if you run into further issues!

from palantir-java-format.

Note that I couldn't get goethe to add the flags in modern gradle (filed here: palantir/goethe#91)

from palantir-java-format.