You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oozie.apache.org by rk...@apache.org on 2012/12/19 00:56:09 UTC
svn commit: r1423722 - in /oozie/trunk: pom.xml release-log.txt
src/main/resources/ src/main/resources/checkstyle-header.txt
src/main/resources/checkstyle.xml
Author: rkanter
Date: Tue Dec 18 23:56:08 2012
New Revision: 1423722
URL: http://svn.apache.org/viewvc?rev=1423722&view=rev
Log:
OOZIE-1126 see if checkstyle works for oozie development. (jaoki via rkanter)
Added:
oozie/trunk/src/main/resources/
oozie/trunk/src/main/resources/checkstyle-header.txt
oozie/trunk/src/main/resources/checkstyle.xml
Modified:
oozie/trunk/pom.xml
oozie/trunk/release-log.txt
Modified: oozie/trunk/pom.xml
URL: http://svn.apache.org/viewvc/oozie/trunk/pom.xml?rev=1423722&r1=1423721&r2=1423722&view=diff
==============================================================================
--- oozie/trunk/pom.xml (original)
+++ oozie/trunk/pom.xml Tue Dec 18 23:56:08 2012
@@ -753,6 +753,26 @@
<findbugsXmlWithMessages>true</findbugsXmlWithMessages>
</configuration>
</plugin>
+
+ <!-- checkstyle plugin. Execute 'mvn verify' and look for checkstyle-result.xml under target folder -->
+ <plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-checkstyle-plugin</artifactId>
+ <version>2.9.1</version>
+ <executions>
+ <execution>
+ <goals>
+ <goal>check</goal>
+ </goals>
+ <configuration>
+ <consoleOutput>true</consoleOutput>
+ <includeTestSourceDirectory>true</includeTestSourceDirectory>
+ <configLocation>src/main/resources/checkstyle.xml</configLocation>
+ </configuration>
+ </execution>
+ </executions>
+ </plugin>
+
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
Modified: oozie/trunk/release-log.txt
URL: http://svn.apache.org/viewvc/oozie/trunk/release-log.txt?rev=1423722&r1=1423721&r2=1423722&view=diff
==============================================================================
--- oozie/trunk/release-log.txt (original)
+++ oozie/trunk/release-log.txt Tue Dec 18 23:56:08 2012
@@ -1,5 +1,6 @@
-- Oozie 3.4.0 release (trunk - unreleased)
+OOZIE-1126 see if checkstyle works for oozie development. (jaoki via rkanter)
OOZIE-1124 Split pig unit tests to a separate module (rohini via virag)
OOZIE-1087 Remove requirement of hive-default.xml from Hive action (rkanter)
OOZIE-1129 Add documentation for configurable filesystem support (rkanter)
Added: oozie/trunk/src/main/resources/checkstyle-header.txt
URL: http://svn.apache.org/viewvc/oozie/trunk/src/main/resources/checkstyle-header.txt?rev=1423722&view=auto
==============================================================================
--- oozie/trunk/src/main/resources/checkstyle-header.txt (added)
+++ oozie/trunk/src/main/resources/checkstyle-header.txt Tue Dec 18 23:56:08 2012
@@ -0,0 +1,18 @@
+/**
+ * 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.
+*/
+
Added: oozie/trunk/src/main/resources/checkstyle.xml
URL: http://svn.apache.org/viewvc/oozie/trunk/src/main/resources/checkstyle.xml?rev=1423722&view=auto
==============================================================================
--- oozie/trunk/src/main/resources/checkstyle.xml (added)
+++ oozie/trunk/src/main/resources/checkstyle.xml Tue Dec 18 23:56:08 2012
@@ -0,0 +1,44 @@
+<?xml version="1.0"?>
+<!DOCTYPE module PUBLIC "-//Puppy Crawl//DTD Check Configuration 1.2//EN" "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">
+<!--
+ 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.
+-->
+
+<module name="Checker">
+
+ <module name="RegexpSingleline">
+ <property name="severity" value="warning"/>
+ <property name="format" value="\s+$"/>
+ <property name="message" value="Line has trailing spaces."/>
+ </module>
+
+ <module name="Header">
+ <property name="severity" value="warning"/>
+ <property name="headerFile" value="src/main/resources/checkstyle-header.txt"/>
+ </module>
+
+ <module name="TreeWalker">
+
+ <module name="LineLength">
+ <property name="severity" value="warning"/>
+ <property name="max" value="132"/>
+ </module>
+
+ </module>
+
+</module>
+