[jira] [Commented] (MPLUGIN-388) Make reporting optional dependency

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MPLUGIN-388?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17482524#comment-17482524 ] 

ASF GitHub Bot commented on MPLUGIN-388:
----------------------------------------

slawekjaranowski commented on a change in pull request #66:
URL: https://github.com/apache/maven-plugin-tools/pull/66#discussion_r792699198



##########
File path: maven-plugin-plugin/pom.xml
##########
@@ -270,8 +249,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-plugin-plugin</artifactId>
-        <version>3.6.0</version>
-        <!-- will use previous maven-plugin-plugin release to build current maven-plugin-plugin as configured in parent -->
+        <version>3.6.4</version>

Review comment:
       please also update version for reporting -> m-p-p




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Make reporting optional dependency
> ----------------------------------
>
>                 Key: MPLUGIN-388
>                 URL: https://issues.apache.org/jira/browse/MPLUGIN-388
>             Project: Maven Plugin Tools
>          Issue Type: Task
>          Components: Plugin Plugin
>            Reporter: Tamás Cservenák
>            Priority: Major
>             Fix For: 3.7.0
>
>
> Current m-p-p is far from being "lightweight", mostly due maven-reporting-impl being pulled in (that pulls in doxia, http-client and almost whole universe), and all this due one single mojo, the {{report}} mojo (org.apache.maven.plugin.plugin.PluginReport), and all these dependencies are needed ONLY when mojo directly invoked (ie. {{{}mvn m-p-p:report{}}}), as when used as part of site, all these are provided by site plugin.
> Make this dependency optional. This implies, that users of this release using report mojo "standalone" MUST ADD maven-reporting-impl:3.0.0 as dependency to m-p-p plugin as plugin dependency.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)