Corrected build script for Linux.

MartinKoeditz · MartinKoeditz · commit 23aca98bfb4d · 2026-03-30T15:48:03.000+02:00
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
+php-firebird-build/
 .vscode
 .deps
 .libs
diff --git a/build_scripts/README.md b/build_scripts/README.md
@@ -1,4 +1,4 @@
-# Scripts building php-firebird extension on Windows
+# Scripts building php-firebird extension (Windows + Linux)
 
 ## How it works
 
@@ -21,3 +21,46 @@ Make sure ``git`` is in you PATH
     PFB_SOURCE_DIR=D:\php-firebird\ then your source should reside in D:\php-firebird\php-firebird\
     ``
 6. cd into php-sdk and from there run ``<path_to>\php-fb-build-all.bat`` to build for all PHP versions or run ``php-fb-build.bat 7.4 vc15`` to build for particular version.
+
+---
+
+# Build on Linux
+
+Linux builds are handled by `php-fb-build-linux.sh`. It clones the extension source into a local build folder and builds one `.so` per configured PHP version.
+
+## Prerequisites
+
+- A working C/C++ build toolchain: `build-essential`, `autoconf`, `automake`, `libtool`, `pkg-config`, `g++`
+- Firebird client headers + library (`fbclient`) installed
+    - Default script paths assume a Firebird install under `/opt/firebird/`:
+        - headers: `/opt/firebird/include`
+        - library: `/opt/firebird/lib`
+- For each PHP version you want to build, the corresponding packages must exist (Debian/Ubuntu naming):
+    - `phpX.Y-dev`, `phpX.Y-cli`, `phpX.Y-common`
+
+Note: the script will try to install missing PHP packages via `sudo apt-get`. On non-Debian-based distros, install the equivalents manually and make sure `phpX.Y`, `phpizeX.Y` and `php-configX.Y` exist in `/usr/bin/`.
+
+## Configure
+
+Open `php-fb-build-linux.sh` and adjust if needed:
+
+- `FIREBIRD_INCLUDE_DIR` (default: `/opt/firebird/include`)
+- `PHP_VERSIONS` (default: `("7.4" "8.0" "8.1" "8.2" "8.3" "8.4")`)
+- `INSTALL_DIR` (default: `../ext` relative to the repo root)
+
+## Run
+
+From the repository root:
+
+```bash
+./build_scripts/php-fb-build-linux.sh
+```
+
+## Output
+
+Successful builds are copied into the install directory (default: `../ext`) and named like:
+
+`php_<PHP_FULL_VERSION>-interbase-<DRIVER_VERSION>-<os>-<arch>.so`
+
+If a particular PHP version fails to build, the script continues with the next one and reports all failed versions at the end.
+
diff --git a/build_scripts/php-fb-build-linux.sh b/build_scripts/php-fb-build-linux.sh
@@ -9,56 +9,109 @@ PHP_VERSIONS=("7.4" "8.0" "8.1" "8.2" "8.3" "8.4") # Adjust as needed
 BUILD_DIR="php-firebird-build"
 
 set -e
+set -o pipefail
 
-mkdir -p "$INSTALL_DIR"
-rm -rf "$BUILD_DIR"
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+ROOT_DIR="$(cd "$SCRIPT_DIR/.." && pwd)"
+INSTALL_DIR_ABS="$(realpath -m "$ROOT_DIR/$INSTALL_DIR")"
+BUILD_DIR_ABS="$ROOT_DIR/$BUILD_DIR"
+
+mkdir -p "$INSTALL_DIR_ABS"
+rm -rf "$BUILD_DIR_ABS"
 echo "Cloning repository from $REPO_URL (branch: $BRANCH_OR_COMMIT)..."
-git clone --depth 1 --branch "$BRANCH_OR_COMMIT" "$REPO_URL" "$BUILD_DIR"
-
-# Read version from driver
-if [[ -f "$BUILD_DIR/php_interbase.h" ]]; then
-    DRIVER_VERSION=$(grep -Eo 'PHP_INTERBASE_VERSION\s+"[0-9.]+"' "$BUILD_DIR/php_interbase.h" | grep -Eo '[0-9.]+')
-elif [[ -f "$BUILD_DIR/package.xml" ]]; then
-    DRIVER_VERSION=$(grep -oPm1 "(?<=<version>)[^<]+" "$BUILD_DIR/package.xml")
-else
-    DRIVER_VERSION="unknown"
-fi
+git clone --depth 1 --branch "$BRANCH_OR_COMMIT" "$REPO_URL" "$BUILD_DIR_ABS"
+
+extract_driver_version() {
+    local header="$1"
+    local pkgxml="$2"
+
+    if [[ -f "$header" ]]; then
+        # Legacy style (if present)
+        local legacy
+        legacy="$(awk '$1=="#define" && $2=="PHP_INTERBASE_VERSION" { gsub(/\"/,"",$3); print $3; exit }' "$header" 2>/dev/null || true)"
+        if [[ -n "$legacy" ]]; then
+            echo "$legacy"
+            return 0
+        fi
+
+        # Current style: parse MAJOR/MINOR/REV and optional PRE
+        local major minor rev pre
+        major="$(awk '/^#define[[:space:]]+PHP_INTERBASE_VER_MAJOR[[:space:]]+/{print $3; exit}' "$header" 2>/dev/null || true)"
+        minor="$(awk '/^#define[[:space:]]+PHP_INTERBASE_VER_MINOR[[:space:]]+/{print $3; exit}' "$header" 2>/dev/null || true)"
+        rev="$(awk '/^#define[[:space:]]+PHP_INTERBASE_VER_REV[[:space:]]+/{print $3; exit}' "$header" 2>/dev/null || true)"
+        pre="$(awk '/^#define[[:space:]]+PHP_INTERBASE_VER_PRE[[:space:]]+/{print $3; exit}' "$header" 2>/dev/null || true)"
+        pre="${pre%\"}"
+        pre="${pre#\"}"
+
+        if [[ -n "$major" && -n "$minor" && -n "$rev" ]]; then
+            echo "${major}.${minor}.${rev}${pre}"
+            return 0
+        fi
+    fi
+
+    if [[ -f "$pkgxml" ]]; then
+        local v
+        v="$(grep -oPm1 '(?<=<version>)[^<]+' "$pkgxml" 2>/dev/null || true)"
+        if [[ -n "$v" ]]; then
+            echo "$v"
+            return 0
+        fi
+    fi
+
+    echo "unknown"
+}
+
+DRIVER_VERSION="$(extract_driver_version "$BUILD_DIR_ABS/php_interbase.h" "$BUILD_DIR_ABS/package.xml")"
+
+FAILED_VERSIONS=()
 
 for VERSION in "${PHP_VERSIONS[@]}"; do
     echo "==> Building for PHP $VERSION"
 
-    PHP_BIN="/usr/bin/php$VERSION"
-    PHPIZE="/usr/bin/phpize$VERSION"
-    PHP_CONFIG="/usr/bin/php-config$VERSION"
+    if (
+        PHP_BIN="/usr/bin/php$VERSION"
+        PHPIZE="/usr/bin/phpize$VERSION"
+        PHP_CONFIG="/usr/bin/php-config$VERSION"
 
-    if [[ ! -x "$PHP_BIN" || ! -x "$PHPIZE" || ! -x "$PHP_CONFIG" ]]; then
-        echo "--> Installing missing PHP $VERSION packages..."
-        sudo apt-get install -y "php$VERSION-dev" "php$VERSION-cli" "php$VERSION-common"
-    fi
+        if [[ ! -x "$PHP_BIN" || ! -x "$PHPIZE" || ! -x "$PHP_CONFIG" ]]; then
+            echo "--> Installing missing PHP $VERSION packages..."
+            sudo apt-get install -y "php$VERSION-dev" "php$VERSION-cli" "php$VERSION-common"
+        fi
 
-    cd "$BUILD_DIR"
+        cd "$BUILD_DIR_ABS"
 
-    echo "--> Cleaning previous build (if any)..."
-    make clean || true
-    echo "--> Running phpize..."
-    "$PHPIZE"
-    echo "--> Configuring build..."
-    CPPFLAGS="-I$FIREBIRD_INCLUDE_DIR" ./configure --with-php-config="$PHP_CONFIG"
-    echo "--> Compiling..."
-    make -j"$(nproc)"
+        echo "--> Cleaning previous build (if any)..."
+        make clean || true
+        echo "--> Running phpize..."
+        "$PHPIZE"
+        echo "--> Configuring build..."
+        CPPFLAGS="-I$FIREBIRD_INCLUDE_DIR" ./configure --with-php-config="$PHP_CONFIG"
+        echo "--> Compiling..."
+        make -j"$(nproc)"
 
-    PHP_FULL_VERSION=$("$PHP_BIN" -r 'echo PHP_VERSION;')
-    ARCH=$(uname -m)
-    OS=$(uname -s | tr '[:upper:]' '[:lower:]')
+        PHP_FULL_VERSION=$("$PHP_BIN" -r 'echo PHP_VERSION;')
+        ARCH=$(uname -m)
+        OS=$(uname -s | tr '[:upper:]' '[:lower:]')
 
-    OUTPUT_FILE="php_${PHP_FULL_VERSION}-interbase-${DRIVER_VERSION}-${OS}-${ARCH}.so"
-    mkdir -p "$INSTALL_DIR"
-    echo "--> Copying output to $INSTALL_DIR/$OUTPUT_FILE"
-    cp modules/interbase.so "$INSTALL_DIR/$OUTPUT_FILE"
+        OUTPUT_FILE="php_${PHP_FULL_VERSION}-interbase-${DRIVER_VERSION}-${OS}-${ARCH}.so"
+        mkdir -p "$INSTALL_DIR_ABS"
+        echo "--> Copying output to $INSTALL_DIR_ABS/$OUTPUT_FILE"
+        cp modules/interbase.so "$INSTALL_DIR_ABS/$OUTPUT_FILE"
 
-    echo "Build complete for PHP $VERSION: $OUTPUT_FILE"
-    cd ..
+        echo "Build complete for PHP $VERSION: $OUTPUT_FILE"
+    ); then
+        :
+    else
+        echo "Build FAILED for PHP $VERSION" >&2
+        FAILED_VERSIONS+=("$VERSION")
+    fi
 done
 
-echo "All builds completed. Files are located in: $INSTALL_DIR"
+if (( ${#FAILED_VERSIONS[@]} > 0 )); then
+    echo "Some builds failed: ${FAILED_VERSIONS[*]}" >&2
+    echo "Artifacts (successful builds) are located in: $INSTALL_DIR_ABS"
+    exit 1
+fi
+
+echo "All builds completed. Files are located in: $INSTALL_DIR_ABS"
 

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+php-firebird-build/`
`1`	`2`	`.vscode`
`2`	`3`	`.deps`
`3`	`4`	`.libs`