436: pre-packaged binary exts

composer.json

@@ -28,26 +28,26 @@
     ],
     "require": {
         "php": "8.1.*||8.2.*||8.3.*||8.4.*||8.5.*",
-        "composer/composer": "^2.9.2",
+        "composer/composer": "^2.9.5",
         "composer/pcre": "^3.3.2",
         "composer/semver": "^3.4.4",
         "fidry/cpu-core-counter": "^1.3.0",
         "illuminate/container": "^10.49.0",
         "psr/container": "^2.0.2",
-        "symfony/console": "^6.4.27",
-        "symfony/event-dispatcher": "^6.4.25",
-        "symfony/process": "^6.4.26",
+        "symfony/console": "^6.4.32",
+        "symfony/event-dispatcher": "^6.4.32",
+        "symfony/process": "^6.4.33",
         "thephpf/attestation": "^0.0.5",
         "webmozart/assert": "^1.12.1"
     },
     "require-dev": {
         "ext-openssl": "*",
-        "behat/behat": "^3.27.0",
+        "behat/behat": "^3.29.0",
         "bnf/phpstan-psr-container": "^1.1",
         "doctrine/coding-standard": "^14.0.0",
-        "phpstan/phpstan": "^2.1.32",
+        "phpstan/phpstan": "^2.1.37",
         "phpstan/phpstan-webmozart-assert": "^2.0",
-        "phpunit/phpunit": "^10.5.58"
+        "phpunit/phpunit": "^10.5.63"
     },
     "replace": {
         "symfony/polyfill-php81": "*",

docs/extension-maintainers.md

@@ -228,9 +228,11 @@ The `build-path` may contain some templated values which are replaced:
 ##### `download-url-method`
 
 The `download-url-method` directive allows extension maintainers to
-change the behaviour of downloading the source package.
+change the behaviour of downloading the source package. This should be defined
+as a list of supported methods, but for backwards compatibility a single
+string may be used.
 
- * Setting this to `composer-default`, which is the default value if not
+ * Setting this to `composer-default`, which is the default value if nothing is
    specified, will use the default behaviour implemented by Composer, which is
    to use the standard ZIP archive from the GitHub API (or other source control
    system).
@@ -240,6 +242,50 @@ change the behaviour of downloading the source package.
    * `php_{ExtensionName}-{Version}-src.zip` (e.g. `php_myext-1.20.1-src.zip`)
    * `{ExtensionName}-{Version}.tgz` (this is intended for backwards
      compatibility with PECL packages)
+ * Using `pre-packaged-binary` will locate a tgz or zip archive in the release
+   assets list based on matching one of the following naming conventions:
+   * `php_{ExtensionName}-{Version}_php{PhpVersion}-{Arch}-{OS}-{Libc}-{Debug}-{TSMode}.{Format}`
+   * The replacements are:
+     * `{ExtensionName}` the name of your extension, e.g. `xdebug` (hint: this
+       is not your Composer package name!)
+     * `{PhpVersion}` the major and minor version of PHP, e.g. `8.5`
+     * `{Version}` the version of your extension, e.g. `1.20.1`
+     * `{Arch}` the architecture of the binary, one of `x86`, `x86_64`, `arm64`
+     * `{OS}` the operating system, one of `windows`, `darwin`, `linux`, `bsd`, `solaris`, `unknown`
+     * `{Libc}` the libc flavour, one of `glibc`, `musl`, `bsdlibc`
+     * `{Debug}` the debug mode, one of `debug`, `nodebug` (or omitted)
+     * `{TSMode}` the thread safety mode, one of `zts`, `nts` (or omitted)
+     * `{Format}` the archive format, one of `zip`, `tgz`
+   * Some examples of valid asset names:
+     * `php_xdebug-4.1_php8.4-x86_64-linux-glibc.tgz` (or `php_xdebug-4.1_php8.4-x86_64-glibc-nts.tgz`)
+     * `php_xdebug-4.1_php8.4-x86_64-linux-musl.tgz` (or `php_xdebug-4.1_php8.4-x86_64-musl-nts.tgz`)
+     * `php_xdebug-4.1_php8.4-arm64-linux-glibc.tgz` (or `php_xdebug-4.1_php8.4-arm64-glibc-nts.tgz`)
+     * `php_xdebug-4.1_php8.4-arm64-linux-musl.tgz` (or `php_xdebug-4.1_php8.4-arm64-musl-nts.tgz`)
+     * `php_xdebug-4.1_php8.4-x86_64-linux-glibc-zts.tgz`
+     * `php_xdebug-4.1_php8.4-x86_64-linux-musl-zts.tgz`
+     * `php_xdebug-4.1_php8.4-arm64-linux-glibc-zts.tgz`
+     * `php_xdebug-4.1_php8.4-arm64-linux-musl-zts.tgz`
+     * `php_xdebug-4.1_php8.4-x86_64-linux-glibc-debug.tgz`
+     * `php_xdebug-4.1_php8.4-x86_64-linux-musl-debug.tgz`
+     * `php_xdebug-4.1_php8.4-arm64-linux-glibc-debug.tgz`
+     * `php_xdebug-4.1_php8.4-arm64-linux-musl-debug.tgz`
+   * It is recommended that `pre-packaged-binary` is combined with `composer-default`
+     as a fallback mechanism, if a particular combination is supported, but not
+     pre-packaged on the release, e.g. `"download-url-method": ["pre-packaged-binary", "composer-default"]`.
+     PIE will try to find a pre-packaged binary asset first, but if it cannot
+     find an appropriate binary, it will download the source code and build it
+     in the traditional manner.
+
+###### Example of using `pre-packaged-binary` with `composer-default` fallback
+
+```json
+{
+    "name": "myvendor/myext",
+    "php-ext": {
+        "download-url-method": ["pre-packaged-binary", "composer-default"]
+    }
+}
+```
 
 ##### `os-families` restrictions
 

phpstan-baseline.neon

@@ -175,31 +175,31 @@ parameters:
 			path: src/DependencyResolver/Package.php
 
 		-
-			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$downloadUrlMethod is assigned outside of the constructor\.$#'
+			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$incompatibleOsFamilies is assigned outside of the constructor\.$#'
 			identifier: property.readOnlyByPhpDocAssignNotInConstructor
 			count: 1
 			path: src/DependencyResolver/Package.php
 
 		-
-			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$incompatibleOsFamilies is assigned outside of the constructor\.$#'
+			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$priority is assigned outside of the constructor\.$#'
 			identifier: property.readOnlyByPhpDocAssignNotInConstructor
 			count: 1
 			path: src/DependencyResolver/Package.php
 
 		-
-			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$priority is assigned outside of the constructor\.$#'
+			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$supportNts is assigned outside of the constructor\.$#'
 			identifier: property.readOnlyByPhpDocAssignNotInConstructor
 			count: 1
 			path: src/DependencyResolver/Package.php
 
 		-
-			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$supportNts is assigned outside of the constructor\.$#'
+			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$supportZts is assigned outside of the constructor\.$#'
 			identifier: property.readOnlyByPhpDocAssignNotInConstructor
 			count: 1
 			path: src/DependencyResolver/Package.php
 
 		-
-			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$supportZts is assigned outside of the constructor\.$#'
+			message: '#^@readonly property Php\\Pie\\DependencyResolver\\Package\:\:\$supportedDownloadUrlMethods is assigned outside of the constructor\.$#'
 			identifier: property.readOnlyByPhpDocAssignNotInConstructor
 			count: 1
 			path: src/DependencyResolver/Package.php
@@ -414,26 +414,14 @@ parameters:
 			count: 1
 			path: test/unit/ComposerIntegration/VendorCleanupTest.php
 
-		-
-			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{download\-url\-method\: array\{''composer\-default''\}\} given\.$#'
-			identifier: argument.type
-			count: 1
-			path: test/unit/DependencyResolver/PackageTest.php
-
-		-
-			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{download\-url\-method\: array\{''pre\-packaged\-source'', ''composer\-default''\}\} given\.$#'
-			identifier: argument.type
-			count: 1
-			path: test/unit/DependencyResolver/PackageTest.php
-
 		-
 			message: '#^Parameter \#2 \$extractedSourcePath of static method Php\\Pie\\Downloading\\DownloadedPackage\:\:fromPackageAndExtractedPath\(\) expects string, string\|false given\.$#'
 			identifier: argument.type
 			count: 3
 			path: test/unit/Downloading/DownloadedPackageTest.php
 
 		-
-			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{extension\-name\: null\} given\.$#'
+			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: list\<string\>\|string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{extension\-name\: null\} given\.$#'
 			identifier: argument.type
 			count: 1
 			path: test/unit/ExtensionNameTest.php

resources/composer-json-php-ext-schema.json

@@ -41,10 +41,28 @@
                     "default": null
                 },
                 "download-url-method": {
-                    "type": "string",
-                    "description": "If specified, this technique will be used to override the URL that PIE uses to download the asset. The default, if not specified, is composer-default.",
-                    "enum": ["composer-default", "pre-packaged-source"],
-                    "example": "composer-default"
+                    "oneOf": [
+                        {
+                            "type": "string",
+                            "description": "If specified, this technique will be used to override the URL that PIE uses to download the asset. The default, if not specified, is composer-default.",
+                            "deprecated": true,
+                            "enum": ["composer-default", "pre-packaged-source", "pre-packaged-binary"],
+                            "example": "composer-default",
+                            "default": "composer-default"
+                        },
+                        {
+                            "type": "array",
+                            "description": "Multiple techniques can be specified, in which case PIE will try each in turn until one succeeds. The first technique that succeeds will be used.",
+                            "items": {
+                                "type": "string",
+                                "description": "If specified, this technique will be used to override the URL that PIE uses to download the asset. The default, if not specified, is composer-default.",
+                                "enum": ["composer-default", "pre-packaged-source", "pre-packaged-binary"],
+                                "example": ["pre-packaged-binary", "composer-default"]
+                            },
+                            "minItems": 1,
+                            "default": ["composer-default"]
+                        }
+                    ]
                 },
                 "os-families": {
                     "type": "array",

src/Building/ExtensionBinaryNotFound.php

@@ -10,6 +10,14 @@
 
 class ExtensionBinaryNotFound extends RuntimeException
 {
+    public static function fromPrePackagedBinary(string $expectedBinaryName): self
+    {
+        return new self(sprintf(
+            'Expected pre-packaged binary does not exist: %s',
+            $expectedBinaryName,
+        ));
+    }
+
     public static function fromExpectedBinary(string $expectedBinaryName): self
     {
         return new self(sprintf(

src/Building/UnixBuild.php

@@ -5,8 +5,10 @@
 namespace Php\Pie\Building;
 
 use Composer\IO\IOInterface;
+use LogicException;
 use Php\Pie\ComposerIntegration\BundledPhpExtensionsRepository;
 use Php\Pie\Downloading\DownloadedPackage;
+use Php\Pie\Downloading\DownloadUrlMethod;
 use Php\Pie\File\BinaryFile;
 use Php\Pie\Platform\TargetPhp\PhpizePath;
 use Php\Pie\Platform\TargetPlatform;
@@ -33,6 +35,45 @@ public function __invoke(
         array $configureOptions,
         IOInterface $io,
         PhpizePath|null $phpizePath,
+    ): BinaryFile {
+        switch (DownloadUrlMethod::fromDownloadedPackage($downloadedPackage)) {
+            case DownloadUrlMethod::PrePackagedBinary:
+                return $this->prePackagedBinary($downloadedPackage, $io);
+
+            case DownloadUrlMethod::ComposerDefaultDownload:
+            case DownloadUrlMethod::PrePackagedSourceDownload:
+                return $this->buildFromSource($downloadedPackage, $targetPlatform, $configureOptions, $io, $phpizePath);
+
+            default:
+                throw new LogicException('Unknown download method');
+        }
+    }
+
+    private function prePackagedBinary(
+        DownloadedPackage $downloadedPackage,
+        IOInterface $io,
+    ): BinaryFile {
+        $expectedSoFile = $downloadedPackage->extractedSourcePath . '/' . $downloadedPackage->package->extensionName()->name() . '.so';
+
+        if (! file_exists($expectedSoFile)) {
+            throw ExtensionBinaryNotFound::fromPrePackagedBinary($expectedSoFile);
+        }
+
+        $io->write(sprintf(
+            '<info>Pre-packaged binary found:</info> %s',
+            $expectedSoFile,
+        ));
+
+        return BinaryFile::fromFileWithSha256Checksum($expectedSoFile);
+    }
+
+    /** @param list<non-empty-string> $configureOptions */
+    private function buildFromSource(
+        DownloadedPackage $downloadedPackage,
+        TargetPlatform $targetPlatform,
+        array $configureOptions,
+        IOInterface $io,
+        PhpizePath|null $phpizePath,
     ): BinaryFile {
         $outputCallback = null;
         if ($io->isVerbose()) {

src/ComposerIntegration/InstallAndBuildProcess.php

@@ -48,6 +48,7 @@ public function __invoke(
             $composerPackage,
         );
 
+        $builtBinaryFile = null;
         if ($composerRequest->operation->shouldBuild()) {
             $builtBinaryFile = ($this->pieBuild)(
                 $downloadedPackage,
@@ -75,6 +76,7 @@ public function __invoke(
             ($this->pieInstall)(
                 $downloadedPackage,
                 $composerRequest->targetPlatform,
+                $builtBinaryFile,
                 $io,
                 $composerRequest->attemptToSetupIniFile,
             ),

src/ComposerIntegration/Listeners/CouldNotDetermineDownloadUrlMethod.php

@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Php\Pie\ComposerIntegration\Listeners;
+
+use Php\Pie\DependencyResolver\Package;
+use Php\Pie\Downloading\DownloadUrlMethod;
+use RuntimeException;
+
+use function array_key_first;
+use function count;
+use function sprintf;
+
+use const PHP_EOL;
+
+class CouldNotDetermineDownloadUrlMethod extends RuntimeException
+{
+    /**
+     * @param non-empty-list<DownloadUrlMethod> $downloadMethods
+     * @param array<string, string>             $failureReasons
+     */
+    public static function fromDownloadUrlMethods(Package $piePackage, array $downloadMethods, array $failureReasons): self
+    {
+        $message = sprintf('Could not download %s', $piePackage->name());
+
+        if (count($downloadMethods) === 1) {
+            $first    = array_key_first($downloadMethods);
+            $message .= sprintf(' using %s method: %s', $downloadMethods[$first]->value, $failureReasons[$downloadMethods[$first]->value] ?? '(unknown failure)');
+
+            return new self($message);
+        }
+
+        $message .= ' using the following methods:' . PHP_EOL;
+
+        foreach ($downloadMethods as $downloadMethod) {
+            $message .= sprintf(' - %s: %s%s', $downloadMethod->value, $failureReasons[$downloadMethod->value] ?? '(unknown failure)', PHP_EOL);
+        }
+
+        return new self($message);
+    }
+}