Dave, Without the actual library, those things will break. A lot of unit testing is testing access to DB. Deploy db is more or less changes to mysql. I'm not sure how you expect Rajesh and Pradeep to fix them. Any suggestions?
--Alex > -----Original Message----- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Monday, August 20, 2012 6:08 PM > To: cloudstack-dev@incubator.apache.org > Cc: Rajesh Battala; Pradeep Soundararajan > Subject: Re: [ASFCS40] Help needed to resolve the mysql-connector blocker > issue > > 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
