Repository: zeppelin
Updated Branches:
  refs/heads/master 085efeb64 -> e0da19324


[ZEPPELIN-3786]. Don't copy dependencies to target/lib for interpreter modules

### What is this PR for?
Only module zeppelin-interpreter,zeppelin,zengine,zeppelin-server needs that. 
They are just use it for testing purpose. Other modules don't need that.

### What type of PR is it?
[Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3786

### How should this be tested?
* CI pass

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Jeff Zhang <zjf...@apache.org>

Closes #3184 from zjffdu/ZEPPELIN-3786 and squashes the following commits:

7db5631b7 [Jeff Zhang] [ZEPPELIN-3786]. Don't copy dependencies to target/lib 
for interpreter modules


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

Branch: refs/heads/master
Commit: e0da19324c2d3c35ff7492f4fd8d0169723f2160
Parents: 085efeb
Author: Jeff Zhang <zjf...@apache.org>
Authored: Thu Sep 20 10:54:36 2018 +0800
Committer: Jeff Zhang <zjf...@apache.org>
Committed: Fri Sep 21 09:36:01 2018 +0800

----------------------------------------------------------------------
 pom.xml                             | 43 ++++++++++++++++----------------
 zeppelin-interpreter-parent/pom.xml |  9 +++++++
 2 files changed, 31 insertions(+), 21 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/e0da1932/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index a565e7c..bdc3334 100644
--- a/pom.xml
+++ b/pom.xml
@@ -339,6 +339,7 @@
   </dependencyManagement>
 
   <build>
+
     <plugins>
       <plugin>
         <artifactId>maven-compiler-plugin</artifactId>
@@ -581,27 +582,6 @@
         </plugin>
 
         <plugin>
-          <artifactId>maven-dependency-plugin</artifactId>
-          <version>${plugin.dependency.version}</version>
-          <executions>
-            <execution>
-              <id>copy-dependencies</id>
-              <phase>process-test-resources</phase>
-              <goals>
-                <goal>copy-dependencies</goal>
-              </goals>
-              <configuration>
-                
<outputDirectory>${project.build.directory}/lib</outputDirectory>
-                <overWriteReleases>false</overWriteReleases>
-                <overWriteSnapshots>false</overWriteSnapshots>
-                <overWriteIfNewer>true</overWriteIfNewer>
-                <includeScope>runtime</includeScope>
-              </configuration>
-            </execution>
-          </executions>
-        </plugin>
-
-        <plugin>
           <artifactId>maven-surefire-plugin</artifactId>
           <version>${plugin.surefire.version}</version>
           <configuration combine.children="append">
@@ -746,6 +726,27 @@
           </configuration>
         </plugin>
 
+        <plugin>
+          <artifactId>maven-dependency-plugin</artifactId>
+          <version>${plugin.dependency.version}</version>
+          <executions>
+            <execution>
+              <id>copy-dependencies</id>
+              <phase>process-test-resources</phase>
+              <goals>
+                <goal>copy-dependencies</goal>
+              </goals>
+              <configuration>
+                
<outputDirectory>${project.build.directory}/lib</outputDirectory>
+                <overWriteReleases>false</overWriteReleases>
+                <overWriteSnapshots>false</overWriteSnapshots>
+                <overWriteIfNewer>true</overWriteIfNewer>
+                <includeScope>runtime</includeScope>
+              </configuration>
+            </execution>
+          </executions>
+        </plugin>
+        
       </plugins>
     </pluginManagement>
   </build>

http://git-wip-us.apache.org/repos/asf/zeppelin/blob/e0da1932/zeppelin-interpreter-parent/pom.xml
----------------------------------------------------------------------
diff --git a/zeppelin-interpreter-parent/pom.xml 
b/zeppelin-interpreter-parent/pom.xml
index d4c5e51..efd3186 100644
--- a/zeppelin-interpreter-parent/pom.xml
+++ b/zeppelin-interpreter-parent/pom.xml
@@ -127,6 +127,15 @@
           </filesets>
         </configuration>
       </plugin>
+
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-dependency-plugin</artifactId>
+        <configuration>
+          <skip>true</skip>
+        </configuration>
+      </plugin>
+
     </plugins>
   </build>
 

Reply via email to