Maven Project Info Reports Plugin
  1. Maven Project Info Reports Plugin
  2. MPIR-80

Add Java requirements to the Dependency Report

    Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.3
    • Component/s: dependencies, summary
    • Labels:
      None
    • Patch Submitted:
      Yes
    • Number of attachments :
      3

      Description

      It would be nice to add a section to the dependency report showing the java version requirements.

      Attaching a patch which discovers the java version and compiler options from the java-compiler-plugin configuration and outputs a "Java Version" section. Has an option to configure whether or not this section is shown (default to false).

      1. maven-projectinfo-dependency-java.patch
        8 kB
        Niall Pemberton
      2. maven-projectinfo-dependency-java-v3.patch
        10 kB
        Niall Pemberton
      3. maven-projectinfo-summary-java.patch
        4 kB
        Niall Pemberton

        Issue Links

          Activity

          Hide
          Dennis Lundberg added a comment -

          I feel that this should be added to the project summary rather than the dependency report. The reason is that a project with no dependencies might choose to remove the dependency report, thereby loosing the JVM info in the process. Also I don't see the JVM as a dependency, but rather a prerequisite which makes it fit poorly into the dependency report.

          Show
          Dennis Lundberg added a comment - I feel that this should be added to the project summary rather than the dependency report. The reason is that a project with no dependencies might choose to remove the dependency report, thereby loosing the JVM info in the process. Also I don't see the JVM as a dependency, but rather a prerequisite which makes it fit poorly into the dependency report.
          Hide
          SebbASF added a comment -

          Why not just add it to both reports?

          The information can be made optional on both.

          But please let's get something done to fix this omission - it's now been over two years since the request was raised.

          Show
          SebbASF added a comment - Why not just add it to both reports? The information can be made optional on both. But please let's get something done to fix this omission - it's now been over two years since the request was raised.
          Hide
          Herve Boutemy added a comment -

          I just added a "JDK Rev" information on summary page: see r981102
          please review and improve (there is room for improvement)

          Show
          Herve Boutemy added a comment - I just added a "JDK Rev" information on summary page: see r981102 please review and improve (there is room for improvement)
          Hide
          SebbASF added a comment -

          I think the output should include both the source and target settings from the POM. Both are important.

          Not sure why the code bothers reading the fork and compilerVersion settings.

          Show
          SebbASF added a comment - I think the output should include both the source and target settings from the POM. Both are important. Not sure why the code bothers reading the fork and compilerVersion settings.
          Hide
          Vincent Siveton added a comment -

          This feature is implemented only for java projects. For other requirements, please create a new issue.

          Show
          Vincent Siveton added a comment - This feature is implemented only for java projects. For other requirements, please create a new issue.

            People

            • Assignee:
              Unassigned
              Reporter:
              Niall Pemberton
            • Votes:
              2 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: