Committed to trunk and branch 0.7.
+1 Good thinking & good fix David. Curious that the test scope is apparently trumping runtime and excluding the log4j dependencies – but I guess a bug in assembly that is ignored because best practice is just to include one module anyway.
+1 the binary release seems to be working fine. I will commit this to trunk and branch-0.7 and build a new RC.
Thanks David. I will give it a try now and if everything looks fine I will build a new RC.
changes the bin assembly so that it only outputs the whirr-cli dependencies, as a bonus the build is a bit faster.