[php-maint] Bug#571714: Bug#571714: Bug#571714: maybe put info into comment

[Raphael Geissert]

On 3 March 2010 00:36, sean finney <seanius at debian.org> wrote:
> On Tue, Mar 02, 2010 at 09:05:04PM +0100, Stefan Fritsch wrote:
>> Two questions:
>>
>> - Shouldn't it be php_admin_flag instead of php_admin_value?
>
> I think in this case it's equivalent, though in general iirc _flag
> could be overridden by ini_set where _value can't.

Er, no. This should indeed be php_admin_flag.

>> - Maybe it would be a good idea to put the info into a comment in the
>> config file? AFAIUI, if someone tries the obvious solution to set
>> php_admin_flag/value engine on, this would prevent users from setting
>> "php_flag engine off" in their .htaccess (but I haven't tested that).
>
> yes, i think it'd be good to have a comment there saying something simply
> like "comment this out if you want php in your public_html dirs") to avoid
> confusion.

Yes to both. This should be included in the next upload (as soon as
I'm able to push commits...)

Cheers,
-- 
Raphael Geissert - Debian Developer
www.debian.org - get.debian.net