From 7a4143a16724525000a0b98671313088e83f2296 Mon Sep 17 00:00:00 2001 From: Jaap Eldering Date: Tue, 17 Sep 2024 13:58:47 +0200 Subject: [PATCH] Pull in composer dependencies before running make tests This is because Makefiles implicitly depend on a distributed tarball (or initialized maintainer/inplace install). Running `make dist` would be a bit too much, so take this shortcut. --- .github/jobs/configure-checks/all.bats | 2 +- .../jobs/configure-checks/setup_configure_image.sh | 12 +++++++----- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/.github/jobs/configure-checks/all.bats b/.github/jobs/configure-checks/all.bats index 207650895fe..e00cd656e66 100755 --- a/.github/jobs/configure-checks/all.bats +++ b/.github/jobs/configure-checks/all.bats @@ -37,7 +37,7 @@ setup() { if [ "$distro_id" = "ID=fedora" ]; then repo-install httpd fi - repo-install gcc g++ libcgroup-dev composer + repo-install gcc g++ libcgroup-dev } run_configure () { diff --git a/.github/jobs/configure-checks/setup_configure_image.sh b/.github/jobs/configure-checks/setup_configure_image.sh index 6be3f188efc..a8991e443db 100755 --- a/.github/jobs/configure-checks/setup_configure_image.sh +++ b/.github/jobs/configure-checks/setup_configure_image.sh @@ -7,16 +7,18 @@ distro_id=$(grep "^ID=" /etc/os-release) # Install everything for configure and testing case $distro_id in "ID=fedora") - dnf install pkg-config make bats autoconf automake util-linux -y ;; + dnf install pkg-config make bats autoconf automake util-linux composer -y ;; *) apt-get update; apt-get full-upgrade -y - apt-get install pkg-config make bats autoconf -y ;; + apt-get install pkg-config make bats autoconf composer -y ;; esac -# Build the configure file -make configure +# Start from a configured, distribution-ready source tree. Ideally, +# we'd like to call `make dist` but that depends on LaTeX for building +# the documentation, so take a shortcut. +make configure composer-dependencies -# Install extra assert statements for bots +# Install extra assert statements for bats cp submit/assert.bash .github/jobs/configure-checks/ # Run the configure tests for this usecase