diff options
author | 2023-01-24 03:04:39 +0000 | |
---|---|---|
committer | 2023-01-24 03:05:32 +0000 | |
commit | cbe23e2638435ddeced39979099fceb77fbb98fb (patch) | |
tree | 23c1049b3d3d8a3333d260651cb1718b499a542b | |
parent | install-xattr: fix chdir arg when OLDPWD is nulL (diff) | |
download | elfix-cbe23e2638435ddeced39979099fceb77fbb98fb.tar.gz elfix-cbe23e2638435ddeced39979099fceb77fbb98fb.tar.bz2 elfix-cbe23e2638435ddeced39979099fceb77fbb98fb.zip |
install-xattr: minor test fixups
- Add set -x like we did for fix-gnustack (87ca5c5bbe63ea3c9227fdd4a01ccdc1ce723323)
for easier debugging when something fails.
- Use Bash tests everywhere given we're already using Bash.
- Use portable Bash shebang.
- Fix shellcheck nits.
Signed-off-by: Sam James <sam@gentoo.org>
-rwxr-xr-x | misc/install-xattr/checkcopyattrs.sh | 52 |
1 files changed, 26 insertions, 26 deletions
diff --git a/misc/install-xattr/checkcopyattrs.sh b/misc/install-xattr/checkcopyattrs.sh index a9149d7..9d5dc13 100755 --- a/misc/install-xattr/checkcopyattrs.sh +++ b/misc/install-xattr/checkcopyattrs.sh @@ -1,5 +1,5 @@ -#!/bin/bash -set -e +#!/usr/bin/env bash +set -ex touch a b c mkdir -p d e h @@ -13,41 +13,41 @@ setfattr -n user.pax.flags -v "r" c ./install-xattr b y ./install-xattr c z -[ "$(getfattr --only-values -n user.foo x)" == "bar" ] -[ "$(getfattr --only-values -n user.bas x)" == "x" ] -[ "$(getfattr --only-values -n user.pax.flags x)" == "mr" ] -[ "$(getfattr --only-values -n user.pax.flags y)" == "p" ] -[ "$(getfattr --only-values -n user.pax.flags z)" == "r" ] +[[ "$(getfattr --only-values -n user.foo x)" == "bar" ]] +[[ "$(getfattr --only-values -n user.bas x)" == "x" ]] +[[ "$(getfattr --only-values -n user.pax.flags x)" == "mr" ]] +[[ "$(getfattr --only-values -n user.pax.flags y)" == "p" ]] +[[ "$(getfattr --only-values -n user.pax.flags z)" == "r" ]] ./install-xattr a b c d -[ "$(getfattr --only-values -n user.foo d/a)" == "bar" ] -[ "$(getfattr --only-values -n user.bas d/a)" == "x" ] -[ "$(getfattr --only-values -n user.pax.flags d/a)" == "mr" ] -[ "$(getfattr --only-values -n user.pax.flags d/b)" == "p" ] -[ "$(getfattr --only-values -n user.pax.flags d/c)" == "r" ] +[[ "$(getfattr --only-values -n user.foo d/a)" == "bar" ]] +[[ "$(getfattr --only-values -n user.bas d/a)" == "x" ]] +[[ "$(getfattr --only-values -n user.pax.flags d/a)" == "mr" ]] +[[ "$(getfattr --only-values -n user.pax.flags d/b)" == "p" ]] +[[ "$(getfattr --only-values -n user.pax.flags d/c)" == "r" ]] # This tests if the src file was inside a directory # the correct dst location should be f/a. NOT f/d/a. ./install-xattr d/a h -[ -x h/a ] -[ ! -x h/d/a ] -[ "$(getfattr --only-values -n user.foo h/a)" == "bar" ] -[ "$(getfattr --only-values -n user.bas h/a)" == "x" ] +[[ -x h/a ]] +[[ ! -x h/d/a ]] +[[ "$(getfattr --only-values -n user.foo h/a)" == "bar" ]] +[[ "$(getfattr --only-values -n user.bas h/a)" == "x" ]] ./install-xattr -t e a b c -[ "$(getfattr --only-values -n user.foo e/a)" == "bar" ] -[ "$(getfattr --only-values -n user.bas e/a)" == "x" ] -[ "$(getfattr --only-values -n user.pax.flags e/a)" == "mr" ] -[ "$(getfattr --only-values -n user.pax.flags e/b)" == "p" ] -[ "$(getfattr --only-values -n user.pax.flags e/c)" == "r" ] +[[ "$(getfattr --only-values -n user.foo e/a)" == "bar" ]] +[[ "$(getfattr --only-values -n user.bas e/a)" == "x" ]] +[[ "$(getfattr --only-values -n user.pax.flags e/a)" == "mr" ]] +[[ "$(getfattr --only-values -n user.pax.flags e/b)" == "p" ]] +[[ "$(getfattr --only-values -n user.pax.flags e/c)" == "r" ]] ./install-xattr a -D f/a -[ "$(getfattr --only-values -n user.foo f/a)" == "bar" ] -[ "$(getfattr --only-values -n user.bas f/a)" == "x" ] -[ "$(getfattr --only-values -n user.pax.flags f/a)" == "mr" ] +[[ "$(getfattr --only-values -n user.foo f/a)" == "bar" ]] +[[ "$(getfattr --only-values -n user.bas f/a)" == "x" ]] +[[ "$(getfattr --only-values -n user.pax.flags f/a)" == "mr" ]] # Check that we can copy large files truncate -s2G a @@ -68,8 +68,8 @@ truncate -s2G a ./install-xattr -p a backup-a ./install-xattr -d g/g/g -./install-xattr -o $(id -u) a mode-a -./install-xattr -g $(id -g) a mode-a +./install-xattr -o "$(id -u)" a mode-a +./install-xattr -g "$(id -g)" a mode-a ./install-xattr -m 666 a mode-a # Let's abuse ourselves |