> On March 2, 2013, 4:35 p.m., Wido den Hollander wrote: > > Seems partially right. This is against the 4.1 branch, correct? Since the > > logfile fixes are already in master. > > > > Best practice is to check master first and cherry pick stuff to 4.1
Yes, this is against the 4.1 branch. cherry-pick from master sounds like a better way to do it. I'll update the patch - Noa ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9714/#review17302 ----------------------------------------------------------- On March 2, 2013, 1:55 p.m., Noa Resare wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9714/ > ----------------------------------------------------------- > > (Updated March 2, 2013, 1:55 p.m.) > > > Review request for cloudstack and Wido den Hollander. > > > Description > ------- > > Changes isolated to the deb package build files, to fix > failure to start management server from deb built packages > > * Adds the tomcat6 bootstrap jar to outer classpath > * Removes install of cloud-server-4.1.0-SNAPSHOT.jar > in /usr/share/cloudstack-management/lib. This causes > /usr/share/cloudstack-management structure to be in the > way tomcat expects it to > * Update paths to logfiles, which restores log4j functionality > > CLOUDSTACK-1490: 4.1 deb management fails to start > > > This addresses bug CLOUDSTACK-1490. > > > Diffs > ----- > > debian/rules d537d86 > packaging/debian/replace.properties d891e79 > > Diff: https://reviews.apache.org/r/9714/diff/ > > > Testing > ------- > > > Thanks, > > Noa Resare > >
