diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2021-03-15 10:49:47 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2021-03-15 11:13:24 +0100 |
commit | e5e957e3308a6a03503b6ded290dc975494204e5 (patch) | |
tree | 10d188fe1bc5378a6665d34778d004a831ffe95c /scripts | |
parent | 8a92f7070941172e1671cf6b0fd1296776daefb7 (diff) | |
download | lvm2-e5e957e3308a6a03503b6ded290dc975494204e5.tar.gz |
shellcheck: avoid assign warnings
Shellcheck seems to be happier when assigning variables with quotes.
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/fsadm.sh | 54 | ||||
-rw-r--r-- | scripts/lvmdump.sh.in | 36 |
2 files changed, 45 insertions, 45 deletions
diff --git a/scripts/fsadm.sh b/scripts/fsadm.sh index 4f59cee13..d98833c3a 100755 --- a/scripts/fsadm.sh +++ b/scripts/fsadm.sh @@ -31,34 +31,34 @@ set -euE -o pipefail -TOOL=fsadm +TOOL="fsadm" _SAVEPATH=$PATH PATH="/sbin:/usr/sbin:/bin:/usr/sbin:$PATH" # utilities -TUNE_EXT=tune2fs -RESIZE_EXT=resize2fs -TUNE_REISER=reiserfstune -RESIZE_REISER=resize_reiserfs -TUNE_XFS=xfs_info -RESIZE_XFS=xfs_growfs - -MOUNT=mount -UMOUNT=umount -MKDIR=mkdir -RMDIR=rmdir -BLOCKDEV=blockdev -BLKID=blkid -DATE=date -GREP=grep -READLINK=readlink +TUNE_EXT="tune2fs" +RESIZE_EXT="resize2fs" +TUNE_REISER="reiserfstune" +RESIZE_REISER="resize_reiserfs" +TUNE_XFS="xfs_info" +RESIZE_XFS="xfs_growfs" + +MOUNT="mount" +UMOUNT="umount" +MKDIR="mkdir" +RMDIR="rmdir" +BLOCKDEV="blockdev" +BLKID="blkid" +DATE="date" +GREP="grep" +READLINK="readlink" READLINK_E="-e" -FSCK=fsck -XFS_CHECK=xfs_check +FSCK="fsck" +XFS_CHECK="xfs_check" # XFS_REPAIR -n is used when XFS_CHECK is not found -XFS_REPAIR=xfs_repair -CRYPTSETUP=cryptsetup +XFS_REPAIR="xfs_repair" +CRYPTSETUP="cryptsetup" # user may override lvm location by setting LVM_BINARY LVM=${LVM_BINARY:-lvm} @@ -69,8 +69,8 @@ VERB= FORCE= EXTOFF=${_FSADM_EXTOFF:-0} DO_LVRESIZE=0 -FSTYPE=unknown -VOLUME=unknown +FSTYPE="unknown" +VOLUME="unknown" TEMPDIR="${TMPDIR:-/tmp}/${TOOL}_${RANDOM}$$/m" DM_DEV_DIR="${DM_DEV_DIR:-/dev}" BLOCKSIZE= @@ -321,10 +321,10 @@ detect_mounted_with_proc_self_mountinfo() { # device (which could have been renamed). # We need to visit every mount point and check it's major minor detect_mounted_with_proc_mounts() { - MOUNTED=$("$GREP" "^$VOLUME[ \\t]" "$PROCMOUNTS") + MOUNTED=$("$GREP" "^${VOLUME}[ \\t]" "$PROCMOUNTS") # for empty string try again with real volume name - test -z "$MOUNTED" && MOUNTED=$("$GREP" "^$RVOLUME[ \\t]" "$PROCMOUNTS") + test -z "$MOUNTED" && MOUNTED=$("$GREP" "^${RVOLUME}[ \\t]" "$PROCMOUNTS") MOUNTDEV=$(echo -n -e "${MOUNTED%% *}") # cut device name prefix and trim everything past mountpoint @@ -335,8 +335,8 @@ detect_mounted_with_proc_mounts() { # for systems with different device names - check also mount output if test -z "$MOUNTED" ; then # will not work with spaces in paths - MOUNTED=$(LC_ALL=C "$MOUNT" | "$GREP" "^$VOLUME[ \\t]") - test -z "$MOUNTED" && MOUNTED=$(LC_ALL=C "$MOUNT" | "$GREP" "^$RVOLUME[ \\t]") + MOUNTED=$(LC_ALL=C "$MOUNT" | "$GREP" "^${VOLUME}[ \\t]") + test -z "$MOUNTED" && MOUNTED=$(LC_ALL=C "$MOUNT" | "$GREP" "^${RVOLUME}[ \\t]") MOUNTDEV=${MOUNTED%% on *} MOUNTED=${MOUNTED##* on } MOUNTED=${MOUNTED% type *} # allow type in the mount name diff --git a/scripts/lvmdump.sh.in b/scripts/lvmdump.sh.in index 24e395661..8507151d8 100644 --- a/scripts/lvmdump.sh.in +++ b/scripts/lvmdump.sh.in @@ -18,24 +18,24 @@ # following external commands are used throughout the script # echo and test are internal in bash at least -MKDIR=mkdir # need -p -TAR=tar # need czf -RM=rm # need -rf -CP=cp -TAIL=tail # we need -n -LS=ls # need -la -PS=ps # need alx -SED=sed -DD=dd -CUT=cut -GREP=grep -DATE=date -BASENAME=basename -UDEVADM=udevadm -UNAME=uname -TR=tr -SOCAT=socat # either socat or nc is needed for dumping lvmetad state -NC=nc +MKDIR="mkdir" # need -p +TAR="tar" # need czf +RM="rm" # need -rf +CP="cp" +TAIL="tail" # we need -n +LS="ls" # need -la +PS="ps" # need alx +SED="sed" +DD="dd" +CUT="cut" +GREP="grep" +DATE="date" +BASENAME="basename" +UDEVADM="udevadm" +UNAME="uname" +TR="tr" +SOCAT="socat" # either socat or nc is needed for dumping lvmetad state +NC="nc" if test "@ODIRECT@" = yes; then DDFLAGS='iflag=direct oflag=direct' |