You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Hervé BOUTEMY <he...@free.fr> on 2016/06/23 19:16:38 UTC

Re: maven git commit: [MNG-3507] ANSI Color logging for improved output visibility.

-1
look at the implementation of these methods, and you'll see that reset() is 
already integrated: that's the intend of these methods

Regards,

Hervé

Le jeudi 23 juin 2016 00:28:59 schulte@apache.org a écrit :
> Repository: maven
> Updated Branches:
>   refs/heads/master 1b2451e79 -> 521e07cfb
> 
> 
> [MNG-3507] ANSI Color logging for improved output visibility.
> 
> o Updated to add missing 'reset' calls.
> 
> 
> Project: http://git-wip-us.apache.org/repos/asf/maven/repo
> Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/521e07cf
> Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/521e07cf
> Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/521e07cf
> 
> Branch: refs/heads/master
> Commit: 521e07cfb053f802b478e4406ce19c89e954ad3c
> Parents: 1b2451e
> Author: Christian Schulte <sc...@apache.org>
> Authored: Thu Jun 23 02:28:09 2016 +0200
> Committer: Christian Schulte <sc...@apache.org>
> Committed: Thu Jun 23 02:28:09 2016 +0200
> 
> ----------------------------------------------------------------------
>  .../org/apache/maven/lifecycle/LifecycleExecutionException.java  | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> ----------------------------------------------------------------------
> 
> 
> http://git-wip-us.apache.org/repos/asf/maven/blob/521e07cf/maven-core/src/ma
> in/java/org/apache/maven/lifecycle/LifecycleExecutionException.java
> ---------------------------------------------------------------------- diff
> --git
> a/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutionExc
> eption.java
> b/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutionExc
> eption.java index 5645abd..ef6fd1d 100644
> ---
> a/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutionExc
> eption.java +++
> b/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutionExc
> eption.java @@ -93,12 +93,12 @@ public class LifecycleExecutionException
>          if ( project != null )
>          {
>              buffer.a( " on project " );
> -            buffer.project( project.getArtifactId() );
> +            buffer.project( project.getArtifactId() ).reset();
>          }
> 
>          if ( cause != null )
>          {
> -            buffer.a( ": " ).failure( cause.getMessage() );
> +            buffer.a( ": " ).failure( cause.getMessage() ).reset();
>          }
> 
>          return buffer.toString();


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: maven git commit: [MNG-3507] ANSI Color logging for improved output visibility.

Posted by Christian Schulte <cs...@schulte.it>.
Am 06/23/16 um 21:16 schrieb Herv BOUTEMY:
> -1
> look at the implementation of these methods, and you'll see that reset() is 
> already integrated: that's the intend of these methods
> 

Ok. I am running into an issue where 'cause.getMessage()' returns
multiple lines screwing up the console displaying red characters which
should not have been red. Will try to reproduce it here reliably now.



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org