Uploaded image for project: 'Maven Plugin Tools'
  1. Maven Plugin Tools
  2. MPLUGIN-452

Maven scope warning should be logged at WARN level

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 3.7.1, 3.8.1
    • Plugin Plugin
    • None

    Description

      Currently the plugin emits ERROR like this (from ITs):

      [INFO] --- plugin:3.7.0:descriptor (default-descriptor) @ plugin-extension ---
      [ERROR]
      
      Some dependencies of Maven Plugins are expected to be in provided scope.
      Please make sure that dependencies listed below declared in POM
      have set '<scope>provided</scope>' as well.
      
      The following dependencies are in wrong scope:
       * org.apache.maven:maven-model:jar:3.8.6:compile
       * org.apache.maven:maven-artifact:jar:3.8.6:compile
       * org.apache.maven:maven-resolver-provider:jar:3.8.6:compile
       * org.apache.maven:maven-model-builder:jar:3.8.6:compile
       * org.apache.maven:maven-builder-support:jar:3.8.6:compile
       * org.apache.maven:maven-repository-metadata:jar:3.8.6:compile
      
      
      
      [INFO] Using 'UTF-8' encoding to read mojo source files.
      [INFO] java-annotations mojo extractor found 0 mojo descriptor.
      [INFO] java-javadoc mojo extractor found 1 mojo descriptor.
      [WARNING]
      [WARNING] Deprecated extractor java-javadoc extracted 1 descriptor. Upgrade your Mojo definitions.
      [WARNING] You should use Mojo Annotations instead of Javadoc tags.
      [WARNING]
      [INFO] ant mojo extractor found 0 mojo descriptor.
      [INFO] bsh mojo extractor found 0 mojo descriptor. 

      The log level should be WARN instead as we do not fail the build.

      This output confuses Verifier and considers build failed even if it is not.

      Important implication: this also means that m-plugin-p versions currently released and logging at ERROR level are not usable in conjuction with Verifier (when any of two ERRORs are being logged).

      Attachments

        Issue Links

          Activity

            People

              cstamas Tamas Cservenak
              cstamas Tamas Cservenak
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: