Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8169417

JavaFX to include jake-compatible versions of module-info.java with import bundles

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P2
    • Resolution: Fixed
    • Affects Version/s: 9
    • Fix Version/s: 9
    • Component/s: javafx
    • Labels:

      Description

      There are several incompatible changes coming in the syntax of module-info.java. The two that affect existing javafx.* modules are:

      1. "requires public" will become "requires transitive"

      2. Multiple service provider definitions for the same service will no longer be allowed. Instead, a single definition will be used with a comma-separated list of providers.

      These changes are already in the JDK jigsaw/jake sandbox. As of now, both the old and new syntax is supported, but the Jigsaw team would like to remove support for the old syntax prior to integrating into jdk9 mainline.

      In order to avoid a complicated, coordinated integration of JDK jake sandbox and FX at the same time, and to be able to more easily test nightly builds, we will provide a new "modules_src_jake" directory in the imported modules in addition to the existing "modules_src" directory. This will go into FX 9-dev and is targeted for build 145.

      The module-info.java files in modules_src_jake will use the new syntax, and the jigsaw/jake Makefile(s) will be changes to use that directory, if present, in preference to modules_src. By doing it this way, the FX side and JDK side of these changes can be done independently, and there will be no need to coordinate the eventual integration of jigsaw/jake into jdk9 mainline.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                kcr Kevin Rushforth
                Reporter:
                kcr Kevin Rushforth
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: