Skip to content

build(deps): upgrade JavaFX from 26-ea+6 to 27-ea+16 in spoon-visualisation#6728

Open
MartinWitt wants to merge 6 commits into
masterfrom
chore/javafx-27-migration
Open

build(deps): upgrade JavaFX from 26-ea+6 to 27-ea+16 in spoon-visualisation#6728
MartinWitt wants to merge 6 commits into
masterfrom
chore/javafx-27-migration

Conversation

@MartinWitt

@MartinWitt MartinWitt commented May 17, 2026

Copy link
Copy Markdown
Collaborator

Copilot AI review requested due to automatic review settings May 17, 2026 13:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the JavaFX dependency used by the spoon-visualisation module, and (additionally) changes the module’s parent POM version.

Changes:

  • Bump JavaFX from 26-ea+6 to 27-ea+16 via the jfx.version property.
  • Update the spoon-pom parent version from 11.1.0 to 11.3.1-beta-8.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread spoon-visualisation/pom.xml Outdated
<groupId>fr.inria.gforge.spoon</groupId>
<artifactId>spoon-pom</artifactId>
<version>11.1.0</version>
<version>11.3.1-beta-8</version>
Comment thread spoon-visualisation/pom.xml
@MartinWitt MartinWitt changed the title build(deps): upgrade JavaFX from 26-ea+6 to 27-ea+16 in spoon-visuali… build(deps): upgrade JavaFX from 26-ea+6 to 27-ea+16 in spoon-visualisation May 17, 2026
…sation

JavaFX 27-ea moved module-info.class to platform-specific JARs only.
Add OS-detection profiles and platform classifier to dependencies so
the compiler finds the JPMS modules on all platforms (win/mac/linux).
…sation

JavaFX 27-ea removed module-info.class from the base JAR. With
maven-compiler-plugin 3.15.0 (from updated parent POM), this causes
'module not found' errors. Fix: remove module-info.java and set
useModulePath=false in the compiler plugin, running the project as a
plain classpath app — which it already is at runtime via the shade JAR.
@monperrus

Copy link
Copy Markdown
Collaborator

@MartinWitt that would be so cool if you get this one green!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants