Closes what was left of TODO.md sections 1, 2 and 3, and rewrites that
file to hold outstanding items only.
The API break gets a minor bump, because pre-1.0 the soname carries
MAJOR.MINOR and 0.1 and 0.2 are therefore different ABIs. Five
signatures changed and the ato* contract with them; UPGRADING.md is new
and lists every one, with the before/after for the cases the compiler
cannot warn about.
Section 3.1 is finished: reallocarray with the multiplication checked,
aligned_alloc and posix_memalign, asprintf/vasprintf, scanf/vscanf.
Four functions on that list are deliberately absent rather than missing
-- sprintf, strtok, setbuf and perror -- and TODO.md now says which and
why, so nobody adds them thinking they were forgotten.
Section 1.9, the cross-cutting tests:
tests/test_pool.c drives every failure path AKERR_MAX_ARRAY_ERROR
+ 10 times and checks the pool after each round,
because a wrapper that leaks a slot fails a
hundred calls later in unrelated code. It also
asserts that each error names the function and
file it was raised from, which is what catches a
FAIL that migrates into a helper during a
refactor: status right, message right, origin
quietly lying.
tests/negative/ two sources that must FAIL to compile, built with
-Werror and registered WILL_FAIL. AKERR_NOIGNORE
and the format attributes are enforced by the
compiler and by nothing else; drop either and
every ordinary test still passes.
Thread safety is answered rather than tested: the library is not
thread-safe and cannot be made so from here, because libakerror's error
pool is an unlocked process-global array. README.md says so plainly and
TODO.md carries it as the item blocking any future pthread wrappers.
Doxygen is configured and gated. All 147 public functions have @brief,
a @param each, @throws per status and @return; EXTRACT_ALL is off and
WARN_NO_PARAMDOC on, so `cmake --build build --target docs` fails on an
undocumented entity. It ran to 0 warnings. The Doxyfile carries no
version -- cmake/RunDoxygen.cmake feeds PROJECT_NUMBER in from
project(), so that stays the one place a version is written.
CI now builds against the submodule it pins instead of also installing
libakerror@main and never linking it, adds -Werror, and gains a
sanitizer job. The pre-push hook matches, and runs the docs check too.
Coverage: 99.5% of lines (1643/1651), 100% of functions (147/147). The
eight uncovered lines are each uncovered on purpose and TODO.md says
which and why.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
120 lines
4.1 KiB
Bash
Executable File
120 lines
4.1 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
#
|
|
# pre-push: build libakstdlib and run the test harnesses before anything leaves
|
|
# this machine.
|
|
#
|
|
# Install (once per clone):
|
|
#
|
|
# git config core.hooksPath .githooks
|
|
#
|
|
# Runs by default, a few seconds all in:
|
|
#
|
|
# * default build + ctest (with -Werror, as CI has it)
|
|
# * ASan/UBSan build + ctest
|
|
# * doxygen -- fails on an undocumented public function
|
|
#
|
|
# Opt in to the slow harness (~25 minutes -- it rebuilds and re-runs the whole
|
|
# suite once per mutant):
|
|
#
|
|
# AKSL_HOOK_MUTATION=1 git push
|
|
#
|
|
# Bypass everything (git's own escape hatch):
|
|
#
|
|
# git push --no-verify
|
|
#
|
|
# Builds go under .git/aksl-prepush so the hook never disturbs whatever is in
|
|
# your own build/ directory. Override with AKSL_HOOK_BUILD_DIR if you want them
|
|
# somewhere else.
|
|
|
|
set -u
|
|
|
|
# Keep in sync with the --threshold in .gitea/workflows/ci.yaml, so a push that
|
|
# would fail CI fails here first.
|
|
MUTATION_THRESHOLD="${AKSL_MUTATION_THRESHOLD:-80}"
|
|
|
|
ZERO_SHA=0000000000000000000000000000000000000000
|
|
|
|
root=$(git rev-parse --show-toplevel) || exit 1
|
|
cd "$root" || exit 1
|
|
|
|
# git feeds us one line per ref being pushed. A push that only *deletes* refs
|
|
# has no commits to test, so there is nothing to do. An empty stdin (nothing to
|
|
# push) lands here too, which is equally fine to skip.
|
|
has_updates=0
|
|
while read -r _local_ref local_sha _remote_ref _remote_sha; do
|
|
if [ "$local_sha" != "$ZERO_SHA" ]; then
|
|
has_updates=1
|
|
fi
|
|
done
|
|
if [ "$has_updates" -eq 0 ]; then
|
|
exit 0
|
|
fi
|
|
|
|
if [ ! -f deps/libakerror/CMakeLists.txt ]; then
|
|
echo "pre-push: deps/libakerror is empty. Run:" >&2
|
|
echo " git submodule update --init --recursive" >&2
|
|
exit 1
|
|
fi
|
|
|
|
builddir="${AKSL_HOOK_BUILD_DIR:-$(git rev-parse --git-dir)/aksl-prepush}"
|
|
mkdir -p "$builddir" || exit 1
|
|
logfile="$builddir/last.log"
|
|
|
|
# Run a step quietly; on failure, dump what it said and abort the push.
|
|
run() {
|
|
if ! "$@" > "$logfile" 2>&1; then
|
|
echo >&2
|
|
echo "pre-push: FAILED: $*" >&2
|
|
echo "---------------------------------------------------------------" >&2
|
|
cat "$logfile" >&2
|
|
echo "---------------------------------------------------------------" >&2
|
|
echo "pre-push: push aborted. Use 'git push --no-verify' to override." >&2
|
|
exit 1
|
|
fi
|
|
}
|
|
|
|
# -DAKSL_WERROR=ON here and not in a plain local build: a warning that stops you
|
|
# mid-thought teaches people to turn warnings off, but a warning that reaches
|
|
# main is one nobody will look at again. This is the last point where it is
|
|
# still cheap to fix.
|
|
echo "pre-push: default build + ctest"
|
|
run cmake -S . -B "$builddir/default" -DAKSL_WERROR=ON
|
|
run cmake --build "$builddir/default"
|
|
run ctest --test-dir "$builddir/default" --output-on-failure
|
|
|
|
echo "pre-push: sanitizer build + ctest"
|
|
run cmake -S . -B "$builddir/asan" -DAKSL_SANITIZE=ON -DAKSL_WERROR=ON
|
|
run cmake --build "$builddir/asan"
|
|
run ctest --test-dir "$builddir/asan" --output-on-failure
|
|
|
|
# Doxygen runs with WARN_IF_UNDOCUMENTED and WARN_NO_PARAMDOC on and EXTRACT_ALL
|
|
# off, so the target fails when a public function, parameter or return value has
|
|
# nobody describing it. Skipped where doxygen is not installed rather than
|
|
# failing: it is a documentation check, not a build dependency.
|
|
if command -v doxygen > /dev/null 2>&1; then
|
|
echo "pre-push: doxygen"
|
|
run cmake --build "$builddir/default" --target docs
|
|
else
|
|
echo "pre-push: doxygen not installed, skipping the documentation check"
|
|
fi
|
|
|
|
if [ "${AKSL_HOOK_MUTATION:-0}" = "1" ]; then
|
|
echo "pre-push: mutation testing, threshold ${MUTATION_THRESHOLD}% (this takes a while)"
|
|
# Deliberately not wrapped in run(): this one is slow enough that you want
|
|
# to watch it make progress.
|
|
if ! python3 scripts/mutation_test.py \
|
|
--target src/stdlib.c \
|
|
--target src/string.c \
|
|
--target src/stream.c \
|
|
--target src/collections.c \
|
|
--threshold "$MUTATION_THRESHOLD"; then
|
|
echo >&2
|
|
echo "pre-push: mutation score below ${MUTATION_THRESHOLD}%. Push aborted." >&2
|
|
echo "pre-push: use 'git push --no-verify' to override." >&2
|
|
exit 1
|
|
fi
|
|
fi
|
|
|
|
echo "pre-push: OK"
|
|
exit 0
|