On 3 March 2010 00:36, sean finney <sean...@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 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org