maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MNG-6296) New option -Dstyle.color is not working
Date Thu, 18 Jan 2018 20:36:01 GMT

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

Hudson commented on MNG-6296:
-----------------------------

Build succeeded in Jenkins: maven-3.x-jenkinsfile » master #157

See https://builds.apache.org/job/maven-3.x-jenkinsfile/job/master/157/

> New option -Dstyle.color is not working
> ---------------------------------------
>
>                 Key: MNG-6296
>                 URL: https://issues.apache.org/jira/browse/MNG-6296
>             Project: Maven
>          Issue Type: Bug
>    Affects Versions: 3.5.2
>            Reporter: Robin Müller
>            Assignee: Robert Scholte
>            Priority: Major
>             Fix For: 3.5.3
>
>
> The new option {{-Dstyle.color=(auto|always|never)}} is not working. This happens as
the corresponding code accesses the {{UserProperties}} of the {{CliRequest}} before they are
set. I'll provide a quick fix for this by changing the order of the methods in the {{doMain}}
method so the properties will be set at the right time.
> But I'd suggest to refactor the whole stuff so the code will become easier to understand
and maintain. IMHO the functionality of the {{properties}} method in the {{MavenCli}} should
be moved to the constructor of the {{CliRequest}}. The {{CliRequest}} itself should be immutable
so all fields will have the correct content all the time.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message