commit 732141c72481ce8ecce020b8154a4b623cb37f5f
author Sijie Guo <sijie@apache.org> Tue Jan 10 22:59:27 2017 -0800
committer Sijie Guo <sijie@apache.org> Tue Jan 10 22:59:27 2017 -0800
tree d9354028ca44ac711c1dfb2c9a7bf04c0a3033ad
parent c5e921efeedbb4f49e0f777f66ffb8c2c4ddaa22

DL-183: Configure DL to generate source jar

also move the plugin version definition to the properties

Author: Sijie Guo <sijie@apache.org>

Reviewers: Jia Zhai <zhaijia@apache.org>

Closes #113 from sijie/sijie/generate_source and squashes the following commits:

6fa1ce5 [Sijie Guo] Merge branch 'master' into sijie/generate_source
d355d6a [Sijie Guo] Generate source jar during jar phase

pom.xml

diff --git a/pom.xml b/pom.xml
index 9915207..7b6d58f 100644
--- a/pom.xml
+++ b/pom.xml
@@ -114,11 +114,17 @@
     <stats-util.version>0.0.58</stats-util.version>
     <zookeeper.version>3.5.1-alpha</zookeeper.version>
     <!-- plugin dependencies -->
+    <apache-rat-plugin.version>0.7</apache-rat-plugin.version>
+    <cobertura-maven-plugin.version>2.7</cobertura-maven-plugin.version>
+    <coveralls-maven-plugin.version>4.1.0</coveralls-maven-plugin.version>
+    <findbugs-maven-plugin.version>3.0.3</findbugs-maven-plugin.version>
     <puppycrawl.checkstyle.version>6.19</puppycrawl.checkstyle.version>
     <maven-assembly-plugin.version>2.2.1</maven-assembly-plugin.version>
     <maven-checkstyle-plugin.version>2.17</maven-checkstyle-plugin.version>
     <maven-compiler-plugin.version>3.1</maven-compiler-plugin.version>
     <maven-jar-plugin.version>2.2</maven-jar-plugin.version>
+    <maven-javadoc-plugin.version>2.8</maven-javadoc-plugin.version>
+    <maven-source-plugin.version>2.2.1</maven-source-plugin.version>
     <maven-surefire-plugin.version>2.19.1</maven-surefire-plugin.version>
     <scrooge-maven-plugin.version>3.17.0</scrooge-maven-plugin.version>
   </properties>
@@ -127,7 +133,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
-        <version>2.8</version>
+        <version>${maven-javadoc-plugin.version}</version>
         <configuration>
           <!-- Avoid for missing javadoc comments to be marked as errors -->
           <additionalparam>-Xdoclint:none</additionalparam>
@@ -157,7 +163,7 @@
       </plugin>
       <plugin>
         <artifactId>maven-assembly-plugin</artifactId>
-        <version>2.2.1</version>
+        <version>${maven-assembly-plugin.version}</version>
         <configuration>
 	  <tarLongFileMode>gnu</tarLongFileMode>
           <descriptors>
@@ -168,11 +174,11 @@
       <plugin>
         <groupId>org.codehaus.mojo</groupId>
         <artifactId>findbugs-maven-plugin</artifactId>
-        <version>3.0.3</version>
+        <version>${findbugs-maven-plugin.version}</version>
       </plugin>
       <plugin>
         <artifactId>maven-compiler-plugin</artifactId>
-        <version>3.1</version>
+        <version>${maven-compiler-plugin.version}</version>
         <configuration>
           <source>1.7</source>
           <target>1.7</target>
@@ -181,11 +187,10 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-source-plugin</artifactId>
-        <version>2.2.1</version>
+        <version>${maven-source-plugin.version}</version>
         <executions>
           <execution>
             <id>attach-sources</id>
-            <phase>deploy</phase>
             <goals>
               <goal>jar</goal>
             </goals>
@@ -195,7 +200,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
-        <version>2.19.1</version>
+        <version>${maven-surefire-plugin.version}</version>
         <configuration>
           <redirectTestOutputToFile>true</redirectTestOutputToFile>
           <argLine>-Xmx3G -Djava.net.preferIPv4Stack=true -XX:MaxDirectMemorySize=2G</argLine>
@@ -206,7 +211,7 @@
       <plugin>
         <groupId>org.apache.rat</groupId>
         <artifactId>apache-rat-plugin</artifactId>
-        <version>0.7</version>
+        <version>${apache-rat-plugin.version}</version>
         <configuration>
           <excludes>
             <exclude>docs/**/*</exclude>
@@ -231,12 +236,12 @@
       <plugin>
         <groupId>org.eluder.coveralls</groupId>
         <artifactId>coveralls-maven-plugin</artifactId>
-        <version>4.1.0</version>
+        <version>${coveralls-maven-plugin.version}</version>
       </plugin>
       <plugin>
         <groupId>org.codehaus.mojo</groupId>
         <artifactId>cobertura-maven-plugin</artifactId>
-        <version>2.7</version>
+        <version>${cobertura-maven-plugin.version}</version>
         <configuration>
           <formats>
             <format>html</format>