Skip to content

Commit

Permalink
Pull in composer dependencies before running make tests
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
eldering committed Sep 17, 2024
1 parent 42e6d49 commit 7876c61
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/jobs/configure-checks/all.bats
Original file line number Diff line number Diff line change
Expand Up @@ -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 () {
Expand Down
18 changes: 13 additions & 5 deletions .github/jobs/configure-checks/setup_configure_image.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,16 +7,24 @@ 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 -y pkg-config make bats autoconf automake util-linux composer \
php-gd php-cli php-intl php-mbstring php-mysqlnd php-fpm \
php-xml php-zip
;;
*)
apt-get update; apt-get full-upgrade -y
apt-get install pkg-config make bats autoconf -y ;;
apt-get install -y pkg-config make bats autoconf composer \
php-fpm php-gd php-cli php-intl php-mbstring php-mysql \
php-curl php-json php-xml php-zip
;;
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
Expand Down

0 comments on commit 7876c61

Please sign in to comment.