mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2024-11-24 00:20:21 +00:00
570814a31e
This change makes all the pre-commit CI scripts runnable under Bash 3.2, by replacing “mapfile” invocations in them code that first explicitly creates an array, and then uses a while loop to populate the array. Otherwise, without this change, the scripts all fail to run under Bash 3.2 — due to lack of support for “mapfile”. Fixes https://github.com/LadybirdBrowser/ladybird/issues/283 This also drops bash from the list of homebrew dependencies in the build instructions — because with this change, homebrew bash (v4) is no longer needed; things will now work with the Apple-provided bash (v3.2)
33 lines
708 B
Bash
Executable file
33 lines
708 B
Bash
Executable file
#!/usr/bin/env bash
|
|
|
|
set -e
|
|
|
|
script_path=$(cd -P -- "$(dirname -- "$0")" && pwd -P)
|
|
cd "${script_path}/.." || exit 1
|
|
|
|
if [ "$#" -eq "0" ]; then
|
|
files=()
|
|
while IFS= read -r file; do
|
|
files+=("$file")
|
|
done < <(
|
|
git ls-files '*.py'
|
|
)
|
|
else
|
|
files=()
|
|
for file in "$@"; do
|
|
if [[ "${file}" == *".py" ]]; then
|
|
files+=("${file}")
|
|
fi
|
|
done
|
|
fi
|
|
|
|
if (( ${#files[@]} )); then
|
|
if ! command -v flake8 >/dev/null 2>&1 ; then
|
|
echo "flake8 is not available, but python files need linting! Either skip this script, or install flake8."
|
|
exit 1
|
|
fi
|
|
|
|
flake8 "${files[@]}" --max-line-length=120
|
|
else
|
|
echo "No py files to check."
|
|
fi
|