Skip to content

8386888: Enhance IntelliJ Idea support for OpenJFX development - #2192

Draft
tsayao wants to merge 4 commits into
openjdk:masterfrom
tsayao:8386888_intellij_idea
Draft

8386888: Enhance IntelliJ Idea support for OpenJFX development#2192
tsayao wants to merge 4 commits into
openjdk:masterfrom
tsayao:8386888_intellij_idea

Conversation

@tsayao

@tsayao tsayao commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator
  • Fixes systemTests dependency issues when importing the project from Gradle.
  • Adds IntelliJ IDEA native code (C/C++) support.

IntelliJ IDEA 2026.1 includes native C/C++ development support. This PR adds the required configuration to improve the development experience for native sources.

Plugins required for native support:

To generate compile_commands.json use ./gradlew generateCompileCommands

To debug native code, run the application and press CTRL + ALT + 5 to attach the native debugger.

image image

Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue

Error

 ⚠️ Pull request body is missing required line: - [x] I confirm that I make this contribution in accordance with the [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).

Issue

  • JDK-8386888: Enhance IntelliJ Idea support for OpenJFX development (Enhancement - P5)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jfx.git pull/2192/head:pull/2192
$ git checkout pull/2192

Update a local copy of the PR:
$ git checkout pull/2192
$ git pull https://git.openjdk.org/jfx.git pull/2192/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 2192

View PR using the GUI difftool:
$ git pr show -t 2192

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jfx/pull/2192.diff

@bridgekeeper

bridgekeeper Bot commented Jun 18, 2026

Copy link
Copy Markdown

👋 Welcome back tsayao! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk

openjdk Bot commented Jun 18, 2026

Copy link
Copy Markdown

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

@Maran23

Maran23 commented Jun 18, 2026

Copy link
Copy Markdown
Member

I will review when it's out of draft.

From a first look, removing misc.xml is the right thing. Not just that it is outdated, IntelliJ will correctly auto-generate it out of the gradle config.

@Maran23

Maran23 commented Jun 19, 2026

Copy link
Copy Markdown
Member

Another file that is always changed locally for me is encodings.xml.

it looks like this for me:

<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="Encoding" native2AsciiForPropertiesFiles="true" defaultCharsetForPropertiesFiles="UTF-8">
    <file url="PROJECT" charset="UTF-8" />
  </component>
</project>

I think we probably can also remvove it from Version Control as it will always regenerate with UTF-8 settings.

@tsayao

tsayao commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator Author

Native Objective-C/C++ support is not working on macOS. The project generates compile_commands.json correctly, but IntelliJ IDEA does not detect or use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants