# [RFC] Allow loading extensions by name

- Thread: https://externals.io/message/99308
- Original archive: https://news-web.php.net/php.internals/99308
- Started by flaupretre@free.fr on 2017-06-01T12:46:10+00:00
- Messages: 6
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #99308 — flaupretre@free.fr — 2017-06-01T12:46:10+00:00

URL: https://externals.io/message/99308#99308

Hi,

Feature was discussed last year. I am now opening the vote for a merge 
in 7.2.

URL: https://wiki.php.net/rfc/load-ext-by-name

Voting period ends Monday, June 19, 2017, 00:00 UTC.

Regards

François

---

## #99310 — flaupretre@free.fr — 2017-06-01T15:18:05+00:00

URL: https://externals.io/message/99308#99310 · In reply to: #99308

Sorry, forgot [VOTE] in subject...

Le 01/06/2017 à 14:46, François Laupretre a écrit :
> Hi,
>
> Feature was discussed last year. I am now opening the vote for a merge 
> in 7.2.
>
> URL: https://wiki.php.net/rfc/load-ext-by-name
>
> Voting period ends Monday, June 19, 2017, 00:00 UTC.
>
> Regards
>
> François

---

## #99553 — francois@tekwire.net — 2017-06-19T17:09:36+00:00

URL: https://externals.io/message/99308#99553 · In reply to: #99310

Hi,

RFC was appoved (17 to 4). Thanks to all who took the time to discuss 
and vote about this.

I just rebased the PR to the current master. Could someone with 
appropriate rights merge it ? Thanks.

Regards

François

Le 01/06/2017 à 17:18, François Laupretre a écrit :
> Sorry, forgot [VOTE] in subject...
>
> Le 01/06/2017 à 14:46, François Laupretre a écrit :
> > […]

---

## #99565 — Sara Golemon — 2017-06-19T21:33:03+00:00

URL: https://externals.io/message/99308#99565 · In reply to: #99553

On Mon, Jun 19, 2017 at 1:09 PM, François Laupretre
<francois@tekwire.net> wrote:
> RFC was appoved (17 to 4). Thanks to all who took the time to discuss and
> vote about this.
>
> I just rebased the PR to the current master. Could someone with appropriate
> rights merge it ? Thanks.
>
I was about to merge this, but ran into some issues (mostly minor).
Could you at least address the fwrite(stderr, ...) item (and
preferably clean up the style nits while you're at it)?

Oh, and I forgot to include it in the CR, but there are some
tabs/spaces issues as well.

-Sara

---

## #99571 — francois@tekwire.net — 2017-06-20T10:53:53+00:00

URL: https://externals.io/message/99308#99571 · In reply to: #99565

Hi Sara,

Le 19/06/2017 à 23:33, Sara Golemon a écrit :
> I was about to merge this, but ran into some issues (mostly minor).
> Could you at least address the fwrite(stderr, ...) item (and
> preferably clean up the style nits while you're at it)?
>
> Oh, and I forgot to include it in the CR, but there are some
> tabs/spaces issues as well.
I fixed typos and style issues.

About the 'fwrite(stderr,', I just used the same mechanism used by 
zend_load_extension() to raise an error. If you consider it is safe to 
replace it with a call to zend_error(), I'll do it.

Regards

François

---

## #99572 — Sara Golemon — 2017-06-20T13:30:29+00:00

URL: https://externals.io/message/99308#99572 · In reply to: #99571

I'll double check later today, it might be that this comes in from the
startup phase and we can't do zend_error() for some reason.  Either
way I'll get it merged soonish.
Thanks.
-Sara

On Tue, Jun 20, 2017 at 6:53 AM, François Laupretre
<francois@tekwire.net> wrote:
> Hi Sara,
>
> Le 19/06/2017 à 23:33, Sara Golemon a écrit :
> > […]
>
> I fixed typos and style issues.
>
> About the 'fwrite(stderr,', I just used the same mechanism used by
> zend_load_extension() to raise an error. If you consider it is safe to
> replace it with a call to zend_error(), I'll do it.
>
> Regards
>
> François