Age | Commit message (Collapse) | Author |
|
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.
|
|
Using `xargs -i <cmd> {}` is just doing the default behavior of xargs,
but with extra steps that also don't work on macOS.
|
|
grep -E and -F are POSIX standard, and meets all our matching needs.
|
|
This can also be tripped when the change was simply not being committed,
avoid confusing the author by simply mentioning that.
|
|
|
|
Not sure how this didn't cause a build break when I originally
moved them in #6891. Better fix them now.
|
|
|
|
Problem:
- It is possible for a new test file to be added to the `AK/Tests`
directory without being added to the corresponding
`CMakeLists.txt`. This results in the tests not being run.
Solution:
- As part of CI linting, verify that all the `AK/Tests/Test*.cpp`
files are mentioned in the `CMakeLists.txt`.
|