Merge pull request #270 from NoiseByNorthwest/php8.4-support

Add PHP8.4 support

Author: NoiseByNorthwest

.github/workflows/main.yml

@@ -21,7 +21,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3' ]
+        php: [ '5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3', '8.4' ]
 
         name:
           - linux
@@ -34,7 +34,7 @@ jobs:
           # Debian (docker)
           - { name: debian, ts: 'nts', compiler: 'gcc',   os: ubuntu-20.04 }
           # macOS
-          - { name: mac,    ts: 'nts', compiler: 'clang', os: macos-12 }
+          - { name: mac,    ts: 'nts', compiler: 'clang', os: macos-13 }
 
     steps:
       - uses: actions/checkout@v3

README.md

@@ -40,7 +40,7 @@ Current requirements are:
 * x86-64 or ARM64
 * **GNU/Linux**, **macOS** or **FreeBSD**
 * zlib dev package (e.g. zlib1g-dev on Debian based distros)
-* PHP 5.4 to 8.3
+* PHP 5.4 to 8.4
 
 ## Installation
 
@@ -503,7 +503,7 @@ See the [LICENSE][:link-license:] file for more information.
 [:badge-ci:]:           https://github.com/NoiseByNorthwest/php-spx/actions/workflows/main.yml/badge.svg
 [:link-ci:]:            https://github.com/NoiseByNorthwest/php-spx/actions/workflows/main.yml
 
-[:badge-php-versions:]: https://img.shields.io/badge/php-5.4--8.3-blue.svg
+[:badge-php-versions:]: https://img.shields.io/badge/php-5.4--8.4-blue.svg
 [:badge-supported-platforms:]: https://img.shields.io/badge/platform-GNU/Linux%20|%20macOS%20|%20FreeBSD%20-yellow
 [:badge-supported-arch:]: https://img.shields.io/badge/architecture-x86--64%20|%20ARM64%20-silver
 

config.m4

@@ -14,6 +14,12 @@ if test "$PHP_SPX" = "yes"; then
 
     CFLAGS="-Werror -Wall -O3 -pthread -std=gnu90"
 
+    if test "$(uname -s 2>/dev/null)" = "Darwin"
+    then
+        # see discussion here https://github.com/NoiseByNorthwest/php-spx/pull/270
+        CFLAGS="$CFLAGS -Wno-typedef-redefinition"
+    fi
+
     if test "$PHP_SPX_DEV" = "yes"
     then
         CFLAGS="$CFLAGS -g"

src/php_spx.h

@@ -31,8 +31,8 @@
 #   error "Only x86-64 and ARM64 architectures are supported"
 #endif
 
-#if ZEND_MODULE_API_NO < 20100525 || ZEND_MODULE_API_NO > 20230831 // 8.3-RC5
-#   error "Only the following PHP versions are supported: 5.4 to 8.3"
+#if ZEND_MODULE_API_NO < 20100525 || ZEND_MODULE_API_NO > 20240924
+#   error "Only the following PHP versions are supported: 5.4 to 8.4"
 #endif
 
 #define PHP_SPX_EXTNAME "SPX"