Re: Adding runkit7 to PECL
| From: | tyson andre | Date: | Sat, 06 Apr 2019 16:19:08 +0000 |
| Subject: | Re: Adding runkit7 to PECL | ||
| References: | 1 2 | Groups: | php.pecl.dev |
| Request: | Send a blank email to pecl-dev+get-15875@lists.php.net to get a copy of this message | ||
This has been added as https://pecl.php.net/package/runkit7, but Windows
builds fail. Is anyone able to change the Windows ./configure options?
The output of
configure and make are empty when building for Windows: https://windows.php.net/downloads/pecl/releases/runkit7/2.0.2/logs/
For the windows.php.net builds, I think they'd need to have the configure CLI options changed
from the default (not sure what) to configure --enable-runkit. I'm not familiar
with how this is changed.
- This is buildable under Windows in Appveyor: https://github.com/runkit7/runkit7/blob/master/.appveyor.yml
This has a name of runkit7, but <providesextension>runkit</providesextension> and builds
runkit.so/runkit.dll, because the "runkit7" fork is a substitute that has the same
function definitions as the php5 "runkit" PECL. That's probably related to the
windows dll build issues.
From: Christoph M. Becker <cmbecker69@gmx.de>
Sent: April 4, 2019 6:39 AM
To: tyson andre; pecl-dev@lists.php.net
Subject: Re: Adding runkit7 to PECL
On 03.04.2019 at 01:20, tyson andre wrote:
> I'd like to add https://github.com/runkit7/runkit7 to
> PECL as "runkit7".
>
> This is a fork of http://pecl.php.net/runkit providing a
> smaller feature set (due to internals changes).
>
> - Having this on PECL will make installation easier (e.g. on Windows)
>
> I (https://pecl.php.net/user/tandre) will be uploading all releases to PECL.
>
> (I proposed something similar earlier in
> https://marc.info/?l=pecl-dev&m=148870228924766&w=2 but was
> unclear of the approval process for forks of existing repos)
>
> Any objections?
No objections; on the contrary, I'd like to see runkit7 on PECL (plus to
have respective docs in the PHP manual; will try to help with that, if
time permits).
--
Christoph M. Becker