Support native google-java-format executables

[vorburger]

In google/google-java-format#868 (comment) (via google/google-java-format#1045, et al), we have made "native" (non java -jar google-java-format*.jar) binaries available; the first release to include that was https://github.com/google/google-java-format/releases/tag/v1.20.0.

E.g. JoseVSeb/google-java-format-for-vs-code#17 adopted this in a VSC extension.

This can make checks a) run a bit faster, and b) avoiding requiring having a java on the PATH, which makes this hook easier to run in certain cases (example).

Would you consider adding support for this in this project?

[macisamuele]

I would be considering the option of not running the tool via java -jar prelude for versions of the tool that suport that.
We do need to make sure that this would work across OSes as well.

I'm open to contributors helping to improve the project.
Feel free to open a PR for it and I'll be prioritising reviewing and releasing it.

[vorburger]

@macisamuele I got reminded today of this issue in MariaDB4j/MariaDB4j#1172.

I would be considering the option of not running the tool via java -jar prelude for versions of the tool that suport that. We do need to make sure that this would work across OSes as well.

How do you think we should best do this? Instead of a (potentially) "complicated heuristic", would you be open to a design (in a PR) which simply uses (adds) some sort of args: [--native=true] sort of flag, which would make it download (and run) e.g. the google-java-format_linux-x86-64 instead of the google-java-format-1.26.0-all-deps.jar? Or would you prefer (need) this to be more "automagic" and "implict" than with such an "explicit" native-instead-of-JAR opt-in flag?

[macisamuele]

I would be open to the idea of adding the ability to run the native binary via a flag.

An alternative that I would also consider is:

• pin GoogleJavaFormatter min version to have the support of the native versions
• run the native tool whether the platform supports it
• run the jar if the platform does not have the native binary