----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9900/#review18352 -----------------------------------------------------------
Ship it! applied to 4.1 - Chip Childers On March 22, 2013, 10:21 a.m., Pradeep Soundararajan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9900/ > ----------------------------------------------------------- > > (Updated March 22, 2013, 10:21 a.m.) > > > Review request for cloudstack, David Nalley, Chip Childers, Hugo Trippaers, > Alex Huang, Wido den Hollander, and Noa Resare. > > > Description > ------- > > BUG-ID: CLOUDSTACK-1252 Summary: Failed to download default template in VMware > > Proposing to modify package.sh script. Package as per OSS or NONOSS arguments. > > Signed-off-by: Pradeep <pradeep.soundarara...@citrix.com> > -------------------------------------------------------------------------------------- > > Now package.sh will act based on the below usage i.e. with or without > arguments: > > ./package.sh -h > > usage: ./package.sh [-p|--pack] [-h|--help] [ARGS] > > The commonly used Arguments are: > oss|OSS To package OSS specific > nonoss|NONOSS To package NONOSS specific > > Examples: ./package.sh -p|--pack oss|OSS > ./package.sh -p|--pack nonoss|NONOSS > ./package.sh (Default OSS) > > --------------------------------------------------------------------------------------- > > > Diffs > ----- > > packaging/centos63/cloud.spec a7a6927 > packaging/centos63/package.sh 2515ecb > > Diff: https://reviews.apache.org/r/9900/diff/ > > > Testing > ------- > > I have tested these changes. They are working well. Please review and submit > it if you are fine. > > In future, we can even change the mvn commands as per our need from > cloud.spec. > > > Thanks, > > Pradeep Soundararajan > >