maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Osipov (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (MSHARED-513) Regression: Created-By row in manifest does not include Maven version
Date Wed, 27 Apr 2016 12:46:12 GMT

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

Michael Osipov edited comment on MSHARED-513 at 4/27/16 12:45 PM:
------------------------------------------------------------------

Are you certain that {{systemProperties}} is the proper scope? If you are, please go ahead
and fix it since you know better ;-)


was (Author: michael-o):
You are certain that {{systemProperties}} is the proper scope? If you are, please go ahead
and fix it since you know better ;-)

> Regression: Created-By row in manifest does not include Maven version
> ---------------------------------------------------------------------
>
>                 Key: MSHARED-513
>                 URL: https://issues.apache.org/jira/browse/MSHARED-513
>             Project: Maven Shared Components
>          Issue Type: Bug
>          Components: maven-archiver
>    Affects Versions: maven-archiver-3.0.1
>         Environment: Maven 3.3.3, Oracle JDK 1.7.0_79
>            Reporter: Anders Hammar
>            Assignee: Anders Hammar
>
> This is a regression of MSHARED-38.
> The code has been changed to:
> {code}
> String mavenVersion = session.getUserProperties().getProperty( "maven.version" );
> {code}
> which is wrong. It should be:
> {code}
> String mavenVersion = session.getExecutionProperties().getProperty( "maven.version" );
> {code}
> The root cause is probably that the unit test has been incorrectly changed so that the
dummy Maven session is not correctly created. Possibly this slipped through when moving to
Maven 3.0 requirements.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message