incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Wido den Hollander" <w...@widodh.nl>
Subject Re: Review Request: 4.1 deb packaging tomcat fixes
Date Sat, 02 Mar 2013 16:35:20 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message