Edit report at https://bugs.php.net/bug.php?id=63085&edit=1

 ID:                 63085
 Patch added by:     r...@php.net
 Reported by:        r...@php.net
 Summary:            Systemd integration and daemonize
 Status:             Open
 Type:               Feature/Change Request
 Package:            FPM related
 Operating System:   GNU/Linux (Fedora 17)
 PHP Version:        5.4.7
 Block user comment: N
 Private report:     N

 New Comment:

The following patch has been added/updated:

Patch Name: php-fpm-systemd.patch
Revision:   1348066817
URL:        
https://bugs.php.net/patch-display.php?bug=63085&patch=php-fpm-systemd.patch&revision=1348066817


Previous Comments:
------------------------------------------------------------------------
[2012-09-14 05:43:02] r...@php.net

I have join the systemd unit file used in Fedora.

Note : "type=Forking" is not present, 
so this rely on daemonize=no in config file.

It could be great to have this unit file (made more generic if needed) provided 
in php distribution.

------------------------------------------------------------------------
[2012-09-14 05:41:08] r...@php.net

The following patch has been added/updated:

Patch Name: php-fpm.service
Revision:   1347601268
URL:        
https://bugs.php.net/patch-display.php?bug=63085&patch=php-fpm.service&revision=1347601268

------------------------------------------------------------------------
[2012-09-14 05:40:12] r...@php.net

Description:
------------
Currently "daemonize" is a option taken from configuration file.

I think is is quite ugly to have a starting script of systemd unit file to rely 
on a external configuration file.

I think a command line option will be more robust.

For ex, 
- in init script we could launch php-fpm --daemonize
- in systemd unit, we could launch php-fpm --nodaemonize

Without command line option, I propose to keep reading option from the config 
file (to not break existing configuration).

We could also imagine another option (--debug) which run in foreground and log 
everything need thing to screen (for interactive run).

What do you think ?
I could propose a patch



------------------------------------------------------------------------



-- 
Edit this bug report at https://bugs.php.net/bug.php?id=63085&edit=1

Reply via email to