Re: [PATCH] Enable extension loading from .user.ini when run as CGI.

On Sat, Jul 28, 2012 at 8:27 AM, Edward Z. Yang <ezyang@mit.edu> wrote: > Excerpts from Pierre Joye's message of Sat Jul 28 02:20:51 -0400 2012: >> > Functionality is not included for other SAPIs because they either do not >> > support .user.ini (e.g. CLI) or they serve multiple requests and thus >> > do not support dl() (e.g. FastCGI). >> >> dl has been disabled in FastCGI and other web SAPIs for a reason, it >> is not working well and brings php to instability. I do not think >> adding this to .user.ini will be any better. > > It has been a while, but last winter we discussed this patch and decided > that extension loading was OK for short-lived PHP processes, e.g. CGI. > So this patch only enables it for CGI. Oh, I miss the CGI only part. It makes sense then, while I am not over warm about the idea tho' :-). >> Can you create a patch only for this bug fix please? Ideally a bug >> report + patch or pull request :-) > > Sure, I can split in two. Does PHP use pull requests? Yes, via github's php mirror, see https://wiki.php.net/vcs/gitworkflow Cheers, -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org