From 335f5943f40f2e368b5bfdad3e425ba437130092 Mon Sep 17 00:00:00 2001
From: Ignasi Barrera <ignasi.barrera@gmail.com>
Date: Sun, 30 Jun 2013 18:34:18 +0200
Subject: [PATCH] JCLOUDS-149: Reuse checkstyle configuration

---
 project/pom.xml                       | 20 ++++++++++++++++++--
 resources/checkstyle.xml              |  3 ---
 resources/checkstyle_suppressions.xml | 27 ---------------------------
 resources/pom.xml                     |  7 +++++++
 4 files changed, 25 insertions(+), 32 deletions(-)
 delete mode 100644 resources/checkstyle_suppressions.xml

diff --git a/project/pom.xml b/project/pom.xml
index 339ce3329e..6fd82933a1 100644
--- a/project/pom.xml
+++ b/project/pom.xml
@@ -652,11 +652,18 @@
         </configuration>
       </plugin>
       <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-checkstyle-plugin</artifactId>
         <version>2.9.1</version>
+        <dependencies>
+          <dependency>
+            <groupId>org.apache.jclouds</groupId>
+            <artifactId>jclouds-resources</artifactId>
+            <version>${project.version}</version>
+          </dependency>
+        </dependencies>
         <configuration>
-          <configLocation>../resources/checkstyle.xml</configLocation>
+          <!-- jclouds-resources has the checkstyle config in the classpath -->
+          <configLocation>resources/checkstyle.xml</configLocation>
           <violationSeverity>warning</violationSeverity>
         </configuration>
       </plugin>
@@ -1017,6 +1024,15 @@
               </execution>
             </executions>
           </plugin>        
+          <plugin>
+            <!-- When building jclouds-project, override the config to use the local file -->
+            <artifactId>maven-checkstyle-plugin</artifactId>
+            <version>2.9.1</version>
+            <configuration>
+              <configLocation>../resources/checkstyle.xml</configLocation>
+              <violationSeverity>warning</violationSeverity>
+            </configuration>
+          </plugin>
         </plugins>            
       </build>
     </profile>
diff --git a/resources/checkstyle.xml b/resources/checkstyle.xml
index de81e0f3d1..008f4ba91e 100644
--- a/resources/checkstyle.xml
+++ b/resources/checkstyle.xml
@@ -22,9 +22,6 @@
 <module name="Checker">
     <property name="severity" value="warning"/>
     <module name="NewlineAtEndOfFile"/>
-    <module name="SuppressionFilter">
-        <property name="file" value="resources/checkstyle_suppressions.xml"/>
-    </module>
     <module name="TreeWalker">
         <module name="AvoidStarImport"/>
         <module name="EmptyStatement"/>
diff --git a/resources/checkstyle_suppressions.xml b/resources/checkstyle_suppressions.xml
deleted file mode 100644
index 54c685015c..0000000000
--- a/resources/checkstyle_suppressions.xml
+++ /dev/null
@@ -1,27 +0,0 @@
-<?xml version="1.0"?>
-<!--
-
-    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.
-
--->
-
-<!DOCTYPE suppressions PUBLIC
-    "-//Puppy Crawl//DTD Suppressions 1.1//EN"
-    "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd">
-
-<suppressions>
-    <suppress files="Base64\.java" checks="UnnecessaryParentheses"/>
-</suppressions>
diff --git a/resources/pom.xml b/resources/pom.xml
index b1fd20e5a4..b6092b25ab 100644
--- a/resources/pom.xml
+++ b/resources/pom.xml
@@ -40,6 +40,13 @@
           <include>NOTICE.txt</include>
         </includes>
       </resource>
+      <resource>
+        <targetPath>resources</targetPath>
+        <directory>${project.basedir}</directory>
+        <includes>
+            <include>checkstyle.xml</include>
+        </includes>
+      </resource>
     </resources>
     <plugins>
       <plugin>