Uploaded image for project: 'Maven'
  1. Maven
  2. MNG-6189

WARN if maven-site-plugin configuration contains reportPlugins element

    Details

      Description

      with MNG-4162, a bunch of reporting logic was removed from Maven core.
      This was globally successful, but not on one topic: it was expected to remove pom's reporting section and replace it with a normal reportPlugins parameter in maven-site-plugin configuration.
      We later discovered that with this normal parameter, we missed one crucial feature: report plugins inheritance, which would require a new mechanism MSITE-484
      Then we reverted our instructions in maven-site-plugin documentation MSITE-647: the new reportPlugins parameter is now just an implementation detail, but should not be used by end-users, reporting section remains the way to configure report plugins MSITE-684

      Now adding a warning in Maven core when it detects a reportPlugins parameter in a maven-site-plugin configuration is easy to do, will help users come back to normal reporting configuration, and will permit future enhancements (we need to finally completely clean-up the situation we created with this story...)

        Issue Links

          Activity

          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Jenkins build maven-3.x #1575 (See https://builds.apache.org/job/maven-3.x/1575/)
          MNG-6189 warn if maven-site-plugin reportPlugins is configured (hboutemy: http://git-wip-us.apache.org/repos/asf/?p=maven.git&a=commit&h=16382802611939a8876fe553208e8b0f4da606a6)

          • (edit) maven-model-builder/src/main/java/org/apache/maven/model/plugin/ReportingConverter.java
          • (edit) maven-model-builder/src/main/java/org/apache/maven/model/plugin/DefaultReportingConverter.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build maven-3.x #1575 (See https://builds.apache.org/job/maven-3.x/1575/ ) MNG-6189 warn if maven-site-plugin reportPlugins is configured (hboutemy: http://git-wip-us.apache.org/repos/asf/?p=maven.git&a=commit&h=16382802611939a8876fe553208e8b0f4da606a6 ) (edit) maven-model-builder/src/main/java/org/apache/maven/model/plugin/ReportingConverter.java (edit) maven-model-builder/src/main/java/org/apache/maven/model/plugin/DefaultReportingConverter.java

            People

            • Assignee:
              hboutemy Hervé Boutemy
              Reporter:
              hboutemy Hervé Boutemy
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development