try to improve support for 8.7

Author: jcupitt

CHANGELOG.md

@@ -10,6 +10,7 @@ All notable changes to `:vips` will be documented in this file.
 - added streaming examples [jcupitt]
 - fix php 8.3 compatibility [uuf6429]
 - better library finding [uuf6429]
+- fix compat with libvips before 8.9 [allanvb]
 
 ## 2.3.0 - 2023-09-26
 

src/FFI.php

@@ -812,14 +812,19 @@ private static function init(): void
             "VipsOperation" => self::$vips->type("VipsOperation*"),
             "VipsImage" => self::$vips->type("VipsImage*"),
             "VipsInterpolate" => self::$vips->type("VipsInterpolate*"),
-            "VipsConnection" => self::$vips->type("VipsConnection*"),
-            "VipsSource" => self::$vips->type("VipsSource*"),
-            "VipsSourceCustom" => self::$vips->type("VipsSourceCustom*"),
-            "VipsTarget" => self::$vips->type("VipsTarget*"),
-            "VipsTargetCustom" => self::$vips->type("VipsTargetCustom*"),
             "VipsProgress" => self::$vips->type("VipsProgress*"),
         ];
 
+        if (self::atLeast(8, 9)) {
+            self::$ctypes = array_merge(self::$ctypes, [
+                "VipsConnection" => self::$vips->type("VipsConnection*"),
+                "VipsSource" => self::$vips->type("VipsSource*"),
+                "VipsSourceCustom" => self::$vips->type("VipsSourceCustom*"),
+                "VipsTarget" => self::$vips->type("VipsTarget*"),
+                "VipsTargetCustom" => self::$vips->type("VipsTargetCustom*"),
+            ]);
+        }
+
         self::$gtypes = [
             "gboolean" => self::$gobject->g_type_from_name("gboolean"),
             "gint" => self::$gobject->g_type_from_name("gint"),

tests/StreamingTest.php

@@ -3,6 +3,7 @@
 namespace Jcupitt\Vips\Test;
 
 use Generator;
+use Jcupitt\Vips\FFI;
 use Jcupitt\Vips\Exception;
 use Jcupitt\Vips\Image;
 use Jcupitt\Vips\Source;
@@ -13,6 +14,15 @@
 
 class StreamingTest extends TestCase
 {
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        if (!FFI::atLeast(8, 9)) {
+            $this->markTestSkipped('libvips too old for streaming tests');
+        }
+    }
+
     /**
      * @throws Exception
      */