You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2015/01/18 17:41:02 UTC
svn commit: r1652790 - in
/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle:
./ exec/
Author: hboutemy
Date: Sun Jan 18 16:41:01 2015
New Revision: 1652790
URL: http://svn.apache.org/r1652790
Log:
use injected log
Modified:
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleAggregateReport.java
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleViolationCheckMojo.java
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorRequest.java
maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.java
Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleAggregateReport.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleAggregateReport.java?rev=1652790&r1=1652789&r2=1652790&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleAggregateReport.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleAggregateReport.java Sun Jan 18 16:41:01 2015
@@ -70,7 +70,7 @@ public class CheckstyleAggregateReport
.setResourceIncludes( resourceIncludes )
.setResourceExcludes( resourceExcludes )
.setIncludeTestSourceDirectory( includeTestSourceDirectory ).setListener( getListener() )
- .setLog( getLog() ).setProject( project ).setSourceDirectories( getSourceDirectories() )
+ .setProject( project ).setSourceDirectories( getSourceDirectories() )
.setResources( resources ).setTestResources( testResources )
.setStringOutputStream( stringOutputStream ).setSuppressionsLocation( suppressionsLocation )
.setTestSourceDirectories( getTestSourceDirectories() ).setConfigLocation( configLocation )
Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java?rev=1652790&r1=1652789&r2=1652790&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleReport.java Sun Jan 18 16:41:01 2015
@@ -170,7 +170,7 @@ public class CheckstyleReport
.setIncludeResources( includeResources )
.setIncludeTestResources( includeTestResources )
.setIncludeTestSourceDirectory( includeTestSourceDirectory ).setListener( getListener() )
- .setLog( getLog() ).setProject( project ).setSourceDirectories( getSourceDirectories() )
+ .setProject( project ).setSourceDirectories( getSourceDirectories() )
.setResources( resources )
.setStringOutputStream( stringOutputStream ).setSuppressionsLocation( suppressionsLocation )
.setTestSourceDirectories( getTestSourceDirectories() ).setConfigLocation( configLocation )
Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleViolationCheckMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleViolationCheckMojo.java?rev=1652790&r1=1652789&r2=1652790&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleViolationCheckMojo.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleViolationCheckMojo.java Sun Jan 18 16:41:01 2015
@@ -520,7 +520,7 @@ public class CheckstyleViolationCheckMoj
.setIncludeResources( includeResources )
.setIncludeTestResources( includeTestResources )
.setIncludeTestSourceDirectory( includeTestSourceDirectory ).setListener( getListener() )
- .setLog( getLog() ).setProject( project ).setSourceDirectories( getSourceDirectories() )
+ .setProject( project ).setSourceDirectories( getSourceDirectories() )
.setResources( resources )
.setStringOutputStream( stringOutputStream ).setSuppressionsLocation( suppressionsLocation )
.setTestSourceDirectories( getTestSourceDirectories() ).setConfigLocation( configLocation )
Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorRequest.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorRequest.java?rev=1652790&r1=1652789&r2=1652790&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorRequest.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorRequest.java Sun Jan 18 16:41:01 2015
@@ -27,7 +27,6 @@ import java.util.List;
import org.apache.maven.artifact.Artifact;
import org.apache.maven.model.Resource;
-import org.apache.maven.plugin.logging.Log;
import org.apache.maven.project.MavenProject;
import com.puppycrawl.tools.checkstyle.DefaultLogger;
@@ -63,8 +62,6 @@ public class CheckstyleExecutorRequest
private MavenProject project;
- private Log log;
-
private String suppressionsLocation;
private boolean includeTestSourceDirectory;
@@ -195,17 +192,6 @@ public class CheckstyleExecutorRequest
return this;
}
- public Log getLog()
- {
- return log;
- }
-
- public CheckstyleExecutorRequest setLog( Log log )
- {
- this.log = log;
- return this;
- }
-
public String getSuppressionsLocation()
{
return suppressionsLocation;
Modified: maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.java?rev=1652790&r1=1652789&r2=1652790&view=diff
==============================================================================
--- maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.java (original)
+++ maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.java Sun Jan 18 16:41:01 2015
@@ -187,9 +187,8 @@ public class DefaultCheckstyleExecutor
if ( outputDirectoryFile.exists() )
{
URL outputDirectoryUrl = outputDirectoryFile.toURL();
- request.getLog().debug(
- "Adding the outputDirectory " + outputDirectoryUrl.toString()
- + " to the Checkstyle class path" );
+ getLogger().debug( "Adding the outputDirectory " + outputDirectoryUrl.toString()
+ + " to the Checkstyle class path" );
urls.add( outputDirectoryUrl );
}
}
@@ -274,7 +273,7 @@ public class DefaultCheckstyleExecutor
if ( message.length() > 0 )
{
- request.getLog().info( message );
+ getLogger().info( message );
}
}
@@ -291,7 +290,7 @@ public class DefaultCheckstyleExecutor
}
else
{
- request.getLog().info( message );
+ getLogger().info( message );
}
}
@@ -361,9 +360,8 @@ public class DefaultCheckstyleExecutor
if ( StringUtils.isEmpty( request.getEncoding() ) )
{
- request.getLog().warn(
- "File encoding has not been set, using platform encoding " + effectiveEncoding
- + ", i.e. build is platform dependent!" );
+ getLogger().warn( "File encoding has not been set, using platform encoding " + effectiveEncoding
+ + ", i.e. build is platform dependent!" );
}
if ( "Checker".equals( config.getName() )
@@ -387,7 +385,7 @@ public class DefaultCheckstyleExecutor
}
else
{
- request.getLog().warn( "Failed to configure file encoding on module " + config );
+ getLogger().warn( "Failed to configure file encoding on module " + config );
}
}
Configuration[] modules = config.getChildren();
@@ -416,7 +414,7 @@ public class DefaultCheckstyleExecutor
}
else
{
- request.getLog().warn( "Failed to configure cache file on module " + module );
+ getLogger().warn( "Failed to configure cache file on module " + module );
}
}
}