Re: Allow loading extensions by name

Hi, Le 29/01/2016 08:09, Martin Keckeis a écrit :

Hello, 2016-01-28 21:20 GMT+01:00 François Laupretre <francois@php.net>:

Hi, Can you please give your thoughts about a PR I just created : https://github.com/php/php-src/pull/1741 Loading extensions by name (instead of file name) provides a portable way of specifying extensions in an INI file. Example : extension=bz2 zend_extension=xdebug This will be converted to the right file name, depending on the platform. Of course, current configurations using filenames remain supported (no BC break), but the recommended way of specifying additional extensions in an INI file becomes 'by name' (See example php.ini files). If I get positive returns, I'll write an RFC targeting 7.1. Regards François -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

i generally like it. Small improvement and optional. In your PR you wrote

A typical example is the coexistence of a Windows development environment

and a Linux production. In such cases, it is impossible to write a single configuration file that will work in both environments, forcing developers to manually maintain two separate versions of the file. That will also not work (in some cases) with this change either. Some other settings can also be OS dependent - error_log - mail.log, SMTP, sendmail_path - some session settings - some path settings -... I think 100% portability will not be achieved very soon

You're right, incompatibilities will remain everywhere absolute paths are provided but, in many cases, the only differences are the 'extension=' lines. Anyway, the main objective of this RFC is to hide the platform-specific details of extension file names, making life easier for beginners and documentation/script maintainers. Regards François