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

Better context for some jlink exceptions

    Details

    • Subcomponent:
    • Resolved In Build:
      b145
    • CPU:
      generic
    • OS:
      generic
    • Verification:
      Not verified

      Description

      As part of the supportability review for jlink JEP, I saw a few areas which we might be able to capture better context in the exceptions thrown.

      jdk/tools/jlink/internal/
      ImagePluginConfiguration.java:

              if (!Files.isDirectory(dirPath)) {
                  throw new IllegalArgumentException("Not a directory");
              }

      == print dirPath ?

      TaskHelper.java:

                      if (!Files.exists(path) || !Files.isDirectory(path)) {
                          throw newBadArgs("err.existing.image.must.exist");
                      }
      == print path ?

      jdk/tools/jlink/plugin
      ExecutableImage.java:
              if (!Files.exists(home)) {
                  throw new IllegalArgumentException("Invalid image home");
              }
      == print home ?

        Attachments

          Activity

            People

            • Assignee:
              jlaskey Jim Laskey
              Reporter:
              coffeys Sean Coffey
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Due:
                Created:
                Updated:
                Resolved: