> On June 30, 2014, 8:07 p.m., Sebastien Goasguen wrote: > > After the updated review it looks like you are only changing the centOS > > scripts. What about Ubuntu/debs ? is that already fixed ?
Yes, I'm only changing the CentOS scripts. I have no Debian server available just now, so I didn't test the .deb packaging, but it seems already fixed for Debian/Ubuntu: In packaging/debian/init/cloud-usage, JDK_DIRS is set to: /usr/lib/jvm/java-7-openjdk-amd64 /usr/lib/jvm/java-7-openjdk-i386 /usr/lib/jvm/java-6-openjdk... This has been set by Wido den Hollander in commit 20070ee07f1718e1eee0f7256e3f1004116cfee8, and it includes common locations for OpenJDK 7 in Debian. - Olivier ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23168/#review47000 ----------------------------------------------------------- On June 30, 2014, 5:54 p.m., Olivier Lemasle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23168/ > ----------------------------------------------------------- > > (Updated June 30, 2014, 5:54 p.m.) > > > Review request for cloudstack and Sebastien Goasguen. > > > Repository: cloudstack-git > > > Description > ------- > > cloudstack-usage is currently configured to search JAVA_HOME only in a small > set of paths corresponding to Java 6 (/usr/lib/jvm/java-6-openjdk, > /usr/lib/jvm/java-6-openjdk-i386, etc). > > If cloudstack-usage is installed on a server with only Java 7, the service > cannot start and fails with the following error: > jsvc error: Cannot locate Java Home > > This commit adds Java 7 paths to the possible set of JAVA_HOME locations. > > > Diffs > ----- > > packaging/centos63/cloud-usage.rc e78f8c1 > > Diff: https://reviews.apache.org/r/23168/diff/ > > > Testing > ------- > > Package CloudStack RPMs for CentOS and install them on a system with > java-1.7.0-openjdk and no Java 6 JRE. > Check that cloudstack-usage works correctly. > > > Thanks, > > Olivier Lemasle > >
