maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Osipov (JIRA)" <j...@codehaus.org>
Subject [jira] (MSHARED-335) Remove redundant executionId from reports
Date Sat, 17 May 2014 11:17:10 GMT

    [ https://jira.codehaus.org/browse/MSHARED-335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=346641#comment-346641
] 

Michael Osipov commented on MSHARED-335:
----------------------------------------

Why can't it be removed? {{MojoExecution}} has constructors where {{executionId}} is set to
null.

> Remove redundant executionId from reports
> -----------------------------------------
>
>                 Key: MSHARED-335
>                 URL: https://jira.codehaus.org/browse/MSHARED-335
>             Project: Maven Shared Components
>          Issue Type: Improvement
>          Components: maven-reporting-exec
>    Affects Versions: maven-reporting-exec-1.2
>            Reporter: Michael Osipov
>
> Currently, [following|http://maven.apache.org/shared/maven-reporting-exec/xref/org/apache/maven/reporting/exec/DefaultMavenReportExecutor.html#239]
is constructed:
> {code}
> MojoExecution mojoExecution = new MojoExecution( plugin, report.getGoal(), "report:"
+ report.getGoal() );
> {code}
> The output looks like this:
> {noformat}
> [INFO] >>> maven-javadoc-plugin:2.9.1:javadoc (report:javadoc) > generate-sources
@ apache-maven >>>
> {noformat}
> *report:* is always static and *:javadoc* is duplicated. So, the executionId contains
fixed and redundant information. Remove them and make the entire line concise.
> The output shall be:
> {noformat}
> [INFO] >>> maven-javadoc-plugin:2.9.1:javadoc > generate-sources @ apache-maven
>>>
> {noformat}
> This was discovered with MNG-5630.



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)

Mime
View raw message