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