Files
Nico Weber 4aae1fda2f Meta+CI: Check jbig2 json files
This adds a check that makes sure that:

1. All json files below Tests/LibGfx/test-inputs/jbig2/json/ can
   be compiled by `jbig2-from-json`

2. The output of `jbig2-from-json` is identical to the corresponding
   jbig2 file in Tests/LibGfx/test-inputs/jbig2/

This makes sure we don't forget to update the checked-in jbig2 files
(e.g. with Tests/LibGfx/test-inputs/jbig2/json/compile.sh) after
changing json files, and that we don't forget to update the json files
after updating jbig2-from-json.

The implementation follows what other lint steps that need lagom
binaries do: There's a new check-jbig2-json.sh script that does
nothing if the binary does not yet exist, and it's called from both
lint-ci.sh and the GitHub actions lagom workflow. People who have
built jbig2-from-json locally and run lint-ci.sh as a precommit
hook will run the script that way, while the lint step on CI will
skip this script and then run it later after lagom has been built.
2025-10-24 19:11:13 -04:00

81 lines
2.2 KiB
Bash
Executable File

#!/usr/bin/env bash
set -e
script_path=$(cd -P -- "$(dirname -- "$0")" && pwd -P)
cd "${script_path}/.." || exit 1
ports=true
if [ "$1" == "--no-ports" ]; then
ports=false
shift
fi
RED='\033[0;31m'
GREEN='\033[0;32m'
NC='\033[0m' # No Color
FAILURES=0
set +e
for cmd in \
Meta/check-ak-test-files.sh \
Meta/check-debug-flags.sh \
Meta/check-emoji.py \
Meta/check-idl-files.py \
Meta/check-jbig2-json.sh \
Meta/check-markdown.sh \
Meta/check-newlines-at-eof.py \
Meta/check-png-sizes.sh \
Meta/check-style.py \
Meta/lint-executable-resources.sh \
Meta/lint-gml-format.sh \
Meta/lint-gn.sh \
Meta/lint-keymaps.py \
Meta/lint-prettier.sh \
Meta/lint-python.sh \
Meta/lint-shell-scripts.sh; do
if "${cmd}" "$@"; then
echo -e "[${GREEN}OK${NC}]: ${cmd}"
else
echo -e "[${RED}FAIL${NC}]: ${cmd}"
((FAILURES+=1))
fi
done
if [ -x ./Build/lagom/bin/IPCMagicLinter ]; then
if { git ls-files '*.ipc' | xargs ./Build/lagom/bin/IPCMagicLinter; }; then
echo -e "[${GREEN}OK${NC}]: IPCMagicLinter (in Meta/lint-ci.sh)"
else
echo -e "[${RED}FAIL${NC}]: IPCMagicLinter (in Meta/lint-ci.sh)"
((FAILURES+=1))
fi
else
echo -e "[${GREEN}SKIP${NC}]: IPCMagicLinter (in Meta/lint-ci.sh)"
fi
if Meta/lint-clang-format.sh --overwrite-inplace "$@" && git diff --exit-code; then
echo -e "[${GREEN}OK${NC}]: Meta/lint-clang-format.sh"
else
echo -e "[${RED}FAIL${NC}]: Meta/lint-clang-format.sh"
((FAILURES+=1))
fi
# lint-ports.py is handled separately as it scans all Ports/ all the time.
# This is fine when running lint-ci.sh from the PR validation workflow.
# However when running from the pre-commit workflow it takes an excessive
# amount of time. This condition allows the pre-commit program to detect
# when Ports/ files have changed and only invoke lint-ports.py when needed.
#
if [ "$ports" = true ]; then
if Meta/lint-ports.py; then
echo -e "[${GREEN}OK${NC}]: Meta/lint-ports.py"
else
echo -e "[${RED}FAIL${NC}]: Meta/lint-ports.py"
((FAILURES+=1))
fi
fi
exit "${FAILURES}"