From eb5d7730e1b3b1bddecb80be37e5a4c938183f61 Mon Sep 17 00:00:00 2001 From: Frantisek Sumsal Date: Mon, 7 Nov 2022 11:55:29 +0100 Subject: [PATCH] test: don't ignore non-existent paths in inst_recursive() The process substitution in the while loop hides errors raised by the find utility, which might (and did), in turn, hide errors in test setup. --- test/test-functions | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/test-functions b/test/test-functions index 16d9da637b..80ce383e64 100644 --- a/test/test-functions +++ b/test/test-functions @@ -2773,6 +2773,10 @@ inst_recursive() { local p item for p in "$@"; do + # Make sure the source exists, as the process substitution below + # suppresses errors + stat "$p" >/dev/null || return 1 + while read -r item; do if [[ -d "$item" ]]; then inst_dir "$item"