On Fri, 21 Aug 2026 12:03:09 GMT, Kevin Rushforth <[email protected]> wrote:
>> modules/jfx.incubator.richtext/src/main/java/module-info.java line 40: >> >>> 38: requires transitive javafx.controls; >>> 39: requires transitive jfx.incubator.input; >>> 40: requires java.desktop; >> >> Is this now required in modules/javafx.swing/src/main/java/module-info.java >> >> >> exports com.sun.javafx.embed.swing to >> javafx.graphics; > > No, quite the opposite: The `module-info.java` in `javafx.graphics` needs to > do a qualified export of the utility package to `javafx.swing`. Since it > already does, no other change was needed. I get that` javafx.graphics` need to export `com.sun.jfx.incubator` package to `javafx.swing` but after recent changes, nothing from` javafx.swing `is being used in `javafx.graphics` so why javafx.swing needs to export itself to javafx.graphics... I think we should remove this line from javafx.swing module-info ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2267#discussion_r3840795200
