Repository: flume
Updated Branches:
  refs/heads/flume-1.7 a1b55d227 -> 511d86855


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/511d8685
Tree: http://git-wip-us.apache.org/repos/asf/flume/tree/511d8685
Diff: http://git-wip-us.apache.org/repos/asf/flume/diff/511d8685

Branch: refs/heads/flume-1.7
Commit: 511d868555dd4d16e6ce4fedc72c2d1454546707
Parents: a1b55d2
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 20:02:21 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/511d8685/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 185eac9..68296a0 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