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 <[email protected]> Authored: Wed Oct 12 19:32:13 2016 +0200 Committer: Mike Percy <[email protected]> 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>
