Merge pull request #155 from asgrim/20-detect-setup-php-ini

Automatically setup php.ini entries

Author: asgrim

src/Command/BuildCommand.php

@@ -54,6 +54,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 PieOperation::Resolve,
                 [], // Configure options are not needed for resolve only
                 null,
+                false, // setting up INI not needed for build
             ),
         );
 
@@ -74,6 +75,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 PieOperation::Build,
                 $configureOptionsValues,
                 CommandHelper::determinePhpizePathFromInputs($input),
+                false, // setting up INI not needed for build
             ),
         );
 

src/Command/CommandHelper.php

@@ -38,6 +38,7 @@ final class CommandHelper
     private const OPTION_WITH_PHP_PATH              = 'with-php-path';
     private const OPTION_WITH_PHPIZE_PATH           = 'with-phpize-path';
     private const OPTION_MAKE_PARALLEL_JOBS         = 'make-parallel-jobs';
+    private const OPTION_SKIP_ENABLE_EXTENSION      = 'skip-enable-extension';
 
     /** @psalm-suppress UnusedConstructor */
     private function __construct()
@@ -79,6 +80,12 @@ public static function configureDownloadBuildInstallOptions(Command $command): v
             InputOption::VALUE_REQUIRED,
             'The path to the `phpize` binary to use as the target PHP platform, e.g. --' . self::OPTION_WITH_PHPIZE_PATH . '=/usr/bin/phpize7.4',
         );
+        $command->addOption(
+            self::OPTION_SKIP_ENABLE_EXTENSION,
+            null,
+            InputOption::VALUE_NONE,
+            'Specify this to skip attempting to enable the extension in php.ini',
+        );
 
         self::configurePhpConfigOptions($command);
 
@@ -154,6 +161,11 @@ public static function determineTargetPlatformFromInputs(InputInterface $input,
         return $targetPlatform;
     }
 
+    public static function determineAttemptToSetupIniFile(InputInterface $input): bool
+    {
+        return ! $input->hasOption(self::OPTION_SKIP_ENABLE_EXTENSION) || ! $input->getOption(self::OPTION_SKIP_ENABLE_EXTENSION);
+    }
+
     public static function determinePhpizePathFromInputs(InputInterface $input): PhpizePath|null
     {
         if ($input->hasOption(self::OPTION_WITH_PHPIZE_PATH)) {

src/Command/DownloadCommand.php

@@ -56,6 +56,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 PieOperation::Download,
                 [], // Configure options are not needed for download only
                 null,
+                false, // setting up INI not needed for download
             ),
         );
 

src/Command/InfoCommand.php

@@ -54,6 +54,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 PieOperation::Resolve,
                 [], // Configure options are not needed for resolve only
                 null,
+                false, // setting up INI not needed for info
             ),
         );
 

src/Command/InstallCommand.php

@@ -59,6 +59,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 PieOperation::Resolve,
                 [], // Configure options are not needed for resolve only
                 null,
+                false, // setting up INI not needed for resolve step
             ),
         );
 
@@ -79,6 +80,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 PieOperation::Install,
                 $configureOptionsValues,
                 CommandHelper::determinePhpizePathFromInputs($input),
+                CommandHelper::determineAttemptToSetupIniFile($input),
             ),
         );
 

src/ComposerIntegration/InstallAndBuildProcess.php

@@ -77,6 +77,7 @@ public function __invoke(
                 $downloadedPackage,
                 $composerRequest->targetPlatform,
                 $output,
+                $composerRequest->attemptToSetupIniFile,
             ),
         );
     }

src/ComposerIntegration/PieComposerRequest.php

@@ -24,6 +24,7 @@ public function __construct(
         public readonly PieOperation $operation,
         public readonly array $configureOptions,
         public readonly PhpizePath|null $phpizePath,
+        public readonly bool $attemptToSetupIniFile,
     ) {
     }
 }

src/Container.php

@@ -19,6 +19,7 @@
 use Php\Pie\DependencyResolver\ResolveDependencyWithComposer;
 use Php\Pie\Downloading\GithubPackageReleaseAssets;
 use Php\Pie\Downloading\PackageReleaseAssets;
+use Php\Pie\Installing\Ini;
 use Php\Pie\Installing\Install;
 use Php\Pie\Installing\UnixInstall;
 use Php\Pie\Installing\WindowsInstall;
@@ -71,6 +72,19 @@ static function (ContainerInterface $container): Build {
             },
         );
 
+        $container->singleton(
+            Ini\SetupIniApproach::class,
+            static function (ContainerInterface $container): Ini\SetupIniApproach {
+                return new Ini\PickBestSetupIniApproach([
+                    $container->get(Ini\PreCheckExtensionAlreadyLoaded::class),
+                    $container->get(Ini\OndrejPhpenmod::class),
+                    $container->get(Ini\DockerPhpExtEnable::class),
+                    $container->get(Ini\StandardAdditionalPhpIniDirectory::class),
+                    $container->get(Ini\StandardSinglePhpIni::class),
+                ]);
+            },
+        );
+
         $container->singleton(
             Install::class,
             static function (ContainerInterface $container): Install {

src/DependencyResolver/Package.php

@@ -47,6 +47,7 @@ public function __construct(
         public readonly string|null $buildPath,
         public readonly array|null $compatibleOsFamilies,
         public readonly array|null $incompatibleOsFamilies,
+        public readonly int $priority,
     ) {
     }
 
@@ -93,6 +94,7 @@ public static function fromComposerCompletePackage(CompletePackageInterface $com
             $buildPath,
             self::convertInputStringsToOperatingSystemFamilies($compatibleOsFamilies),
             self::convertInputStringsToOperatingSystemFamilies($incompatibleOsFamilies),
+            $phpExtOptions['priority'] ?? 80,
         );
     }
 

src/Installing/Ini/AddExtensionToTheIniFile.php

@@ -0,0 +1,102 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Php\Pie\Installing\Ini;
+
+use Php\Pie\DependencyResolver\Package;
+use Php\Pie\ExtensionType;
+use Php\Pie\Platform\TargetPhp\PhpBinaryPath;
+use Symfony\Component\Console\Output\OutputInterface;
+use Throwable;
+
+use function file_get_contents;
+use function file_put_contents;
+use function is_string;
+use function is_writable;
+use function sprintf;
+
+use const PHP_EOL;
+
+/** @internal This is not public API for PIE, so should not be depended upon unless you accept the risk of BC breaks */
+class AddExtensionToTheIniFile
+{
+    /** @param callable():bool|null $additionalEnableStep */
+    public function __invoke(
+        string $ini,
+        Package $package,
+        PhpBinaryPath $phpBinaryPath,
+        OutputInterface $output,
+        callable|null $additionalEnableStep,
+    ): bool {
+        if (! is_writable($ini)) {
+            $output->writeln(
+                sprintf(
+                    'PHP is configured to use %s, but it is not writable by PIE.',
+                    $ini,
+                ),
+                OutputInterface::VERBOSITY_VERBOSE,
+            );
+
+            return false;
+        }
+
+        $originalIniContent = file_get_contents($ini);
+
+        if (! is_string($originalIniContent)) {
+            $output->writeln(
+                sprintf(
+                    'Tried making a backup of %s but could not read it, aborting enablement of extension',
+                    $ini,
+                ),
+                OutputInterface::VERBOSITY_VERBOSE,
+            );
+
+            return false;
+        }
+
+        try {
+            file_put_contents(
+                $ini,
+                $originalIniContent . $this->iniFileContent($package),
+            );
+            $output->writeln(
+                sprintf(
+                    'Enabled extension %s in the INI file %s',
+                    $package->extensionName->name(),
+                    $ini,
+                ),
+                OutputInterface::VERBOSITY_VERBOSE,
+            );
+
+            if ($additionalEnableStep !== null && ! $additionalEnableStep()) {
+                return false;
+            }
+
+            $phpBinaryPath->assertExtensionIsLoadedInRuntime($package->extensionName, $output);
+
+            return true;
+        } catch (Throwable $anything) {
+            file_put_contents($ini, $originalIniContent);
+
+            $output->writeln(sprintf(
+                '<error>Something went wrong enabling the %s extension: %s</error>',
+                $package->extensionName->name(),
+                $anything->getMessage(),
+            ));
+
+            return false;
+        }
+    }
+
+    /** @return non-empty-string */
+    private function iniFileContent(Package $package): string
+    {
+        return PHP_EOL
+            . '; PIE automatically added this to enable the ' . $package->name . ' extension' . PHP_EOL
+            . '; priority=' . $package->priority . PHP_EOL
+            . ($package->extensionType === ExtensionType::PhpModule ? 'extension' : 'zend_extension')
+            . '='
+            . $package->extensionName->name() . PHP_EOL;
+    }
+}