[jira] [Commented] (MNG-6220) Add CLI options to control color output

2020-03-28 Thread Hudson (Jira)


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

Hudson commented on MNG-6220:
-

Build failed in Jenkins: Maven TLP » maven-studies » maven-metrics #4

See 
https://builds.apache.org/job/maven-box/job/maven-studies/job/maven-metrics/4/

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
>Priority: Major
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-11-08 Thread Tom Wieczorek (JIRA)

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

Tom Wieczorek commented on MNG-6220:


[~jgl...@netbeans.org] Thanks for pointing out. Just saw "Resolution Fixed" but 
not the link to the follow-up.

Sorry for the noise.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-11-08 Thread Jesse Glick (JIRA)

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

Jesse Glick commented on MNG-6220:
--

I was directed to MNG-6296 which tracks the problem.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-11-08 Thread Jesse Glick (JIRA)

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

Jesse Glick commented on MNG-6220:
--

The usual risks of relying on unit tests: you get what you pay for. :-/ 
[~rfscholte] can this be reopened? (I have no permissions to do so.)

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-11-08 Thread Daniel Beland (JIRA)

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

Daniel Beland commented on MNG-6220:


I'm sorry to say but it is not working and the test is wrong.

The test goes (MavenCliTest.java):


{code:java}
MessageUtils.setColorEnabled( true );
request = new CliRequest( new String[] { "-Dstyle.color=never" }, null 
);
cli.cli( request );
cli.properties( request );
cli.logging( request );
assertFalse( MessageUtils.isColorEnabled() );
{code}

While the real code goes (MavenCli.java):

{code:java}
try
{
initialize( cliRequest );
cli( cliRequest );
logging( cliRequest );
version( cliRequest );
properties( cliRequest );
localContainer = container( cliRequest );
commands( cliRequest );
configure( cliRequest );
toolchains( cliRequest );
populateRequest( cliRequest );
encryption( cliRequest );
repository( cliRequest );
return execute( cliRequest );
}
{code}

cli.logging is called before cli.properties while the test was calling them in 
a different order.
When I debug maven the userProperties is always empty and then default value of 
auto is assumed for the color logging.


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-11-08 Thread Tom Wieczorek (JIRA)

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

Tom Wieczorek commented on MNG-6220:


[~dcendents]: The PR was closed in favor of 
[785bad693c60ad60d7b307af8fab9e9234ff57bd|http://git-wip-us.apache.org/repos/asf/maven/commit/785bad69].

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-11-08 Thread Daniel Beland (JIRA)

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

Daniel Beland commented on MNG-6220:


It seems like it was not added to the maven 3.5.2 release.
Was the pull request merged?

Looking at the pull request, the parameter is '--color=always|never|auto': 
https://github.com/apache/maven/pull/114/files

But when I check the source code for CLIManager on tag maven-3.5.2 it is not 
there: 
https://github.com/apache/maven/blob/maven-3.5.2/maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java

I'm a bit confused...

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-11-08 Thread Tom Wieczorek (JIRA)

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

Tom Wieczorek commented on MNG-6220:


I'm somehow not able to see any change when using {{-Dstyle.color=XXX}} with 
Maven 3.5.2. It always uses the auto detection. Can somebody confirm that it's 
working? I tried {{never}} at the terminal and had colors. I tried {{always}} 
when piping, and got no colors. I also tried some bogus value to trigger the 
exception, but to no avail.

I tried both {{MAVEN_OPTS}} and as an argument to {{mvn}}.

{noformat}
Apache Maven 3.5.2 (138edd61fd100ec658bfa2d307c43b76940a5d7d; 
2017-10-18T09:58:13+02:00)
Maven home: /opt/apache-maven-3.5.2
Java version: 1.8.0_151, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-oracle/jre
Default locale: de_DE, platform encoding: UTF-8
OS name: "linux", version: "4.10.0-37-generic", arch: "amd64", family: "unix"
{noformat}


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.2
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-16 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user mryan43 closed the pull request at:

https://github.com/apache/maven-shared/pull/22


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.1
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-16 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user mryan43 commented on the issue:

https://github.com/apache/maven/pull/114
  
Great thanks @rfscholte ! 


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.1
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-16 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user mryan43 closed the pull request at:

https://github.com/apache/maven/pull/114


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.1
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-16 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user agudian commented on the issue:

https://github.com/apache/maven/pull/114
  
@rfscholte: looks good! 👍


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.1
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-15 Thread Hudson (JIRA)

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

Hudson commented on MNG-6220:
-

SUCCESS: Integrated in Jenkins build maven-3.x #1663 (See 
[https://builds.apache.org/job/maven-3.x/1663/])
[MNG-6220] Add CLI options to control color output Introduce (rfscholte: 
[http://git-wip-us.apache.org/repos/asf/?p=maven.git&a=commit&h=785bad693c60ad60d7b307af8fab9e9234ff57bd])
* (edit) maven-embedder/pom.xml
* (edit) maven-embedder/src/test/java/org/apache/maven/cli/MavenCliTest.java
* (edit) maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.1
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-15 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on MNG-6220:
-

Fixed in 
[785bad693c60ad60d7b307af8fab9e9234ff57bd|http://git-wip-us.apache.org/repos/asf/maven/commit/785bad69]

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>Assignee: Robert Scholte
> Fix For: 3.5.1
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-09 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user rfscholte commented on the issue:

https://github.com/apache/maven/pull/114
  
Here's my proposal: 
https://git-wip-us.apache.org/repos/asf?p=maven.git;a=commit;h=fd57754218e749305be1dd745fda9407960cf985


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-09 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user michael-o commented on the issue:

https://github.com/apache/maven/pull/114
  
I don't might to pick this up, but it won't happen before Sep for personal 
priorities.


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-09 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user mryan43 commented on the issue:

https://github.com/apache/maven/pull/114
  
I feel stuck until someone with more experience of the maven code base 
update the version of maven-shared-utils to 3.2.1-SNAPSHOT on the default 
branch, anyone up to the task ?


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-08-09 Thread Yves Schumann (JIRA)

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

Yves Schumann commented on MNG-6220:


Fixing this issue with the next Maven version would be really great. I'm having 
developers asking for colored output on the Jenkins builds nearly every day...

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-06-15 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on MNG-6220:
-

Jesse, I discussed this in detail in a PR on GitHUb that our support is 
mediorce. Cited several {{tree(1)}} code spots how we should make our stuff 
idiotproof. Yes, {{TERM=dumb}} is not checked but should be: 
https://github.com/apache/maven/pull/114

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-06-15 Thread Jesse Glick (JIRA)

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

Jesse Glick commented on MNG-6220:
--

I was surprised to find that {{TERM=dumb mvn willfail}} still prints ANSI 
sequences, contrary to the expectations of Unix users.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-05-29 Thread Cyrille Le Clerc (JIRA)

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

Cyrille Le Clerc commented on MNG-6220:
---

The [Jenkins Pipeline Maven 
Plugin|https://wiki.jenkins-ci.org/display/JENKINS/Pipeline+Maven+Plugin] would 
be very interested in having the colored output enable when the batch mode is 
enabled.
We run Maven builds in batch mode to be not interactive but we would like to 
benefit of the colorized output.

Reference https://issues.jenkins-ci.org/browse/JENKINS-44543

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-25 Thread Stephen Connolly (JIRA)

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

Stephen Connolly commented on MNG-6220:
---

I'd like to propose that we include this issue (however we end up fixing it) 
into scope for 3.5.1

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
> Fix For: 3.5.1-candidate
>
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-21 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on MNG-6220:
-

I still fail to see how you want to enfore Jansi to write color codes even if 
{{stdout}} is not attached to a terminal? the enum {{Style}} won't do this. Do 
you want to read it from {{MavenCli}}? Though, the property itself is fine for 
me.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-21 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on MNG-6220:
-

Let's do it like all colors are controlled as well:
e.g.
{{set MAVEN_OPTS=-Dstyle.color=enabled}}
This is one option to enable it for *every* build.

{{mvn  | less -R}}
or
{{mvn -Dstyle.color=enabled | less -R}}


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-21 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

GitHub user mryan43 opened a pull request:

https://github.com/apache/maven-shared/pull/22

Add color support auto detection in MessageUtils

Linked to PR https://github.com/apache/maven/pull/114 and task 
https://issues.apache.org/jira/browse/MNG-6220

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mryan43/maven-shared trunk

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/maven-shared/pull/22.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #22


commit 6f3e71b3a2d9cc57b4a45a7d4bd354094cebcc5c
Author: mryan43 
Date:   2017-04-20T22:38:39Z

add utility method to auto detect color support

commit 6839dd1d45a71d0667c58460a4e5a0e9588ea1f7
Author: mryan43 
Date:   2017-04-21T07:08:06Z

test for jansi availability when auto detecting color support




> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-20 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on MNG-6220:
-

Can you elaborate? How would you implement {{mvn --color=always | less -R}}?

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-20 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on MNG-6220:
-

I don't think that colors are build specific properties. I can't think of a 
reason why you want to have colors for some builds and no colors for others. 
Instead it is a global settings for your environment/system. Hence, 
SystemProperties  makes more sense to me.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-19 Thread Michael Osipov (JIRA)

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

Michael Osipov commented on MNG-6220:
-

[~rfscholte], in my opinion there is. If you are used to {{$command 
--color=always | less -R}} you want to have this with Maven. Even if I redirect 
the Maven output, I'd like to page through the file with colors. Nobody forces 
you to invoke it, but you can if you need to. I will make it simply consistent 
with many other tools.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-19 Thread Arnaud HERITIER (JIRA)

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

Arnaud HERITIER commented on MNG-6220:
--

I'm interested to have the batch mode with color but a system property could be 
enough

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-19 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on MNG-6220:
-

[~mryan] Just to let you know: Arnaud is working on both Maven and Jenkins 
(Cloudbees). 

We should try to only keep a limit set of CLI options. Based on Arnauds answer 
there's no need for such an option. And to me that makes sense: the output for 
Maven (console) and Jenkins (webpage) is different, so Jenkins must do output 
parsing anyway to enable/disable color support.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread Manuel Ryan (JIRA)

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

Manuel Ryan commented on MNG-6220:
--

Hello [~rfscholte] and [~aheritier]

A system property or environment variable would be ok for CI use cases, but a 
programmatic extension point for this is a bit more cumbersome as it would need 
specific integration code in all CI systems when they already support cli 
options and env vars out of the box.

We also need to consider how existing CLI options influence color (-B and -l). 
Configuring color behavior elsewhere could make it a harder to understand what 
is happening as some cli options would be starting to override system 
properties or vice-versa.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread Arnaud HERITIER (JIRA)

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

Arnaud HERITIER commented on MNG-6220:
--

Hi [~rfscholte], [~mryan]

I agree that it might be interesting to have the ability to programatically 
activate / deactivate the color output but from my POV a command line option is 
too much. It might be an extension point like proposed by Robert or a system 
property but not more. End users (developers) won't really need it. This is 
only useful for IDE, CI, .. and others integrations like mentioned here.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread Robert Scholte (JIRA)

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

Robert Scholte commented on MNG-6220:
-

I really wonder if there's a need for this in the form of a commandline option. 
[~aheritier] WDYT? My idea is that we need to introduce extension points for 
better CI support, just like we did with Maven 3 for better IDE support.

> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user mryan43 commented on the issue:

https://github.com/apache/maven/pull/114
  
Unfortunately, the system property is read only once in a static init block 
in the JansiConsole class before the cli main method is invoked, so my change 
doesn't work.

We have a kind of chicken and egg situation and we need to decide if we 
prefer 

a) not setting up Jansi until arguments/config have been parsed and 
validated (and call MessageUtils.systemInstall(); later in the loggin() method)

b) forcing jansi until arguments and config have been parsed (by adding 
System.setProperty( "jansi.force", "true" ); before 
MessageUtils.systemInstall(); in MavenCli#main and then eventually disable it 
later in the loggin() method

What do you think ?
 



> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user mryan43 commented on the issue:

https://github.com/apache/maven/pull/114
  
I will test with less and jenkins and comment again with results.


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user mryan43 commented on the issue:

https://github.com/apache/maven/pull/114
  
Good idea to align the name of choices with grep, I just changed it.

About "isatty", do you think we should set the jansi.force system property 
when --color is set to "always" ? 
https://github.com/fusesource/jansi/blob/2616142fda4425d779ac94a3d9bfa76412021b23/jansi/src/main/java/org/fusesource/jansi/AnsiConsole.java
 


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user michael-o commented on the issue:

https://github.com/apache/maven/pull/114
  
I think yes, grep doesn't do different. Please try the patch thoroughly!


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user michael-o commented on the issue:

https://github.com/apache/maven/pull/114
  
Two issues:

1. Unfortunately, Commons CLI does not offer valid values. If it would, it 
would fail if the option value is wrong. I would do so too.
2. You blindly `return` from `always`, but still forget that Jansi will 
ignore that because `isatty` returns false as soon as `stdout` is passed via 
pipe.


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user michael-o commented on the issue:

https://github.com/apache/maven/pull/114
  
They are several issues. People are used to 
[GNU](https://www.gnu.org/software/grep/manual/grep.html#General-Output-Control)
 and [BSD](https://www.freebsd.org/cgi/man.cgi?query=grep&sourceid=opensearch) 
tools like grep. They use values `never`, `always`, `auto`. Having said that I 
do think we need to do this exactly the same way: 
[`grep.c`](https://github.com/freebsd/freebsd/blob/master/usr.bin/grep/grep.c#L627-L647).
 Option `always` requires special attention due to `isatty` in combination with 
`less(1)`.


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

Github user michael-o commented on the issue:

https://github.com/apache/maven/pull/114
  
I will comment shortly.


> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (MNG-6220) Add CLI options to control color output

2017-04-18 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on MNG-6220:
-

GitHub user mryan43 opened a pull request:

https://github.com/apache/maven/pull/114

MNG-6220 add color CLI option

https://issues.apache.org/jira/browse/MNG-6220

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mryan43/maven master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/maven/pull/114.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #114


commit 472a8866549ed1ee4e2dd53b8f4ff219c7d2f95a
Author: mryan 
Date:   2017-04-18T08:57:23Z

MNG-6220 add color CLI option




> Add CLI options to control color output
> ---
>
> Key: MNG-6220
> URL: https://issues.apache.org/jira/browse/MNG-6220
> Project: Maven
>  Issue Type: New Feature
>Reporter: Manuel Ryan
>
> Currently, the only way to enable/disable color output is to use the 
> batch-mode or log-file options.
> If a user wants colored output but no interactivity (ie: jenkins environment 
> with the ansicolor plugin), there is no CLI option combination to support the 
> use-case.
> I propose to add an option to control output coloring directly.
> {noformat}
> --color=enabled <- color output always enabled
> --color=disabled <- color output always disabled
> --color=auto <- current behavior (default)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)