You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@deltacloud.apache.org by Chris Lalancette <cl...@redhat.com> on 2011/09/13 21:46:50 UTC

[PATCH] Make sure to quote the exec line in the initscript.

This is so we put deltacloudd into the background, *not* daemon.

Signed-off-by: Chris Lalancette <cl...@redhat.com>
---
 server/support/fedora/deltacloud-core |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/server/support/fedora/deltacloud-core b/server/support/fedora/deltacloud-core
index 9dbb900..a0e5fe7 100755
--- a/server/support/fedora/deltacloud-core
+++ b/server/support/fedora/deltacloud-core
@@ -44,7 +44,7 @@ start() {
     [ -x $exec ] || exit 5
 
     echo -n $"Starting $prog: "
-    daemon --user nobody $exec -i $DRIVER -e $API_ENV --port $PORT >> $LOGFILE 2>&1 &
+    daemon --user nobody "$exec -i $DRIVER -e $API_ENV --port $PORT >> $LOGFILE 2>&1 &"
     retval=$?
 
     if [ $retval -eq 0 ] && touch $lockfile ; then
-- 
1.7.4.4


Re: [PATCH] Make sure to quote the exec line in the initscript.

Posted by Chris Lalancette <cl...@redhat.com>.
On 09/14/11 - 11:02:20AM, Michal Fojtik wrote:
> 
> On Sep 13, 2011, at 9:46 PM, Chris Lalancette wrote:
> 
> > This is so we put deltacloudd into the background, *not* daemon.
> > 
> > Signed-off-by: Chris Lalancette <cl...@redhat.com>
> 
> ACK. Good catch.

Thanks, pushed.

-- 
Chris Lalancette

Re: [PATCH] Make sure to quote the exec line in the initscript.

Posted by Michal Fojtik <mf...@redhat.com>.
On Sep 13, 2011, at 9:46 PM, Chris Lalancette wrote:

> This is so we put deltacloudd into the background, *not* daemon.
> 
> Signed-off-by: Chris Lalancette <cl...@redhat.com>

ACK. Good catch.

 -- Michal

> ---
> server/support/fedora/deltacloud-core |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/server/support/fedora/deltacloud-core b/server/support/fedora/deltacloud-core
> index 9dbb900..a0e5fe7 100755
> --- a/server/support/fedora/deltacloud-core
> +++ b/server/support/fedora/deltacloud-core
> @@ -44,7 +44,7 @@ start() {
>     [ -x $exec ] || exit 5
> 
>     echo -n $"Starting $prog: "
> -    daemon --user nobody $exec -i $DRIVER -e $API_ENV --port $PORT >> $LOGFILE 2>&1 &
> +    daemon --user nobody "$exec -i $DRIVER -e $API_ENV --port $PORT >> $LOGFILE 2>&1 &"
>     retval=$?
> 
>     if [ $retval -eq 0 ] && touch $lockfile ; then
> -- 
> 1.7.4.4
> 

----------------------------------------------------------------------
Michal Fojtik, Senior Software Engineer, Red Hat Czech
mfojtik@redhat.com
Deltacloud API: http://deltacloud.org