Use private SASL plugins with the MongoDB extension archive

__tests__/extension-packs.test.ts

@@ -1,64 +1,6 @@
 import {extensionPacks} from '../src/extensions';
-import {execFileSync} from 'node:child_process';
-import * as fs from 'node:fs';
-import * as os from 'node:os';
-import * as path from 'node:path';
 
 describe('Optional macOS extension archives', () => {
-  it('installs packs concurrently and preserves successful packs when another fails', () => {
-    const directory = fs.mkdtempSync(
-      path.join(os.tmpdir(), 'setup-php-packs-')
-    );
-    const cache = path.join(directory, 'cache');
-    fs.mkdirSync(cache);
-    try {
-      for (const name of ['imagick', 'mongodb', 'memcached']) {
-        fs.writeFileSync(path.join(cache, name + '.json'), '{}');
-      }
-      fs.writeFileSync(
-        path.join(cache, 'install-extensions.cjs'),
-        `const fs=require('node:fs'), path=require('node:path');
-const [,directory,name]=process.argv.slice(2);
-fs.writeFileSync(path.join(directory,name+'.started'),'yes');
-setTimeout(()=>process.exit(2),1500);
-setInterval(()=>{
-  if (fs.readdirSync(directory).filter(file=>file.endsWith('.started')).length!==3) return;
-  fs.writeFileSync(path.join(directory,name+'.env'),'SASL_PATH=/cache/'+name+'\\n');
-  console.log('completed '+name);
-  process.exit(name==='mongodb'?1:0);
-},10);
-`
-      );
-      const output = execFileSync(
-        'bash',
-        [
-          '-c',
-          '. "$CACHE_SCRIPT"; add_env() { printf "env %s %s\\n" "$1" "$2"; }; (exit 0) & extension_cache_pid=$!; finish_extension_cache_downloads; test "$extension_cache_memcached_dependencies" = "igbinary msgpack"'
-        ],
-        {
-          encoding: 'utf8',
-          env: {
-            ...process.env,
-            CACHE_SCRIPT: path.resolve(
-              'src/scripts/extensions/darwin_cache.sh'
-            ),
-            SETUP_PHP_NODE: process.execPath,
-            SETUP_PHP_EXTENSION_PACKS: 'imagick mongodb memcached',
-            extension_cache_dir: cache,
-            verbose: ''
-          }
-        }
-      );
-      expect(output).toContain('completed imagick');
-      expect(output).toContain('completed memcached');
-      expect(output).toContain('mongodb cache unavailable');
-      expect(output).toContain('env SASL_PATH /cache/imagick');
-      expect(output).not.toContain('env SASL_PATH /cache/mongodb');
-      expect(fs.existsSync(cache)).toBe(false);
-    } finally {
-      fs.rmSync(directory, {recursive: true, force: true});
-    }
-  });
   it('selects each requested pack once', async () => {
     expect(
       await extensionPacks(
@@ -83,31 +25,8 @@ setInterval(()=>{
     }
   );
 
-  it('selects optional packs for PHP 5.6 through 8.7', async () => {
-    for (const version of [
-      '5.6',
-      '7.0',
-      '7.1',
-      '7.2',
-      '7.3',
-      '7.4',
-      '8.0',
-      '8.1',
-      '8.2',
-      '8.3',
-      '8.4',
-      '8.5',
-      '8.6',
-      '8.7'
-    ]) {
-      expect(
-        await extensionPacks('imagick, mongodb, memcached', version, 'darwin')
-      ).toEqual(['imagick', 'mongodb', 'memcached']);
-    }
-  });
-
-  it('keeps unsupported PHP and other platforms on their existing paths', async () => {
-    for (const version of ['5.4', '5.5', '7.5', '8.8', '9.0']) {
+  it('keeps legacy PHP and other platforms on their existing paths', async () => {
+    for (const version of ['5.4', '7.4', '8.1', '8.6']) {
       expect(await extensionPacks('imagick', version, 'darwin')).toEqual([]);
     }
     for (const os of ['linux', 'win32']) {

src/extensions.ts

@@ -6,7 +6,7 @@ export async function extensionPacks(
   version: string,
   os: string
 ): Promise<string[]> {
-  if (os !== 'darwin' || !/^(5\.6|7\.[0-4]|8\.[0-7])$/.test(version)) return [];
+  if (os !== 'darwin' || !/^8\.[2-5]$/.test(version)) return [];
   const requested = await utils.extensionArray(extension_csv);
   return ['imagick', 'mongodb', 'memcached'].filter(
     name => requested.includes(name) && !requested.includes(':' + name)

src/scripts/extensions/darwin_cache.sh

@@ -29,21 +29,10 @@ finish_extension_cache_downloads() {
     extension_cache_dir=
     return 0
   fi
-  local extension variable value index
-  local cache_install_names=() cache_install_pids=()
+  local extension variable value
   for extension in $SETUP_PHP_EXTENSION_PACKS; do
     [ -s "$extension_cache_dir/$extension.json" ] || continue
-    # Packs own separate directories and module names. Extract and validate them
-    # concurrently, then import their environments only after each one succeeds.
-    "$SETUP_PHP_NODE" "$extension_cache_dir/install-extensions.cjs" install "$extension_cache_dir" "$extension" \
-      > "$extension_cache_dir/$extension.install.log" 2>&1 &
-    cache_install_names+=("$extension")
-    cache_install_pids+=("$!")
-  done
-  for index in "${!cache_install_pids[@]}"; do
-    extension=${cache_install_names[$index]}
-    if wait "${cache_install_pids[$index]}"; then
-      cat "$extension_cache_dir/$extension.install.log"
+    if "$SETUP_PHP_NODE" "$extension_cache_dir/install-extensions.cjs" install "$extension_cache_dir" "$extension"; then
       while IFS='=' read -r variable value; do
         case "$variable" in
           MAGICK_CONFIGURE_PATH|MAGICK_CODER_MODULE_PATH|MAGICK_FILTER_MODULE_PATH|SASL_PATH)
@@ -57,7 +46,6 @@ finish_extension_cache_downloads() {
         extension_cache_memcached_dependencies='igbinary msgpack'
       fi
     else
-      cat "$extension_cache_dir/$extension.install.log"
       printf 'setup-php: %s cache unavailable for this PHP; using the existing installer\n' "$extension"
     fi
   done