> On Aug. 16, 2012, 8:14 p.m., edison su wrote: > > cloud.spec, line 535 > > <https://reviews.apache.org/r/6658/diff/1/?file=141513#file141513line535> > > > > I think we need config(noreplace) for this configuration files: > > db.properties/log4j-cloud.xml/tomcat6.conf. Admin usually will change these > > configuration files to suite their needs, without noreplace, these files > > will be replaced with a new cloudstack RPM upgrade.
The reason that was pulled is because line 531 includes db.properties, and so we get an RPM build error stating that the file was asked for twice. Like 531 is already (noreplace) - Marcus ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6658/#review10428 ----------------------------------------------------------- On Aug. 16, 2012, 7:26 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6658/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2012, 7:26 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > Update cloud.spec for current state of 4.0, minor waf and init script fixes > for RHEL/CENTOS/Fedora > > > Diffs > ----- > > agent/distro/centos/SYSCONFDIR/rc.d/init.d/cloud-agent.in 488ced2 > agent/distro/fedora/SYSCONFDIR/rc.d/init.d/cloud-agent.in 488ced2 > agent/distro/rhel/SYSCONFDIR/rc.d/init.d/cloud-agent.in 488ced2 > cloud.spec 47a457a > wscript 9e9acbd > > Diff: https://reviews.apache.org/r/6658/diff/ > > > Testing > ------- > > Tested on Centos 6.2 x64 > Build RPMS via './waf rpm' : passed > Installed RPMS to management server : passed > Installed RPMs to hypervisor host : passed > Started/stopped via init scripts : passed > Create zone, deploy instance : passed > > > Thanks, > > Marcus Sorensen > >
