Re: RE: [PATCH] adding stream wrappers to include_path
| From: | Greg Beaver | Date: | Fri, 14 Mar 2008 02:39:58 +0000 |
| Subject: | Re: RE: [PATCH] adding stream wrappers to include_path | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-36132@lists.php.net to get a copy of this message | ||
Pierre Joye wrote:
Hi Dmitry, On Thu, Mar 13, 2008 at 12:25 PM, Dmitry Stogov <dmitry@zend.com> wrote:Actually, this feature was requested several times (and immediately dismissed) about 2 years ago, first by Francois Laupretre, I think, but I've heard it requested many times. Until I realized how easy it is to implement, I was actually against this kind of thing. I strongly encourage those who have the impulse to dismiss the patch to actually try it. For no performance hit, it adds big functionality to PHP. To be ultra-clear, the patch is not phar-specific. GregIt is too difficult for extension that is not in the core distribution yet. For now I think it is simple to override zend_stream_open_function() and zend_resolve_path_function() callbacks by ext/phar and provide something like phar.include_path directive.It is indeed a solution. But my point was to support quotes in the include_path, no matter if an extension needs it or not. However it is not that critical and nobody has complained about this problem before