root
/
kafka
mirror of https://github.com/apache/kafka.git
1
0
Fork 0
Code Issues Actions 4 Packages Projects Releases Wiki Activity

MINOR: Remove copyDependantTestLibs from jar dependencies 

_copyDependantTestLibs_ was added temporarily as a dependency of _jar_ task to enable SASL system tests to be run with MiniKdc without changing the automated system test runs which run _gradlew clean jar_. Since the build target _systemTestLibs_ is already in Kafka build.gradle, the Confluent automated test runs can now run _gradlew clean systemTestLibs_ instead. This PR provides the final change to remove _copyDependantTestLibs_ from the _jar_ task. This should be committed only after the Confluent automated sytem test build script is updated, to avoid breaking any builds.

Author: Rajini Sivaram <rajinisivaram@googlemail.com>

Reviewers: Ismael Juma <ismael@juma.me.uk>

Closes #430 from rajinisivaram/minor-systemtestlibs
This commit is contained in:
Rajini Sivaram 2016-05-09 23:36:22 +01:00 committed by Ismael Juma
parent ff8580da6f
commit 22c098b71b
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
build.gradle
View File

@ -437,10 +437,6 @@ project(':core') {
duplicatesStrategy 'exclude' duplicatesStrategy 'exclude'
} }
systemTestLibs {
dependsOn testJar
}
task genProtocolErrorDocs(type: JavaExec) { task genProtocolErrorDocs(type: JavaExec) {
classpath = sourceSets.main.runtimeClasspath classpath = sourceSets.main.runtimeClasspath
main = 'org.apache.kafka.common.protocol.Errors' main = 'org.apache.kafka.common.protocol.Errors'
@ -521,7 +517,7 @@ project(':core') {
} }
jar { jar {
dependsOn('copyDependantLibs', 'copyDependantTestLibs') dependsOn('copyDependantLibs')
} }
jar.manifest { jar.manifest {
@ -538,6 +534,8 @@ project(':core') {
duplicatesStrategy 'exclude' duplicatesStrategy 'exclude'
} }
systemTestLibs.dependsOn('jar', 'testJar', 'copyDependantTestLibs')
checkstyle { checkstyle {
configProperties = [importControlFile: "$rootDir/checkstyle/import-control-core.xml"] configProperties = [importControlFile: "$rootDir/checkstyle/import-control-core.xml"]
} }