diff options
author | Anoop C S <anoopcs@redhat.com> | 2017-04-27 14:34:08 +0530 |
---|---|---|
committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-04-27 16:11:21 +0000 |
commit | ec7a0c7eefacc68532bdc909f05a01eca8b3139d (patch) | |
tree | bef182065b2f75a87276978b234fa379eec57687 | |
parent | 0e50c4b3ea734456c14e2d7a578463999bd332c3 (diff) |
extras/hooks: Use double quotes while using [..]
This avoids the following warning when first operand is null
[: =: unary operator expected
Change-Id: I5439d8f60a6d9e30e6ba04c16c3de2096a87c38f
BUG: 1446126
Signed-off-by: Anoop C S <anoopcs@redhat.com>
Reviewed-on: https://review.gluster.org/17127
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Jeff Darcy <jeff@pl.atyp.us>
-rwxr-xr-x | extras/hook-scripts/start/post/S30samba-start.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extras/hook-scripts/start/post/S30samba-start.sh b/extras/hook-scripts/start/post/S30samba-start.sh index 752eca650d3..3e0f2573b60 100755 --- a/extras/hook-scripts/start/post/S30samba-start.sh +++ b/extras/hook-scripts/start/post/S30samba-start.sh @@ -102,7 +102,7 @@ function get_smb () { } parse_args $@ -if [ $(get_smb "$VOL") = "disable" ]; then +if [ "$(get_smb "$VOL")" = "disable" ]; then exit 0 fi |