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