Kafka has very simple build requirements and a system like Ant is well suited for a clean and concise build. I have an experimental patch that does just this - replaces SBT with Ant+Ivy. IMO, this approach is cleaner, clearer, and more developer friendly.
Dependencies are localized to one directory in the project rather than living in ~/.ivy2 and elsewhere. This makes manual classpath building very simple (just one glob) and also makes packaging the libs very easy.
Testing is done through junit rather than scalatest. The Kafka tests use `org.scalatest.junit.JUnitSuite` which allow the tests to be executed through the junit test runner.
Management of the Scala version is handled through Ivy. The way I have laid out the Ant script, the Scala version can be changed by setting a different runtime property (-Dscala.version=2.8.2). Cross-compilation of the Kafka artifact would be simple to add.
The one downside to this approach is lack of an incremental build. `scalac` is deprecating its incremental build capabilities in coming versions. The suggested solution to this is to use an IDE that supports incremental builds.
The main motivation for this approach, to me at least, is that a developer can look at build.xml and immediately understand what is going on (with the exception maybe of the <ivy: .../> actions which would not be changing). This is largely not true for SBT unless someone is already familiar with SBT.
|Field||Original Value||New Value|
|Attachment||0001-Compile-and-package-Kafka-with-Ant-Ivy.patch [ 12577522 ]|
|Attachment||0002-Tests-all-passing.patch [ 12577523 ]|
|Attachment||0003-Parameterize-scala-version-add-offline-mode.patch [ 12577524 ]|
|Status||Open [ 1 ]||Resolved [ 5 ]|
|Resolution||Fixed [ 1 ]|
|Workflow||no-reopen-closed, patch-avail [ 12775778 ]||Apache Kafka Workflow [ 13050911 ]|
|Workflow||Apache Kafka Workflow [ 13050911 ]||no-reopen-closed, patch-avail [ 13054475 ]|
|Transition||Time In Source Status||Execution Times||Last Executer||Last Execution Date|
|346d 9h 22m||1||Jay Kreps||20/Mar/14 22:05|