You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Robert Scholte (JIRA)" <ji...@apache.org> on 2017/11/21 22:37:00 UTC
[jira] [Closed] (MNG-6296) New option -Dstyle.color is not working
[ https://issues.apache.org/jira/browse/MNG-6296?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Robert Scholte closed MNG-6296.
-------------------------------
Resolution: Fixed
Fix Version/s: (was: 3.5.3-candidate)
3.5.3
Fix in [b2afafe5a252d9f86b3e43cce8a06dc9cdc7a6d7|http://git-wip-us.apache.org/repos/asf/maven/commit/b2afafe5]
Thanks for the patch!
> 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
> 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
(v6.4.14#64029)