> On Aug. 31, 2012, 3:07 p.m., Alex Huang wrote: > > Also, we need to add the mysql connector as a rpm dependency. That change > > is not in.
Wido has added the mysql connector as a rpm dependency: >> Ubuntu and Debian both ship the MySQL connector in a package. I already >> added a dependency for that: >> https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;a=commit;h=9064236879dc9f2f11538d891271545e1ff10e9c Chip>>I *think* that's right. Let's see how the Maven build process shakes out, since this is actually tied to that as well. We know that mysql-connector needs to be a system dependency, and not be distributed with the ASF release. One item that does come to mind though... shouldn't it be included in the DEB packages too? Can you add that? Great! Then I think you're right Pradeep. We can probably close out this topic for now. - Pradeep ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6752/#review10943 ----------------------------------------------------------- On Aug. 24, 2012, 10:19 a.m., Pradeep Soundararajan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6752/ > ----------------------------------------------------------- > > (Updated Aug. 24, 2012, 10:19 a.m.) > > > Review request for cloudstack, David Nalley and Alex Huang. > > > Description > ------- > > CS-15694:Remove MYSQL connector. Added the explicit class path for mysql > connector which will call the jar from the desired location > > > Diffs > ----- > > build/build-cloud.properties 552de7f74db7b715da70cc48ff4dc8945fa066f8 > build/developer.xml f2e5aa6463ec849a3e97343e82423ff0ac622222 > setup/db/deploy-db-dev.sh f149e9efd029bd311b1d247e21764b9103fd01d9 > > Diff: https://reviews.apache.org/r/6752/diff/ > > > Testing > ------- > > Executed: "ant build-all deploy-server deploydb" successfully. > > > Thanks, > > Pradeep Soundararajan > >
