Remove openmp support for imagemagick

src/SPC/builder/extension/imagick.php

@@ -5,20 +5,16 @@
 namespace SPC\builder\extension;
 
 use SPC\builder\Extension;
-use SPC\builder\linux\LinuxBuilder;
+use SPC\store\FileSystem;
 use SPC\util\CustomExt;
 
 #[CustomExt('imagick')]
 class imagick extends Extension
 {
-    public function patchBeforeMake(): bool
+    public function patchBeforeBuildconf(): bool
     {
-        // imagick may call omp_pause_all which requires -lgomp
-        $extra_libs = getenv('SPC_EXTRA_LIBS') ?: '';
-        if ($this->builder instanceof LinuxBuilder) {
-            $extra_libs .= (empty($extra_libs) ? '' : ' ') . '-lgomp ';
-        }
-        f_putenv('SPC_EXTRA_LIBS=' . $extra_libs);
+        // destroy imagick build conf to avoid libgomp build error
+        FileSystem::replaceFileStr(SOURCE_PATH . '/php-src/ext/imagick/config.m4', '#include <omp.h>', '#include <NEVER_INCLUDE_OMP_H>');
         return true;
     }
 

src/SPC/builder/unix/library/imagemagick.php

@@ -19,7 +19,7 @@ trait imagemagick
     protected function build(): void
     {
         // TODO: imagemagick build with bzip2 failed with bugs, we need to fix it in the future
-        $extra = '--without-jxl --without-x --enable-openmp --without-bzlib ';
+        $extra = '--without-jxl --without-x --disable-openmp --without-bzlib ';
         $required_libs = '';
         $optional_libs = [
             'libzip' => 'zip',

src/SPC/util/SPCConfigUtil.php

@@ -90,10 +90,6 @@ private function getLibsString(array $libraries): string
                 }
             }
         }
-        // patch: imagick (imagemagick wrapper) for linux needs -lgomp
-        if (in_array('imagemagick', $libraries) && PHP_OS_FAMILY === 'Linux') {
-            $short_name[] = '-lgomp';
-        }
         return implode(' ', $short_name);
     }
 

src/globals/test-extensions.php

@@ -24,7 +24,7 @@
     // 'macos-13',
     // 'macos-14',
     'ubuntu-latest',
-    'windows-latest',
+    // 'windows-latest',
 ];
 
 // whether enable thread safe
@@ -40,7 +40,7 @@
 
 // If you want to test your added extensions and libs, add below (comma separated, example `bcmath,openssl`).
 $extensions = match (PHP_OS_FAMILY) {
-    'Linux', 'Darwin' => 'pgsql,pdo_pgsql',
+    'Linux', 'Darwin' => 'imagick',
     'Windows' => 'pgsql,pdo_pgsql',
 };