summaryrefslogtreecommitdiff
path: root/tests/ls
diff options
context:
space:
mode:
authorPádraig Brady <P@draigBrady.com>2013-12-07 15:00:06 +0000
committerPádraig Brady <P@draigBrady.com>2013-12-07 19:29:01 +0000
commit970b2ddea2d47f3167f4166e646d414f235f04b1 (patch)
tree0195a56c9e6711404706f76e761b10d8f2f9c68c /tests/ls
parent47a977ca99ad90f6fbe2efe84a17ac382a84ebf1 (diff)
downloadcoreutils-970b2ddea2d47f3167f4166e646d414f235f04b1.tar.xz
tests: fix false failure on platforms using newfstatat
* tests/ls/stat-free-color.sh: Add newfstatat to the list of syscalls to trace. Also add all "stat" syscalls to the list of syscalls that we verify that strace supports. Also only create a single dangling symlink to check, since we already only check for a single "stat" call. Fixes http://bugs.gnu.org/16075 seen on AArch64
Diffstat (limited to 'tests/ls')
-rwxr-xr-xtests/ls/stat-free-color.sh21
1 files changed, 13 insertions, 8 deletions
diff --git a/tests/ls/stat-free-color.sh b/tests/ls/stat-free-color.sh
index 3aacf961c..5fd5bea37 100755
--- a/tests/ls/stat-free-color.sh
+++ b/tests/ls/stat-free-color.sh
@@ -18,12 +18,16 @@
. "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
print_ver_ ls
-require_strace_ stat
+
+# Note this list of _file name_ stat functions must be
+# as cross platform as possible and so doesn't include
+# fstatat64 as that's not available on aarch64 for example.
+stats='stat,lstat,stat64,lstat64,newfstatat'
+
+require_strace_ $stats
require_dirent_d_type_
-for i in 1 2 3; do
- ln -s nowhere dangle-$i || framework_failure_
-done
+ln -s nowhere dangle || framework_failure_
# Disable enough features via LS_COLORS so that ls --color
# can do its job without calling stat (other than the obligatory
@@ -53,17 +57,18 @@ eval $(dircolors -b color-without-stat)
# To avoid counting those, first get a baseline count by running
# ls with only the --help option. Then, compare that with the
# invocation under test.
-strace -o log-help -e stat,lstat,stat64,lstat64 ls --help >/dev/null || fail=1
+strace -o log-help -e $stats ls --help >/dev/null || fail=1
n_lines_help=$(wc -l < log-help)
-strace -o log -e stat,lstat,stat64,lstat64 ls --color=always . || fail=1
+strace -o log -e $stats ls --color=always . || fail=1
n_lines=$(wc -l < log)
n_stat=$(expr $n_lines - $n_lines_help)
-# Expect one or two stat calls.
+# Expect one stat call.
case $n_stat in
- 1) ;;
+ 0) skip_ 'No stat calls recognized on this platform' ;;
+ 1) ;; # Corresponding to stat(".")
*) fail=1; head -n30 log* ;;
esac