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

Hervé Boutemy edited comment on MNG-6242 at 9/17/17 4:40 PM:
-------------------------------------------------------------

ok, I:
- didn't remember these {{$cygwin}} and {{$mingw}} variables that were 
available in {{mvn}} shell script
- didn't think about the {{test -t DESCRIPTOR}} to detect terminal
ok, if I need to code a workaround for Maven 3.5.2, I buy your implementation 
which is perfect :)
thanks for the detailed hint


was (Author: hboutemy):
ok, I didn't remember these {{$cygwin}} and {{$mingw}} variables that were 
available in {{mvn}} shell script: ok, if I need to code a workaround for Maven 
3.5.2, I buy your implementation :)
thanks for the detailed hint

> No color for maven on Cygwin
> ----------------------------
>
>                 Key: MNG-6242
>                 URL: https://issues.apache.org/jira/browse/MNG-6242
>             Project: Maven
>          Issue Type: Bug
>          Components: Command Line
>    Affects Versions: 3.5.0
>         Environment: Cygwin.  For example, my uname currently says:
> CYGWIN_NT-10.0 Beren 2.8.0(0.309/5/3) 2017-04-01 20:47 x86_64 Cygwin
>            Reporter: Brian Oxley
>            Priority: Minor
>             Fix For: 3.5.2-candidate
>
>
> Full description and workaround here: 
> https://binkley.blogspot.com/2017/04/maven-color-logging-on-cygwin.html
> In short: the JANSI library used in Maven 3.5.0 does not detect Cygwin, and 
> falls back to no color.  There is a workaround (Zart Colwing):  Add 
> {{-Djansi.passthrough=true}} to {{MAVEN_OPTS}}.
> As the mvn script understands Cygwin, it would be helpful to add the 
> passthrough flag automatically.  (Of course, there may need to be some smarts 
> about batch and logging modes, and checking if stdout is a console vs 
> pipeline.)
> Apologies: I'm unsure how to issue a PR for the maven repo.



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

Reply via email to