Repository: flume Updated Branches: refs/heads/trunk 4ae19e8f9 -> dd7bfb3c0
FLUME-3299 Fix log4j scopes in pom files Moving log4j dependencies to test scope. Adding log4j as dependency to flume-ng-dist to pack it in the binary tarball. This closes #249 Reviewers: Endre Major, Peter Turcsanyi (Ferenc Szabo via Ferenc Szabo) Project: http://git-wip-us.apache.org/repos/asf/flume/repo Commit: http://git-wip-us.apache.org/repos/asf/flume/commit/dd7bfb3c Tree: http://git-wip-us.apache.org/repos/asf/flume/tree/dd7bfb3c Diff: http://git-wip-us.apache.org/repos/asf/flume/diff/dd7bfb3c Branch: refs/heads/trunk Commit: dd7bfb3c0742fa8fab43f4bc443147cc369c046a Parents: 4ae19e8 Author: Ferenc Szabo <[email protected]> Authored: Wed Nov 28 10:31:38 2018 +0100 Committer: Ferenc Szabo <[email protected]> Committed: Wed Nov 28 10:31:38 2018 +0100 ---------------------------------------------------------------------- flume-ng-auth/pom.xml | 2 ++ flume-ng-configuration/pom.xml | 2 ++ flume-ng-core/pom.xml | 1 + flume-ng-dist/pom.xml | 8 ++++++++ flume-ng-node/pom.xml | 1 + 5 files changed, 14 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/flume/blob/dd7bfb3c/flume-ng-auth/pom.xml ---------------------------------------------------------------------- diff --git a/flume-ng-auth/pom.xml b/flume-ng-auth/pom.xml index ab05dc7..e2b6d65 100644 --- a/flume-ng-auth/pom.xml +++ b/flume-ng-auth/pom.xml @@ -50,11 +50,13 @@ limitations under the License. <dependency> <groupId>org.apache.logging.log4j</groupId> <artifactId>log4j-slf4j-impl</artifactId> + <scope>test</scope> </dependency> <dependency> <groupId>org.apache.logging.log4j</groupId> <artifactId>log4j-1.2-api</artifactId> + <scope>test</scope> </dependency> <dependency> http://git-wip-us.apache.org/repos/asf/flume/blob/dd7bfb3c/flume-ng-configuration/pom.xml ---------------------------------------------------------------------- diff --git a/flume-ng-configuration/pom.xml b/flume-ng-configuration/pom.xml index ced620f..64ae5c6 100644 --- a/flume-ng-configuration/pom.xml +++ b/flume-ng-configuration/pom.xml @@ -48,11 +48,13 @@ limitations under the License. <dependency> <groupId>org.apache.logging.log4j</groupId> <artifactId>log4j-slf4j-impl</artifactId> + <scope>test</scope> </dependency> <dependency> <groupId>org.apache.logging.log4j</groupId> <artifactId>log4j-1.2-api</artifactId> + <scope>test</scope> </dependency> <dependency> http://git-wip-us.apache.org/repos/asf/flume/blob/dd7bfb3c/flume-ng-core/pom.xml ---------------------------------------------------------------------- diff --git a/flume-ng-core/pom.xml b/flume-ng-core/pom.xml index 8dc32e4..999bbb1 100644 --- a/flume-ng-core/pom.xml +++ b/flume-ng-core/pom.xml @@ -304,6 +304,7 @@ limitations under the License. <dependency> <groupId>org.apache.logging.log4j</groupId> <artifactId>log4j-1.2-api</artifactId> + <scope>test</scope> </dependency> <dependency> http://git-wip-us.apache.org/repos/asf/flume/blob/dd7bfb3c/flume-ng-dist/pom.xml ---------------------------------------------------------------------- diff --git a/flume-ng-dist/pom.xml b/flume-ng-dist/pom.xml index bcd8b72..de39c2f 100644 --- a/flume-ng-dist/pom.xml +++ b/flume-ng-dist/pom.xml @@ -236,6 +236,14 @@ <groupId>org.apache.flume</groupId> <artifactId>flume-ng-auth</artifactId> </dependency> + <dependency> + <groupId>org.apache.logging.log4j</groupId> + <artifactId>log4j-slf4j-impl</artifactId> + </dependency> + <dependency> + <groupId>org.apache.logging.log4j</groupId> + <artifactId>log4j-1.2-api</artifactId> + </dependency> </dependencies> </project> http://git-wip-us.apache.org/repos/asf/flume/blob/dd7bfb3c/flume-ng-node/pom.xml ---------------------------------------------------------------------- diff --git a/flume-ng-node/pom.xml b/flume-ng-node/pom.xml index 80ad34b..cc7bb13 100644 --- a/flume-ng-node/pom.xml +++ b/flume-ng-node/pom.xml @@ -87,6 +87,7 @@ <dependency> <groupId>org.apache.logging.log4j</groupId> <artifactId>log4j-1.2-api</artifactId> + <scope>test</scope> </dependency> <dependency>
