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

DOC: jshell Tool Reference -- incorrect separator documentation on --add-modules

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P3
    • Resolution: Fixed
    • Affects Version/s: 9, 10
    • Fix Version/s: 10
    • Component/s: docs
    • Labels:

      Description

      Tool Reference reads --

      --add-modules module[,module...]

          Specifies the root modules to resolve in addition to the initial module. For Linux and macOS, use a colon (:) to separate items in the list. For Windows, use a semicolon (;) to separate items.

      ------

      The separator is shown to be comma not colon/semicolon. Those are path separators, not list separators. Using them causes failure --

      % /w/j/client/build/linux-x86_64-normal-server-release/images/jdk/bin/jshell --add-modules java.xml.bind:java.xml.ws.annotation
      Launching JShell execution engine threw: Error occurred during initialization of boot layer
      java.lang.module.FindException: Module java.xml.bind:java.xml.ws.annotation not found

      % /w/j/client/build/linux-x86_64-normal-server-release/images/jdk/bin/jshell --add-modules java.xml.bind,java.xml.ws.annotation
      | Welcome to JShell -- Version 10-internal
      | For an introduction type: /help intro

      jshell> /ex
      | Goodbye

      ---------------------
      The same problem is present in the command documentation for /env, /reload, and /reset
      ---------------------

      Correspondingly, the same two sentences are used (correctly) for --class-path and --module-path -- however they use the word "list" which is not otherwise used, this is a "path" not a "list". So, these should read "...to separate items in the path...".

        Attachments

          Activity

            People

            • Assignee:
              jgordon Joni Gordon
              Reporter:
              rfield Robert Field
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved: