svn commit: r617973 - in /maven/plugins/trunk/maven-checkstyle-plugin/src/main/java/org/apache/maven/plugin/checkstyle: CheckstyleReport.java CheckstyleViolationCheckMojo.java

[de...@apache.org]

Author: dennisl
Date: Sun Feb  3 03:27:33 2008
New Revision: 617973

URL: http://svn.apache.org/viewvc?rev=617973&view=rev
Log:
[MCHECKSTYLE-54] checkstyle:check does not see provided scope dependencies

Modified:
    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

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=617973&r1=617972&r2=617973&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 Feb  3 03:27:33 2008
@@ -80,6 +80,7 @@
  * @author <a href="mailto:vincent.siveton@gmail.com">Vincent Siveton</a>
  * @author <a href="mailto:joakim@erdfelt.com">Joakim Erdfelt</a>
  * @goal checkstyle
+ * @requiresDependencyResolution compile
  */
 public class CheckstyleReport
     extends AbstractMavenReport

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=617973&r1=617972&r2=617973&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 Feb  3 03:27:33 2008
@@ -37,11 +37,12 @@
  * Perform a violation check against the last Checkstyle run to see if there are
  * any violations. It reads the Checkstyle output file, counts the number of
  * violations found and displays it on the console.
- * 
+ *
  * @author <a href="mailto:joakim@erdfelt.net">Joakim Erdfelt</a>
  * @goal check
  * @phase verify
  * @execute goal="checkstyle"
+ * @requiresDependencyResolution compile
  */
 public class CheckstyleViolationCheckMojo
     extends AbstractMojo
@@ -49,7 +50,7 @@
     /**
      * Specifies the path and filename to save the Checkstyle output. The format
      * of the output file is determined by the <code>outputFileFormat</code>
-     * 
+     *
      * @parameter expression="${checkstyle.output.file}"
      *            default-value="${project.build.directory}/checkstyle-result.xml"
      */
@@ -58,14 +59,14 @@
     /**
      * Specifies the format of the output to be used when writing to the output
      * file. Valid values are "plain" and "xml"
-     * 
+     *
      * @parameter expression="${checkstyle.output.format}" default-value="xml"
      */
     private String outputFileFormat;
 
     /**
      * Do we fail the build on a violation?
-     * 
+     *
      * @parameter expression="${checkstyle.failOnViolation}"
      *            default-value="true"
      */
@@ -73,7 +74,7 @@
 
     /**
      * Skip entire check.
-     * 
+     *
      * @parameter expression="${checkstyle.skip}" default-value="false"
      * @since 2.2
      */