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

jdeprscan will need updates to deal with the removal of the Java EE modules

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: 11
    • Fix Version/s: 11
    • Component/s: tools
    • Resolved In Build:
      b19
    • Verification:
      Verified

      Backports

        Description

        jdeprscan will need to be updated to to be able to run successfully after the Java EE and CORBA modules are deleted.

        The processSelf method adds the java.se.ee and jdk.xml.bind modules:
        ——————
            boolean processSelf(Collection<String> classes) throws IOException {
                options.add("--add-modules");
                options.add("java.se.ee,jdk.xml.bind"); // TODO why jdk.xml.bind?

                if (classes.isEmpty()) {
                    Path modules = FileSystems.getFileSystem(URI.create("jrt:/"))
                                              .getPath("/modules");

                    // names are /modules/<modulename>/pkg/.../Classname.class
                    try (Stream<Path> paths = Files.walk(modules)) {
                        Stream<String> files =
                            paths.filter(p -> p.getNameCount() > 2)
                                 .map(p -> p.subpath(1, p.getNameCount()))
                                 .map(Path::toString);
                        return doModularFileNames(files);
                    }
                } else {
                    return doClassNames(classes);
                }
            }
        -------------------

        The following tests currently fail due to the removal of the Java EE and CORBA modules:

        make run-test TEST="open/test/langtools/tools/jdeprscan/tests/jdk/jdeprscan/TestNotFound.java”
        make run-test TEST="open/test/langtools/tools/jdeprscan/tests/jdk/jdeprscan/TestRelease.java"

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  smarks Stuart Marks
                  Reporter:
                  lancea Lance Andersen
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: