Yaniv Bronhaim has uploaded a new change for review.
Change subject: Removing unnecessary subshell call
......................................................................
Removing unnecessary subshell call
Change-Id: If938a322836a82090671483f47a6a0c49878556a
Signed-off-by: Yaniv Bronhaim <ybron...@redhat.com>
---
M init/sysvinit/vdsmd.init.in
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/61/20061/1
diff --git a/init/sysvinit/vdsmd.init.in b/init/sysvinit/vdsmd.init.in
index efd9c35..e10cca0 100755
--- a/init/sysvinit/vdsmd.init.in
+++ b/init/sysvinit/vdsmd.init.in
@@ -68,8 +68,8 @@
initctl stop "${srv}" || : # stop fails when already down
initctl status "${srv}" | grep -q stop/waiting
elif [ -x "/etc/init.d/${srv}" ]; then
- (! service "${srv}" status >/dev/null 2>&1) ||
- service "${srv}" stop
+ ! service "${srv}" status >/dev/null 2>&1 ||
+ service "${srv}" stop
else
true
fi
--
To view, visit http://gerrit.ovirt.org/20061
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If938a322836a82090671483f47a6a0c49878556a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
