[NEW PATCH] BZ#678039 vdsmd: use bash assignment properly (via gerrit-bot)

Dan Kenigsberg danken at redhat.com
Mon Jun 27 21:56:31 UTC 2011


New patch submitted by Dan Kenigsberg (danken at redhat.com)

You can review this change at: http://gerrit.usersys.redhat.com/624

commit 3c4ae31a3b75346a191ac0b0533e3d7c1de29733
Author: Dan Kenigsberg <danken at redhat.com>
Date:   Mon Jun 27 22:54:45 2011 +0300

    BZ#678039 vdsmd: use bash assignment properly
    
    Bash is not perl.
    
    Change-Id: Ic217f56ac22e14649e1296cae4bff0d36f5a0e68

diff --git a/vdsm/vdsmd b/vdsm/vdsmd
index 488da4e..6b35e23 100755
--- a/vdsm/vdsmd
+++ b/vdsm/vdsmd
@@ -146,7 +146,7 @@ start_needed_srv() {
         then
             echo "Starting $srv..."
             /sbin/service $srv start
-            $ret_val=$?
+            ret_val=$?
             if [ $ret_val -ne 0 ]
             then
                 log_failure_msg "$prog: Dependent $srv failed to start"




More information about the vdsm-patches mailing list