Req #53436 [Fbk->Opn]: RFE: add check for php headers in phpize

Edit report at https://bugs.php.net/bug.php?id=53436&edit=1 ID: 53436 User updated by: fedora at famillecollet dot com Reported by: fedora at famillecollet dot com Summary: RFE: add check for php headers in phpize -Status: Feedback +Status: Open Type: Feature/Change Request Package: Compile Failure Operating System: GNU/Linux (Fedora 14) PHP Version: 5.3SVN-2010-12-01 (SVN) Block user comment: N Private report: N New Comment: > This patch can not be incorporated as-is - it refers to "php-devel package", Yes, the message need to be more "generic" > In general, I'm not sure why PECL command should always fail on --version > if headers are not present, Because phpize get version from headers (see phpize_get_api_numbers) > so how this patch improves things? Giving a comprehensive explanation to the user (the reason why the build fails) Previous Comments: ------------------------------------------------------------------------ [2011-08-28 09:44:45] stas@php.net This patch can not be incorporated as-is - it refers to "php-devel package", which has no meaning outside of certain binary distributions of PHP. In general, I'm not sure why PECL command should always fail on --version if headers are not present, especially since, as you suggest, it may be useful without them. As for the build, if I understand correctly, the build would fail anyway if headers are not present, so how this patch improves things? ------------------------------------------------------------------------ [2010-12-01 15:49:52] fedora at famillecollet dot com Description: ------------ phpize is used to build pecl extension. This script already does a lot of sanity check before the build. On GNU/Linux distribution, PHP headers are available in a sub-package phpize should check if header are available before running the build. See : https://bugzilla.redhat.com/657812 Note than pecl command is usefull, even without "php-devel" to register/unregister packaged extension (--nobuild option), to list regitered extensions, etc. Test script: --------------- # yum remove php-devel # or any other command, apt, ... # pecl install APC Expected result: ---------------- 49 source files, building running: phpize Can't find PHP headers in /usr/include/php The php-devel package is required for use of this command. ERROR: `phpize' failed ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=53436&edit=1