diff options
| author | Jim Meyering <meyering@fb.com> | 2016-11-08 19:57:41 -0600 |
|---|---|---|
| committer | Jim Meyering <meyering@fb.com> | 2016-11-13 17:09:57 -0800 |
| commit | 4d4d8bccc341941404dc92d0adc057e67014e2ea (patch) | |
| tree | faf04706ebd51a3c7e369110fb23bf80b83cef86 /tests/ls | |
| parent | eba871cd3237e8b7dcd9552f544b365934767849 (diff) | |
| download | coreutils-4d4d8bccc341941404dc92d0adc057e67014e2ea.tar.xz | |
tests: use "returns_" rather than explicit comparison with "$?"
The previous "returns_"-using change failed to convert many
uses of "$?". Convert all but two of the remaining ones.
* tests/ls/stat-vs-dirent.sh: Likewise.
* tests/misc/head-write-error.sh: Likewise.
* tests/misc/nice.sh: Likewise.
* tests/misc/nohup.sh: Likewise.
* tests/misc/stdbuf.sh: Likewise.
* tests/misc/sync.sh: Likewise.
* tests/tail-2/pid.sh: Likewise.
* tests/tail-2/wait.sh: Likewise.
Thanks to Bernhard Volker for spotting this.
Diffstat (limited to 'tests/ls')
| -rwxr-xr-x | tests/ls/stat-vs-dirent.sh | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/ls/stat-vs-dirent.sh b/tests/ls/stat-vs-dirent.sh index b9f3cd1c9..fa2087f53 100755 --- a/tests/ls/stat-vs-dirent.sh +++ b/tests/ls/stat-vs-dirent.sh @@ -23,8 +23,7 @@ print_ver_ ls root_dev_ino=$(stat --format=%d-%i /) t=$(pwd) while :; do - ls -i1 "$t" > tmp - if test $? = 0; then + if ls -i1 "$t" > tmp; then # Extract the inode number from the first line of output from ls -i1. # This value comes from dirent.d_ino, on systems with d_ino support. d_ino=$(sed -n '1s/^ *\([0-9][0-9]*\) .*/\1/p;q' tmp) |