You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by an...@apache.org on 2014/10/06 19:01:33 UTC

[1/4] git commit: Removing unused import in RestAnnotationProcessorTest

Repository: jclouds
Updated Branches:
  refs/heads/reactivate-checkstyle 92a6edf84 -> 78b51b2d6 (forced update)


Removing unused import in RestAnnotationProcessorTest


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/b41a1c04
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/b41a1c04
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/b41a1c04

Branch: refs/heads/reactivate-checkstyle
Commit: b41a1c04af45fa51014f3370fa3fcce7ab60215d
Parents: 9d6de66
Author: Andrew Phillips <an...@apache.org>
Authored: Mon Oct 6 13:00:15 2014 -0400
Committer: Andrew Phillips <an...@apache.org>
Committed: Mon Oct 6 13:00:38 2014 -0400

----------------------------------------------------------------------
 .../java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/b41a1c04/core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java b/core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java
index a16eb91..0854d77 100644
--- a/core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java
+++ b/core/src/test/java/org/jclouds/rest/internal/RestAnnotationProcessorTest.java
@@ -146,7 +146,6 @@ import com.google.common.io.ByteSource;
 import com.google.common.io.Files;
 import com.google.common.net.HttpHeaders;
 import com.google.common.reflect.Invokable;
-import com.google.common.util.concurrent.ListenableFuture;
 import com.google.inject.AbstractModule;
 import com.google.inject.ConfigurationException;
 import com.google.inject.Injector;


[4/4] git commit: Overriding checkstyle-plugin config in jclouds-resources

Posted by an...@apache.org.
Overriding checkstyle-plugin config in jclouds-resources

Otherwise, fails with a self-dependency


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/78b51b2d
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/78b51b2d
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/78b51b2d

Branch: refs/heads/reactivate-checkstyle
Commit: 78b51b2d6e6f04b0c220d7d6603ebc0af3e62cd9
Parents: cb86088
Author: Andrew Phillips <an...@apache.org>
Authored: Sun Oct 5 15:02:22 2014 -0500
Committer: Andrew Phillips <an...@apache.org>
Committed: Mon Oct 6 13:00:42 2014 -0400

----------------------------------------------------------------------
 project/pom.xml   |  2 --
 resources/pom.xml | 12 ++++++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/78b51b2d/project/pom.xml
----------------------------------------------------------------------
diff --git a/project/pom.xml b/project/pom.xml
index 52cbfed..c5e6693 100644
--- a/project/pom.xml
+++ b/project/pom.xml
@@ -1112,8 +1112,6 @@
               <includeTestSourceDirectory>true</includeTestSourceDirectory>
               <failOnViolation>true</failOnViolation>
               <failsOnError>true</failsOnError>
-              <!-- fails on itself as it uses the author tag in a module definition -->
-              <resourceExcludes>checkstyle.xml</resourceExcludes>
               <violationSeverity>warning</violationSeverity>
             </configuration>
           </plugin>

http://git-wip-us.apache.org/repos/asf/jclouds/blob/78b51b2d/resources/pom.xml
----------------------------------------------------------------------
diff --git a/resources/pom.xml b/resources/pom.xml
index 6282650..a133e0c 100644
--- a/resources/pom.xml
+++ b/resources/pom.xml
@@ -43,6 +43,18 @@
     </resources>
     <plugins>
       <plugin>
+        <artifactId>maven-checkstyle-plugin</artifactId>
+        <configuration>
+          <configLocation>checkstyle.xml</configLocation>
+          <includeTestSourceDirectory>true</includeTestSourceDirectory>
+          <failOnViolation>true</failOnViolation>
+          <failsOnError>true</failsOnError>
+          <!-- fails on itself as it uses the author tag in a module definition -->
+          <resourceExcludes>checkstyle.xml</resourceExcludes>
+          <violationSeverity>warning</violationSeverity>
+        </configuration>
+      </plugin>
+      <plugin>
         <groupId>org.gaul</groupId>
         <artifactId>modernizer-maven-plugin</artifactId>
         <configuration>


[2/4] git commit: Overriding modernizer-plugin config in jclouds-resources

Posted by an...@apache.org.
Overriding modernizer-plugin config in jclouds-resources

This fixes a self-dependency


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/53660f83
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/53660f83
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/53660f83

Branch: refs/heads/reactivate-checkstyle
Commit: 53660f8369b763f47ccc62a1c76cebdfefb8194a
Parents: b41a1c0
Author: Andrew Phillips <an...@apache.org>
Authored: Sun Oct 5 16:28:57 2014 -0500
Committer: Andrew Phillips <an...@apache.org>
Committed: Mon Oct 6 13:00:39 2014 -0400

----------------------------------------------------------------------
 resources/pom.xml | 12 ++++++++++++
 1 file changed, 12 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/53660f83/resources/pom.xml
----------------------------------------------------------------------
diff --git a/resources/pom.xml b/resources/pom.xml
index 6404210..6282650 100644
--- a/resources/pom.xml
+++ b/resources/pom.xml
@@ -41,5 +41,17 @@
         </includes>
       </resource>
     </resources>
+    <plugins>
+      <plugin>
+        <groupId>org.gaul</groupId>
+        <artifactId>modernizer-maven-plugin</artifactId>
+        <configuration>
+          <javaVersion>1.7</javaVersion>
+          <!-- ${project.basedir} required here as 1.1.0 of the modernizer plugin
+            can't find the exclusions file otherwise -->
+          <exclusionsFile>${project.basedir}/modernizer_exclusions.txt</exclusionsFile>
+        </configuration>
+      </plugin>
+    </plugins>
   </build>
 </project>


[3/4] git commit: Moving checkstyle-plugin config to profiles

Posted by an...@apache.org.
Moving checkstyle-plugin config to profiles


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/cb86088a
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/cb86088a
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/cb86088a

Branch: refs/heads/reactivate-checkstyle
Commit: cb86088a2317367fb017395fe6a7a7964b30a8d6
Parents: 53660f8
Author: Andrew Phillips <an...@apache.org>
Authored: Sun Oct 5 14:41:23 2014 -0500
Committer: Andrew Phillips <an...@apache.org>
Committed: Mon Oct 6 13:00:40 2014 -0400

----------------------------------------------------------------------
 project/pom.xml | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/cb86088a/project/pom.xml
----------------------------------------------------------------------
diff --git a/project/pom.xml b/project/pom.xml
index 0372aaa..52cbfed 100644
--- a/project/pom.xml
+++ b/project/pom.xml
@@ -708,14 +708,7 @@
       <plugin>
         <artifactId>maven-checkstyle-plugin</artifactId>
         <version>2.13</version>
-        <configuration>
-          <!-- configLocation configured via profiles -->
-          <violationSeverity>warning</violationSeverity>
-          <includeTestSourceDirectory>true</includeTestSourceDirectory>
-          <failOnViolation>true</failOnViolation>
-          <failsOnError>true</failsOnError>
-          <violationSeverity>warning</violationSeverity>
-        </configuration>
+        <!-- configuration and dependencies set via profiles -->
       </plugin>
       <plugin>
         <groupId>org.gaul</groupId>
@@ -1074,6 +1067,7 @@
             <artifactId>maven-checkstyle-plugin</artifactId>
             <configuration>
               <configLocation>../resources/checkstyle.xml</configLocation>
+              <includeTestSourceDirectory>true</includeTestSourceDirectory>
               <failOnViolation>true</failOnViolation>
               <failsOnError>true</failsOnError>
               <violationSeverity>warning</violationSeverity>
@@ -1115,6 +1109,7 @@
             <configuration>
               <!-- jclouds-resources has the checkstyle config in the classpath -->
               <configLocation>resources/checkstyle.xml</configLocation>
+              <includeTestSourceDirectory>true</includeTestSourceDirectory>
               <failOnViolation>true</failOnViolation>
               <failsOnError>true</failsOnError>
               <!-- fails on itself as it uses the author tag in a module definition -->