diff options
author | Ben Wiederhake <BenWiederhake.GitHub@gmx.de> | 2022-09-12 21:02:11 +0200 |
---|---|---|
committer | Brian Gianforcaro <b.gianfo@gmail.com> | 2022-09-13 08:29:09 +0000 |
commit | 616b3dc718f82e23d562249e31090c9b95dbe26c (patch) | |
tree | 8ea50189e60f9ff89011c2075de63148a88619ef /Meta/check-ak-test-files.sh | |
parent | 1d846e559150e1ec90486760a8e09fced24b1d67 (diff) | |
download | serenity-616b3dc718f82e23d562249e31090c9b95dbe26c.zip |
Meta: Fix check for AK test inclusion
basename from GNU coreutils 8.32 (the default on Debian Bullseye, as
bash 5.2.0 does not include basename as a built-in command) does not
accept multiple arguments. This will cause the command to fail with no
output, and the error code not being propagated for some reason. This
means that the loop never gets executed, and thus the check never
actually does anything. This commit fixes that behavior by calling
'basename' multiple times.
Diffstat (limited to 'Meta/check-ak-test-files.sh')
-rwxr-xr-x | Meta/check-ak-test-files.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Meta/check-ak-test-files.sh b/Meta/check-ak-test-files.sh index 1a4b1b262a..564c34ceb9 100755 --- a/Meta/check-ak-test-files.sh +++ b/Meta/check-ak-test-files.sh @@ -14,7 +14,7 @@ while IFS= read -r FILENAME; do MISSING_FILES=y fi done < <( - git ls-files 'Tests/AK/Test*.cpp' | xargs basename + git ls-files 'Tests/AK/Test*.cpp' | xargs -n1 basename ) if [ "n" != "${MISSING_FILES}" ] ; then |