Re: [PECL-DEV] I would like to take over PAM extension which isinactive since 2009

[email protected] ("Christoph M. Becker") Mon, 7 Jun 2021 16:20:20 +0200
Newsgroups php.pecl.dev
Message-ID <[email protected]>
On 07.06.2021 at 15:22, Amish wrote:
>
>> 2/ Supported versions
>>
>> You say 7/8 but package.xml states > 8.0
>> Is it expected (will be nice to have all maintained versions)
>
> I use Arch Linux which supported PHP 7 till January. And then Arch Linux
> moved to PHP 8.
>
> And hence I had to fix the code to run on PHP 8.
>
> https://github.com/amishmm/php-pam/commit/efa0bf4079c2fd0cf8880a9aba1a9e7077a60648
>
>
> I am not a regular PHP coder. So I do not know how to "easily" make
> above commit work on both PHP 7 and PHP 8.

That should be fine for PHP 7 as well.  I suggest to setup CI (GH
actions are quite handy) so you can build/test against different PHP
versions.

--
Christoph M. Becker