tony2001 Tue, 15 Dec 2009 13:15:05 +0000 Revision: http://svn.php.net/viewvc?view=revision&revision=292166
Log: use --localstatedir instead of separate options for pid and log files Changed paths: U php/php-src/branches/PHP_5_3_FPM/sapi/fpm/Makefile.frag U php/php-src/branches/PHP_5_3_FPM/sapi/fpm/config.m4 U php/php-src/branches/PHP_5_3_FPM/sapi/fpm/fpm/fpm_conf.c U php/php-src/branches/PHP_5_3_FPM/sapi/fpm/php-fpm.conf.in Modified: php/php-src/branches/PHP_5_3_FPM/sapi/fpm/Makefile.frag =================================================================== --- php/php-src/branches/PHP_5_3_FPM/sapi/fpm/Makefile.frag 2009-12-15 13:13:51 UTC (rev 292165) +++ php/php-src/branches/PHP_5_3_FPM/sapi/fpm/Makefile.frag 2009-12-15 13:15:05 UTC (rev 292166) @@ -10,8 +10,8 @@ install-fpm: install-sapi @echo "Installing PHP FPM binary: $(INSTALL_ROOT)$(sbindir)/" @$(mkinstalldirs) $(INSTALL_ROOT)$(sbindir) - @$(mkinstalldirs) $(INSTALL_ROOT)$(php_fpm_pid_dir) - @$(mkinstalldirs) $(INSTALL_ROOT)$(php_fpm_log_dir) + @$(mkinstalldirs) $(INSTALL_ROOT)$(localstatedir)/log + @$(mkinstalldirs) $(INSTALL_ROOT)$(localstatedir)/run @$(INSTALL) -m 0755 $(SAPI_FPM_PATH) $(INSTALL_ROO)$(sbindir)/$(program_prefix)php-fpm$(program_suffix)$(EXEEXT) @echo "Installing PHP FPM config: $(INSTALL_ROOT)$(sysconfdir)/" && \ Modified: php/php-src/branches/PHP_5_3_FPM/sapi/fpm/config.m4 =================================================================== --- php/php-src/branches/PHP_5_3_FPM/sapi/fpm/config.m4 2009-12-15 13:13:51 UTC (rev 292165) +++ php/php-src/branches/PHP_5_3_FPM/sapi/fpm/config.m4 2009-12-15 13:15:05 UTC (rev 292166) @@ -215,14 +215,6 @@ dnl }}} dnl configure checks {{{ -AC_DEFUN([AC_FPM_CHECKS], -[ - AC_FPM_STDLIBS - AC_FPM_PRCTL - AC_FPM_CLOCK - AC_FPM_TRACE -]) - AC_DEFUN([AC_FPM_STDLIBS], [ AC_CHECK_FUNCS(setenv clearenv) @@ -506,39 +498,39 @@ ]) dnl }}} -dnl configure options {{{ -AC_DEFUN([AC_FPM_ARGS], -[ - PHP_ARG_WITH(fpm-log,, - [ --with-fpm-log[=PATH] Set the path for php-fpm log file. (default: /var/log/php-fpm.log)], yes, no) +AC_MSG_CHECKING(for FPM build) +if test "$PHP_FPM" != "no"; then + AC_MSG_RESULT($PHP_FPM) - PHP_ARG_WITH(fpm-pid,, - [ --with-fpm-pid[=PATH] Set the path for php-fpm pid file. (default: /var/run/php-fpm.pid)], yes, no) + AC_LIB_EVENT([$minimum_libevent_version]) + AC_CACHE_CHECK(whether libevent build works, php_cv_libevent_build_works, [ + PHP_TEST_BUILD(event_init, + [ + php_cv_libevent_build_works=yes + ], [ + AC_MSG_RESULT(no) + AC_MSG_ERROR([build test failed. Please check the config.log for details.]) + ], $LIBEVENT_LIBS) + ]) + + PHP_SETUP_LIBXML(FPM_SHARED_LIBADD, [ + ], [ + AC_MSG_ERROR([xml2-config not found. Please check your libxml2 installation.]) + ]) + + AC_FPM_STDLIBS + AC_FPM_PRCTL + AC_FPM_CLOCK + AC_FPM_TRACE + PHP_ARG_WITH(fpm-user,, [ --with-fpm-user[=USER] Set the user for php-fpm to run as. (default: nobody)], nobody, no) PHP_ARG_WITH(fpm-group,, [ --with-fpm-group[=GRP] Set the group for php-fpm to run as. For a system user, this should usually be set to match the fpm username (default: nobody)], nobody, no) -]) -AC_DEFUN([AC_FPM_VARS], -[ - if test -z "$PHP_FPM_LOG" -o "$PHP_FPM_LOG" = "yes" -o "$PHP_FPM_LOG" = "no"; then - php_fpm_log_path="/var/log/php-fpm.log" - else - php_fpm_log_path="$PHP_FPM_LOG" - fi - php_fpm_log_dir=`dirname $php_fpm_log_path` - - if test -z "$PHP_FPM_PID" -o "$PHP_FPM_PID" = "yes" -o "$PHP_FPM_PID" = "no"; then - php_fpm_pid_path="/var/run/php-fpm.pid" - else - php_fpm_pid_path="$PHP_FPM_PID" - fi - php_fpm_pid_dir=`dirname $php_fpm_pid_path` - if test -z "$PHP_FPM_USER" -o "$PHP_FPM_USER" = "yes" -o "$PHP_FPM_USER" = "no"; then php_fpm_user="nobody" else @@ -551,51 +543,14 @@ php_fpm_group="$PHP_FPM_GROUP" fi - PHP_SUBST_OLD(fpm_version) - PHP_SUBST_OLD(php_fpm_log_path) - PHP_SUBST_OLD(php_fpm_pid_path) - PHP_SUBST_OLD(php_fpm_log_dir) - PHP_SUBST_OLD(php_fpm_pid_dir) PHP_SUBST_OLD(php_fpm_user) PHP_SUBST_OLD(php_fpm_group) AC_DEFINE_UNQUOTED(PHP_FPM_VERSION, "$fpm_version", [fpm version]) - AC_DEFINE_UNQUOTED(PHP_FPM_LOG_PATH, "$php_fpm_log_path", [fpm log file path]) - AC_DEFINE_UNQUOTED(PHP_FPM_PID_PATH, "$php_fpm_pid_path", [fpm pid file path]) AC_DEFINE_UNQUOTED(PHP_FPM_USER, "$php_fpm_user", [fpm user name]) AC_DEFINE_UNQUOTED(PHP_FPM_GROUP, "$php_fpm_group", [fpm group name]) -]) - -dnl }}} - - -AC_MSG_CHECKING(for FPM build) -if test "$PHP_FPM" != "no"; then - AC_MSG_RESULT($PHP_FPM) - - AC_LIB_EVENT([$minimum_libevent_version]) - - AC_CACHE_CHECK(whether libevent build works, php_cv_libevent_build_works, [ - PHP_TEST_BUILD(event_init, - [ - php_cv_libevent_build_works=yes - ], [ - AC_MSG_RESULT(no) - AC_MSG_ERROR([build test failed. Please check the config.log for details.]) - ], $LIBEVENT_LIBS) - ]) - - PHP_SETUP_LIBXML(FPM_SHARED_LIBADD, [ - ], [ - AC_MSG_ERROR([xml2-config not found. Please check your libxml2 installation.]) - ]) - - AC_FPM_CHECKS - AC_FPM_ARGS - AC_FPM_VARS - PHP_OUTPUT(sapi/fpm/php-fpm.conf sapi/fpm/init.d.php-fpm sapi/fpm/php-fpm.1) PHP_ADD_MAKEFILE_FRAGMENT([$abs_srcdir/sapi/fpm/Makefile.frag], [$abs_srcdir/sapi/fpm], [sapi/fpm]) Modified: php/php-src/branches/PHP_5_3_FPM/sapi/fpm/fpm/fpm_conf.c =================================================================== --- php/php-src/branches/PHP_5_3_FPM/sapi/fpm/fpm/fpm_conf.c 2009-12-15 13:13:51 UTC (rev 292165) +++ php/php-src/branches/PHP_5_3_FPM/sapi/fpm/fpm/fpm_conf.c 2009-12-15 13:15:05 UTC (rev 292166) @@ -518,7 +518,11 @@ } if (!fpm_global_config.error_log) { - fpm_global_config.error_log = strdup(PHP_FPM_LOG_PATH); + char *tmp_log_path; + + spprintf(&tmp_log_path, 0, "%s/log/php-fpm.log", PHP_LOCALSTATEDIR); + fpm_global_config.error_log = strdup(tmp_log_path); + efree(tmp_log_path); } fpm_evaluate_full_path(&fpm_global_config.error_log); Modified: php/php-src/branches/PHP_5_3_FPM/sapi/fpm/php-fpm.conf.in =================================================================== --- php/php-src/branches/PHP_5_3_FPM/sapi/fpm/php-fpm.conf.in 2009-12-15 13:13:51 UTC (rev 292165) +++ php/php-src/branches/PHP_5_3_FPM/sapi/fpm/php-fpm.conf.in 2009-12-15 13:15:05 UTC (rev 292166) @@ -6,10 +6,10 @@ <section name="global_options"> Pid file - <value name="pid_file">@php_fpm_pid_path@</value> + <value name="pid_file">@EXPANDED_LOCALSTATEDIR@/run/php-fpm.pid</value> Error log file - <value name="error_log">@php_fpm_log_path@</value> + <value name="error_log">@EXPANDED_LOCALSTATEDIR@/log/php-fpm.log</value> Log level <value name="log_level">notice</value> @@ -110,7 +110,7 @@ <value name="request_slowlog_timeout">0s</value> The log file for slow requests - <value name="slowlog">@php_fpm_log_p...@.slow</value> + <value name="slowlog">@EXPANDED_LOCALSTATEDIR@/log/php-fpm.log.slow</value> Set open file desc rlimit <value name="rlimit_files">1024</value>
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php