diff options
author | maximilian attems <maks@debian.org> | 2009-04-03 14:48:44 +0200 |
---|---|---|
committer | maximilian attems <maks@debian.org> | 2009-04-03 14:52:20 +0200 |
commit | d703b8ca26ac5230d7ffeae1891334791d8dab31 (patch) | |
tree | e16d11c8d6291b789269d510629e09108973296e /scripts | |
parent | e939f98f684326ddc06d54fe0379c552fb94010f (diff) | |
download | initramfs-tools-d703b8ca26ac5230d7ffeae1891334791d8dab31.tar.gz initramfs-tools-d703b8ca26ac5230d7ffeae1891334791d8dab31.zip |
cleanup LABEL handling code
make it more concise:
* use POSIX ${parameter:+word}
* replace *[/]*) by just */*)
* test exit code of command -v rather then running test -x on it
while we are at it fix another command -v usage in scripts/functions.
Reviewed-by: Colin Watson <cjwatson@ubuntu.com>
Signed-off-by: maximilian attems <maks@debian.org>
Diffstat (limited to 'scripts')
-rw-r--r-- | scripts/functions | 2 | ||||
-rwxr-xr-x | scripts/local-premount/resume | 10 |
2 files changed, 4 insertions, 8 deletions
diff --git a/scripts/functions b/scripts/functions index b813529..77de8f3 100644 --- a/scripts/functions +++ b/scripts/functions @@ -304,6 +304,6 @@ configure_networking() # Wait for queued kernel/udev events wait_for_udev() { - [ -x "$(command -v udevadm)" ] || return 0 + command -v udevadm >/dev/null 2>&1 || return 0 udevadm settle ${1:+--timeout=$1} } diff --git a/scripts/local-premount/resume b/scripts/local-premount/resume index b2b0e1d..11acfc7 100755 --- a/scripts/local-premount/resume +++ b/scripts/local-premount/resume @@ -25,8 +25,8 @@ LABEL=*) # support any / in LABEL= path (escape to \x2f) case "${resume}" in - *[/]*) - if [ -x "$(command -v sed)" ]; then + */*) + if command -v sed >/dev/null 2>&1; then resume="$(echo ${resume} | sed 's,/,\\x2f,g')" else if [ "${resume}" != "${resume#/}" ]; then @@ -38,11 +38,7 @@ LABEL=*) IFS='/' newresume= for s in $resume; do - if [ -z "${newresume}" ]; then - newresume="${s}" - else - newresume="${newresume}\\x2f${s}" - fi + newresume="${newresume:+${newresume}\\x2f}${s}" done unset IFS resume="${newresume}" |