Just a heads up - I plan on reverting this change. While build-all still works, unit tests are failing, and deploydb is also failing, even when mysql java library is installed locally. The former I can live with for a short time, and the latter I can't.
Rajesh, Pradeep - Please take a look at this again, and fix these outstanding issues. --David On Mon, Aug 20, 2012 at 1:26 PM, David Nalley <da...@gnsa.us> wrote: > On Mon, Aug 20, 2012 at 1:11 PM, Chip Childers > <chip.child...@sungard.com> wrote: >> On Mon, Aug 20, 2012 at 1:05 PM, Alex Huang <alex.hu...@citrix.com> wrote: >>>> > >>>> > First, we make the mysql-connector-java.jar file a build dependency >>>> > that gets downloaded by the developer. Second, we do NOT distribute >>>> > this JAR in a binary release artifact hosted at ASF. Third, we >>>> > document it as a system dependency, and update the reference deb/spec >>>> > files. Last, if there is a non-package installation process, we >>>> > should include downloading the jar as a pre-requisite step for the >>>> > user. >>>> > >>> >>> Chip, >>> >>> Rajesh and Pradeep worked on removing all mysql specific code out of >>> CloudStack so it shouldn't be a build dependency any more. >>> >>> --Alex >>> >> >> Ok - thanks! >> >> David - I looked at https://reviews.apache.org/r/6180/ , but since >> master doesn't build without the JAR, I figured that it was still >> needed for the build. > > To my knowledge it hasn't been applied to master. I sucked it into deps-ctrl. > > I'll cherry pick to master shortly. > > --David