Re: [PATCH] lighttpd: fix FastCGI configuration for PHP

[Jack Mitchell <ml@communistcode.co.uk>]

Hi Martin,

On 25/06/12 10:13, Marcin Juszkiewicz wrote:
> /usr/local/bin/php was set as PHP interpreter so lighttpd failed to run.
> Also uncommented FastCGI configuration block to make mod_fastcgi
> enablement easier for user.
>
> Signed-off-by: Marcin Juszkiewicz <marcin.juszkiewicz@linaro.org>
> ---
>   meta/recipes-extended/lighttpd/files/lighttpd.conf |   16 ++++++++--------
>   meta/recipes-extended/lighttpd/lighttpd_1.4.30.bb  |    2 +-
>   2 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/meta/recipes-extended/lighttpd/files/lighttpd.conf b/meta/recipes-extended/lighttpd/files/lighttpd.conf
> index 3342470..0838da8 100644
> --- a/meta/recipes-extended/lighttpd/files/lighttpd.conf
> +++ b/meta/recipes-extended/lighttpd/files/lighttpd.conf
> @@ -211,14 +211,14 @@ static-file.exclude-extensions = ( ".php", ".pl", ".fcgi" )
>   #### fastcgi module
>   ## read fastcgi.txt for more info
>   ## for PHP don't forget to set cgi.fix_pathinfo = 1 in the php.ini
> -#fastcgi.server             = ( ".php" =>
> -#                               ( "localhost" =>
> -#                                 (
> -#                                   "socket" => "/tmp/php-fastcgi.socket",
> -#                                   "bin-path" => "/usr/local/bin/php"
> -#                                 )
> -#                               )
> -#                            )
> +fastcgi.server             = ( ".php" =>
> +                               ( "localhost" =>
> +                                 (
> +                                   "socket" => "/tmp/php-fastcgi.socket",
> +                                   "bin-path" => "/usr/bin/php-cgi"

Changing the path here I agree with.

>
> +                                 )
> +                               )
> +                            )
>   
>   #### CGI module
>   #cgi.assign                 = ( ".pl"  => "/usr/bin/perl",
> diff --git a/meta/recipes-extended/lighttpd/lighttpd_1.4.30.bb b/meta/recipes-extended/lighttpd/lighttpd_1.4.30.bb
> index 161c0d9..99b359c 100644
> --- a/meta/recipes-extended/lighttpd/lighttpd_1.4.30.bb
> +++ b/meta/recipes-extended/lighttpd/lighttpd_1.4.30.bb
> @@ -16,7 +16,7 @@ RDEPENDS_${PN} += " \
>                  lighttpd-module-staticfile \
>   "
>   
> -PR = "r1"
> +PR = "r2"
>   
>   SRC_URI = "http://download.lighttpd.net/lighttpd/releases-1.4.x/lighttpd-${PV}.tar.bz2 \
>           file://index.html.lighttpd \

However I don't think we should be enabling php-cgi by default, 
especially as it is not included in oe-core. Uncommenting some lines in 
a configuration file should be manageable by users that can build (and 
add additional layers to) OpenEmbedded.

Regards,

-- 

   Jack Mitchell (jack@embed.me.uk)
   Embedded Systems Engineer
   http://www.embed.me.uk

--