From 62ef9d9be68905d19ed9984db5f40daaf09b8dd5 Mon Sep 17 00:00:00 2001 From: "Joshua (D) Drake" <136637981+ChronicallyJD@users.noreply.github.com> Date: Wed, 26 Aug 2026 11:00:19 -0600 Subject: [PATCH] CI: derive the workspace path instead of hardcoding the repo name The regression-test steps make the workspace traversable for the postgres user by chmod'ing each ancestor directory, and named the middle one literally: /home/runner/work/PL-php. Renaming the repository to plPHP moved the checkout to /home/runner/work/plPHP/plPHP, so that chmod now fails with "cannot access", and since the step runs under "bash -e" every matrix job and the ASAN job fail before a single test runs. Use $(dirname "$GITHUB_WORKSPACE") instead, which is the same directory whatever the repository is called. Co-Authored-By: Claude Opus 5 (1M context) --- .github/workflows/ci.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 72a34ff..4e1ce8c 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -89,7 +89,7 @@ jobs: run: | port=$(pg_lsclusters | awk '$1 == "${{ matrix.pg }}" && $2 == "main" { print $3 }') # let the postgres user reach and write the workspace - sudo chmod a+x /home/runner /home/runner/work /home/runner/work/PL-php + sudo chmod a+x /home/runner /home/runner/work "$(dirname "$GITHUB_WORKSPACE")" chmod -R a+rwX . sudo -u postgres env PGPORT=$port \ make PG_CONFIG=/usr/lib/postgresql/${{ matrix.pg }}/bin/pg_config installcheck @@ -178,7 +178,7 @@ jobs: - name: Run the regression tests under ASAN run: | port=$(pg_lsclusters | awk '$1 == 18 && $2 == "main" { print $3 }') - sudo chmod a+x /home/runner /home/runner/work /home/runner/work/PL-php + sudo chmod a+x /home/runner /home/runner/work "$(dirname "$GITHUB_WORKSPACE")" chmod -R a+rwX . # the cookbook test needs SimpleXML, unavailable with extensions off tests=$(grep '^REGRESS' Makefile | sed 's/REGRESS = //; s/ cookbook//')