Support JetBrainsRuntime distribution

[Syer10]

Description: Add support for JetBrainsRuntime Java distribution https://github.com/JetBrains/JetBrainsRuntime/

Justification: I build a desktop app using Compose Multiplatform, and the best distribution to use when building/packaging it is JetBrains Runtime because it expands the runtime support for GUI apps, expanding Swing and AWT support for every platform and fixing bugs. I use Github Actions to build my app so having it be supported in this action would be very helpful.

Are you willing to submit a PR? Im not familair with TypeScript/JavaScript so I cannot say yes.

[marko-zivic-93]

Hello @Syer10 Thank you for your feature request. We will investigate this and come back to you as soon as possible :)

[panticmilos]

hi @Syer10, after the consultation with the team we have decided we are not currently planning to implement this feature. But I will not close this issue, because we want to see how much attention it is going to get :)

[yeikel]

@panticmilos Would you accept community contributions for this? If so, feel free to assign this to me

[jsoref]

It doesn't seem like they'd accept a community contribution without first showing demand.

But you could make a fork that included it and the evidence of repositories using your fork would be a good way to convince them to integrate it.

[panticmilos]

hi @yeikel, sorry for the delayed response. We are still not going to close this feature request, but still keeping it open for same reasons that I mentioned in the comment above :)

[nicbell]

This sounds like a good idea especially for Android projects as it will match Android Studio.

[SingingBush]

I'd certainly like setup-java to support jbr as a distribution. It's worth doing for people that produce plugins for Jetbrains IDE's.

[jsoref]

@yeikel why not create a branch in your fork that adds the support and then link to it here. We'll be able to see how much your repository is used pretty easily (GitHub tracks dependencies).

[yeikel]

@yeikel why not create a branch in your fork that adds the support and then link to it here. We'll be able to see how much your repository is used pretty easily (GitHub tracks dependencies).

My intention was to contribute to this project in general, and I saw an issue I could try

I have no particular interest or need to support this runtime, and maintaining a fork is not something I am interested at this time

[jsoref]

🤷

There's no obligation to maintain your fork. This repository doesn't have high churn and you aren't obligated to support it.

For anyone curious, adding support for a distribution really doesn't look particularly hard, you basically implement two files installer.js and model.js and then add some references. Here's an example: https://github.com/actions/setup-java/commit/5ffc13f4174014e2d4d4572b3d74c3fa61aeb2c2

Once you've gotten it working in your fork, you can advertise it here.

Once it's advertised, you can look at the equivalent of this to see how much your version is used: https://github.com/actions/setup-java/network/dependents

For example: https://github.com/jord1e/setup-java/network/dependents

[MessiasLima]

Would be nice to have thin in GitHub actions

[rock3r]

Would it be possible to re-consider this, especially now that FooJay discover APIs support the JetBrains Runtime? It's very useful for Compose Multiplatform projects such as Jewel.

[gmitch215]

This is a little late, but I submitted a PR that adds support for this:

- name: Setup JBR 21
  uses: gmitch215/setup-java@6d2c5e1f82f180ae79f799f0ed6e3e5efb4e664d
  with:
    distribution: 'jetbrains'
    java-version: 21
    cache: 'gradle'
  # For GitHub API (use if you believe you'd run into rate limits)
  env:
    GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

[apolostudio]

any news on this?