----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9714/#review17302 -----------------------------------------------------------
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 - Wido den Hollander 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 > >
