PECL exts using real_name are not found by pecl.php.net #1

Closed
opened 2026-01-24 11:41:34 +01:00 by admin · 2 comments
Owner

Originally created by @cmb69 on GitHub (Nov 1, 2019).

Consider pecl_http (which appears to be the only maintained PECL extension using real_name):
6a54106837/data/config/pecl/exts.ini (L242-L243)
The build artifacts are currently named php_http-*.zip and are uploaded to a http folder. However, pecl.php.net expects the binary packages to be pecl_http/php_pecl_http-*. One issue is PECLExt::getPackageName():
6a54106837/include/PeclExt.php (L172-L175)
$this->name is the real_name ("http"), but would have to be the original extension name ("pecl_http"). Another issue is the upload target folder:
6a54106837/script/pecl.php (L330)
PECLExt::getName() again returns the real_name ("http") instead of the original extension name ("pecl_http").

Alternative solutions would be to adapt the respective code in php/web-pecl, or maybe even to drop support for real_name altogether and require packages to have a single name.

What do you think @weltling?

PS: see also m6w6/ext-http#97

Originally created by @cmb69 on GitHub (Nov 1, 2019). Consider pecl_http (which appears to be the only maintained PECL extension using `real_name`): https://github.com/php/web-rmtools/blob/6a54106837758ad96d8ff53ef1bfd2433c6e7f66/data/config/pecl/exts.ini#L242-L243 The build artifacts are currently named `php_http-*.zip` and are uploaded to a `http` folder. However, pecl.php.net expects the binary packages to be `pecl_http/php_pecl_http-*`. One issue is `PECLExt::getPackageName()`: https://github.com/php/web-rmtools/blob/6a54106837758ad96d8ff53ef1bfd2433c6e7f66/include/PeclExt.php#L172-L175 `$this->name` is the `real_name` ("http"), but would have to be the original extension name ("pecl_http"). Another issue is the upload target folder: https://github.com/php/web-rmtools/blob/6a54106837758ad96d8ff53ef1bfd2433c6e7f66/script/pecl.php#L330 `PECLExt::getName()` again returns the `real_name` ("http") instead of the original extension name ("pecl_http"). Alternative solutions would be to adapt the respective code in php/web-pecl, or maybe even to drop support for real_name altogether and require packages to have a single name. What do you think @weltling? PS: see also m6w6/ext-http#97
admin closed this issue 2026-01-24 11:41:34 +01:00
Author
Owner

@m6w6 commented on GitHub (Nov 4, 2019):

It had to be named ˋpecl_httpˋ because there also was a PEAR package with the name ˋhttpˋ.

Just a little history, IIRC.

@m6w6 commented on GitHub (Nov 4, 2019): It had to be named ˋpecl_httpˋ because there also was a PEAR package with the name ˋhttpˋ. Just a little history, IIRC.
Author
Owner

@cmb69 commented on GitHub (Aug 6, 2024):

This PR can be closed, since we're going to archive this repo anyway.

@cmb69 commented on GitHub (Aug 6, 2024): This PR can be closed, since we're going to archive this repo anyway.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: php/web-rmtools#1