[
https://issues.apache.org/jira/browse/MPLUGIN-388?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17482947#comment-17482947
]
ASF GitHub Bot commented on MPLUGIN-388:
----------------------------------------
cstamas commented on a change in pull request #66:
URL: https://github.com/apache/maven-plugin-tools/pull/66#discussion_r793342175
##########
File path: maven-plugin-plugin/pom.xml
##########
@@ -296,7 +276,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-plugin-plugin</artifactId>
- <version>3.6.0</version><!-- cannot use ${mavenPluginToolsVersion}
property because release plugin would try to update -->
Review comment:
Um, no idea, I tend to avoid m-release-p, never used it (aside of Maven
projects). But, if this (otherwise perfectly valid) construct "does not work"
with m-release-p, isn't the release plugin that needs to be fixed? Or we want
to sprinkle "workarounds" all around as "release plugin would try to update"?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
> 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)
