[php-maint] Bug#805222: [Pkg-php-pecl] Bug#805222: php-apcu: FTBFS: PHP Fatal error: Call to a member function getFilelist() on null

[Mathieu Parent (Debian)]

2015-12-07 17:21 GMT+01:00 Ondřej Surý <ondrej at sury.org>:
> Control: reassign -1 php-pear
> Control: found -1 php-pear/5.6.16+dfsg-1
> Control: affects -1 php5-apcu
>
> Hi,
>
> thank you for the report, after some debugging it seems this is a
> generic error in PEAR instead of bug just in the php-apcu. This should
> not affect no users, but it probably broke all PHP extensions, since it
> stops honoring packagingroot after calling PEAR_Registry->setConfig()
>
> I have a fix ready and PHP building, and I am ccing Fedora and SuSE
> maintainers.
>
> Mathieu, this also applies to your standalone src:php-pear:

OK thanks.

> diff --git a/PEAR/Command/Install.php b/PEAR/Command/Install.php
> index 9d572ed..3b1fec9 100644
> --- a/PEAR/Command/Install.php
> +++ b/PEAR/Command/Install.php
> @@ -848,7 +848,7 @@ Run post-installation scripts in package <package>,
> if any exist.
>              $pkg = &$instreg->getPackage($param->getPackage(),
>              $param->getChannel());
>              // $pkg may be NULL if install is a 'fake' install via
>              --packagingroot
>              if (is_object($pkg)) {
> -                $pkg->setConfig($this->config);
> +                $pkg->setConfig($this->config, false);
>                  if ($list = $pkg->listPostinstallScripts()) {
>                      $pn =
>                      $reg->parsedPackageNameToString(array('channel' =>
>                         $param->getChannel(), 'package' =>
>                         $param->getPackage()), true);
>
>
> This fixes the issue right now, but it should be probably reported
> upstream to have a correct fix (since this might break other stuff :)),
> but my PEAR account doesn't work right now, so it might take me a while
> to report this to upstream.

You can propose a PR instead: https://github.com/pear/pear-core/pulls

Cheers
-- 
Mathieu Parent