Change in vdsm[master]: Passing "$*" to shell func with quotation marks passes all v...

ybronhei at redhat.com ybronhei at redhat.com
Mon Feb 17 15:39:45 UTC 2014


Yaniv Bronhaim has uploaded a new change for review.

Change subject: Passing "$*" to shell func with quotation marks passes all vars in one
......................................................................

Passing "$*" to shell func with quotation marks passes all vars in one

Instead of passing the variables as passed to test_conflicting_conf
to the specific test func

Change-Id: Iee8d5f3e9c01bd879a1e41159fa798eefd2c51ca
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M lib/vdsm/tool/libvirt_configure.sh.in
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/80/24580/1

diff --git a/lib/vdsm/tool/libvirt_configure.sh.in b/lib/vdsm/tool/libvirt_configure.sh.in
index 89f1534..49e0e1e 100755
--- a/lib/vdsm/tool/libvirt_configure.sh.in
+++ b/lib/vdsm/tool/libvirt_configure.sh.in
@@ -52,7 +52,7 @@
 }
 
 test_conflicting_conf() {
-    test_ssl_conflict_conf "$*"
+    test_ssl_conflict_conf $*
     # add here additional conf checks
 }
 


-- 
To view, visit http://gerrit.ovirt.org/24580
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iee8d5f3e9c01bd879a1e41159fa798eefd2c51ca
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list