Skip to content

Commit

Permalink
MNG-6093 switched to monkey patched slf4j-simple provider supporting
Browse files Browse the repository at this point in the history
Maven color styles (reverts MNG-6038)
  • Loading branch information
hboutemy committed Oct 19, 2016
1 parent 904f797 commit 5bd5a6d
Show file tree
Hide file tree
Showing 9 changed files with 376 additions and 2 deletions.
4 changes: 2 additions & 2 deletions apache-maven/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ under the License.
<artifactId>aether-transport-wagon</artifactId>
</dependency>
<dependency>
<groupId>com.planet57.gossip</groupId>
<artifactId>gossip-slf4j</artifactId>
<groupId>org.apache.maven</groupId>
<artifactId>maven-slf4j-provider</artifactId>
</dependency>
<dependency>
<groupId>org.fusesource.jansi</groupId>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
# key = Slf4j effective logger factory implementation
# value = corresponding o.a.m.cli.logging.Slf4jConfiguration class
org.slf4j.impl.SimpleLoggerFactory org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
org.slf4j.impl.MavenSimpleLoggerFactory org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
org.apache.logging.slf4j.Log4jLoggerFactory org.apache.maven.cli.logging.impl.Log4j2Configuration
ch.qos.logback.classic.LoggerContext org.apache.maven.cli.logging.impl.LogbackConfiguration
com.planet57.gossip.Gossip org.apache.maven.cli.logging.impl.gossip.GossipConfiguration
116 changes: 116 additions & 0 deletions maven-slf4j-provider/pom.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,116 @@
<?xml version="1.0" encoding="UTF-8"?>

<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->

<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>

<parent>
<groupId>org.apache.maven</groupId>
<artifactId>maven</artifactId>
<version>3.4.0-SNAPSHOT</version>
</parent>

<artifactId>maven-slf4j-provider</artifactId>

<name>Maven SLF4J Simple Provider</name>
<description>
Maven SLF4J provider based on SLF4J's simple provider, monkey-patched to support Maven styled colors
for levels and stacktraces rendering.
</description>

<dependencies>
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-api</artifactId>
</dependency>
<dependency>
<groupId>org.apache.maven.shared</groupId>
<artifactId>maven-shared-utils</artifactId>
</dependency>
</dependencies>

<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-dependency-plugin</artifactId>
<configuration>
<artifactItems>
<artifactItem>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<version>${slf4jVersion}</version>
<type>jar</type>
<classifier>sources</classifier>
<overWrite>false</overWrite>
<outputDirectory>${project.build.directory}/generated-sources/slf4j-simple</outputDirectory>
<includes>org/slf4j/impl/*.java</includes>
</artifactItem>
</artifactItems>
</configuration>
<executions>
<execution>
<id>unzip-slf4j-simple</id>
<goals>
<goal>unpack</goal>
</goals>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>build-helper-maven-plugin</artifactId>
<version>1.12</version>
<executions>
<execution>
<id>add-slf4j-simple</id>
<phase>generate-sources</phase>
<goals>
<goal>add-source</goal>
</goals>
<configuration>
<sources>
<source>${project.build.directory}/generated-sources/slf4j-simple</source>
</sources>
</configuration>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.codehaus.gmaven</groupId>
<artifactId>groovy-maven-plugin</artifactId>
<version>2.0</version>
<executions>
<execution>
<id>patch-slf4j-simple</id>
<phase>process-sources</phase>
<goals>
<goal>execute</goal>
</goals>
<configuration>
<source>${project.basedir}/src/main/script/patch-slf4j-simple.groovy</source>
</configuration>
</execution>
</executions>
</plugin>
</plugins>
</build>
</project>
Original file line number Diff line number Diff line change
@@ -0,0 +1,117 @@
package org.slf4j.impl;

/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/

import static org.apache.maven.shared.utils.logging.MessageUtils.buffer;

import java.io.PrintStream;

/**
* Logger for Maven, that support colorization of levels and stacktraces.
* This class implements 2 methods introduced in slf4j-simple provider local copy.
* @since 3.4.0
*/
public class MavenSimpleLogger
extends SimpleLogger
{
private static final String NEWLINE = System.getProperty( "line.separator" );

MavenSimpleLogger( String name )
{
super( name );
}

@Override
protected String renderLevel( int level )
{
switch ( level )
{
case LOG_LEVEL_TRACE:
return buffer().debug( "TRACE" ).toString();
case LOG_LEVEL_DEBUG:
return buffer().debug( "DEBUG" ).toString();
case LOG_LEVEL_INFO:
return buffer().info( "INFO" ).toString();
case LOG_LEVEL_WARN:
return buffer().warning( "WARNING" ).toString();
case LOG_LEVEL_ERROR:
default:
return buffer().error( "ERROR" ).toString();
}
}

@Override
protected void renderThrowable( Throwable t, PrintStream stream )
{
stream.append( buffer().failure( t.getClass().getName() ).toString() );
if ( t.getMessage() != null )
{
stream.append( ": " );
stream.append( buffer().failure( t.getMessage() ).toString() );
}
stream.append( NEWLINE );

while ( t != null )
{
for ( StackTraceElement e : t.getStackTrace() )
{
stream.append( " " );
stream.append( buffer().strong( "at" ).toString() );
stream.append( " " + e.getClassName() + "." + e.getMethodName() );
stream.append( buffer().a( " (" ).strong( getLocation( e ) ).a( ")" ).toString() );
stream.append( NEWLINE );
}

t = t.getCause();
if ( t != null )
{
stream.append( buffer().strong( "Caused by" ).a( ": " ).a( t.getClass().getName() ).toString() );
if ( t.getMessage() != null )
{
stream.append( ": " );
stream.append( buffer().failure( t.getMessage() ).toString() );
}
stream.append( NEWLINE );
}
}
}

protected String getLocation( final StackTraceElement e )
{
assert e != null;

if ( e.isNativeMethod() )
{
return "Native Method";
}
else if ( e.getFileName() == null )
{
return "Unknown Source";
}
else if ( e.getLineNumber() >= 0 )
{
return String.format( "%s:%s", e.getFileName(), e.getLineNumber() );
}
else
{
return e.getFileName();
}
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
package org.slf4j.impl;

/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/

import org.slf4j.Logger;

public class MavenSimpleLoggerFactory
extends SimpleLoggerFactory
{
/**
* Return an appropriate {@link MavenSimpleLogger} instance by name.
*/
public Logger getLogger( String name )
{
Logger simpleLogger = loggerMap.get( name );
if ( simpleLogger != null )
{
return simpleLogger;
}
else
{
Logger newInstance = new MavenSimpleLogger( name );
Logger oldInstance = loggerMap.putIfAbsent( name, newInstance );
return oldInstance == null ? newInstance : oldInstance;
}
}
}
53 changes: 53 additions & 0 deletions maven-slf4j-provider/src/main/script/patch-slf4j-simple.groovy
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@

/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/

dir = new File( basedir, 'target/generated-sources/slf4j-simple/org/slf4j/impl' );

file = new File( dir, 'StaticLoggerBinder.java' );
content = file.text;

// check if already patched
if ( content.contains( 'MavenSimpleLoggerFactory' ) )
{
println ' slf4j-simple already patched';
return;
}


println ' patching StaticLoggerBinder.java';
content = content.replaceAll( 'SimpleLoggerFactory', 'MavenSimpleLoggerFactory' );
file.write( content );


println ' patching SimpleLogger.java';
file = new File( dir, 'SimpleLogger.java' );
content = file.text;
content = content.replaceAll( 'private static final int LOG_LEVEL_', 'protected static final int LOG_LEVEL_' );
content = content.replaceAll( 't.printStackTrace(TARGET_STREAM)', 'renderThrowable(t, TARGET_STREAM);' );

index = content.indexOf( 'switch (level) {' );
end = content.indexOf( '}', index ) + 1;
content = content.substring( 0, index ) + 'buf.append(renderLevel(level));' + content.substring( end );

content = content.substring( 0, content.lastIndexOf( '}' ) );
content += ' protected void renderThrowable(Throwable t, PrintStream stream) {}\n';
content += ' protected String renderLevel(int level) { return ""; }\n}\n';

file.write( content );
36 changes: 36 additions & 0 deletions maven-slf4j-provider/src/site/site.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
<?xml version="1.0" encoding="UTF-8"?>

<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->

<project xmlns="http://maven.apache.org/DECORATION/1.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/DECORATION/1.0.0 http://maven.apache.org/xsd/decoration-1.0.0.xsd">
<body>
<menu name="Overview">
<item name="Introduction" href="index.html"/>
<item name="JavaDocs" href="apidocs/index.html"/>
<item name="Source Xref" href="xref/index.html"/>
<!--item name="FAQ" href="faq.html"/-->
</menu>

<menu ref="parent"/>
<menu ref="reports"/>
</body>
</project>
Loading

0 comments on commit 5bd5a6d

Please sign in to comment.