Repository: flume
Updated Branches:
  refs/heads/trunk 14ab61ef1 -> 87d4c2c13


Remove test dependencies from binary release artifact

This patch removes some test-specific dependencies from the binary
release artifact. These were introduced by the new
flume-shared-kafka-test module that is intended for sharing test code.
Please see the new comment in bin.xml for more information.

Reviewers: Bessenyei Balázs Donát

Closes #70


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

Branch: refs/heads/trunk
Commit: 87d4c2c13862144eb578b211bcf800b2206834ff
Parents: 14ab61e
Author: Mike Percy <mpe...@apache.org>
Authored: Wed Oct 12 19:32:13 2016 +0200
Committer: Mike Percy <mpe...@apache.org>
Committed: Wed Oct 12 19:59:37 2016 +0200

----------------------------------------------------------------------
 flume-ng-dist/src/main/assembly/bin.xml | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flume/blob/87d4c2c1/flume-ng-dist/src/main/assembly/bin.xml
----------------------------------------------------------------------
diff --git a/flume-ng-dist/src/main/assembly/bin.xml 
b/flume-ng-dist/src/main/assembly/bin.xml
index 68814aa..932ccb2 100644
--- a/flume-ng-dist/src/main/assembly/bin.xml
+++ b/flume-ng-dist/src/main/assembly/bin.xml
@@ -38,6 +38,12 @@
       <useProjectArtifact>false</useProjectArtifact>
       <excludes>
         
<exclude>org.apache.flume.flume-ng-clients:flume-ng-log4jappender:jar:jar-with-dependencies</exclude>
+        <!-- The below test classes get pulled in by flume-shared-kafka-test,
+             but we don't want them in the final release classpath. Maybe there
+             is a better way to exclude all of the dependencies of that module;
+             for now, specifically exclude these test deps. -->
+        <exclude>junit:junit</exclude>
+        <exclude>org.hamcrest:hamcrest-core</exclude>
       </excludes>
     </dependencySet>
     <dependencySet>
@@ -50,8 +56,7 @@
     </dependencySet>
   </dependencySets>
 
-
-   <fileSets>
+  <fileSets>
     <fileSet>
       <directory>../</directory>
 

Reply via email to