Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.0.0

Request #16638 Possibility to add more signals
Submitted: 2009-09-24 13:46 UTC
From: leening Assigned: kvz
Status: Closed Package: System_Daemon (version 0.9.2)
PHP Version: 5.2.5 OS: Ubuntu Linux 9.04
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 8 + 10 = ?

 
 [2009-09-24 13:46 UTC] leening (Michiel van Leening)
Description: ------------ It would be nice to have the ability to add more signals Eg. right now only SIGUSR1 is defined, but SIGUSR2 would come in handy (or any other signal for that matter) Test script: --------------- System_Daemon::setSigHandler(SIGUSR2, "myActionTwo"); Expected result: ---------------- That the daemon responds to SIGUSR2 :-)

Comments

 [2009-09-24 15:56 UTC] kvz (Kevin van Zonneveld)
-Status: Open +Status: Feedback
Hi Michiel, Thanks for your request. Could you provide me a list of all the signals that you would like to see implemented? Then I can wrap them up in 0.9.3
 [2009-09-24 16:28 UTC] leening (Michiel van Leening)
Hi Kevin, Thanks for your speedy reply! I think all possible signals except unblockable ones, which i believe are KILL and STOP. That leaves us with: QUIT ILL TRAP BUS FPE SEGV USR2 STKFLT TSTP TTIN TTOU URG XCPU XFSZ VTALRM PROF WINCH POLL PWR SYS RTMIN RTMIN+1 RTMIN+2 RTMIN+3 RTMAX-3 RTMAX-2 RTMAX-1 RTMAX Please see "man kill" : http://unixhelp.ed.ac.uk/CGI/man-cgi?kill
 [2009-09-24 17:00 UTC] kvz (Kevin van Zonneveld)
-Status: Feedback +Status: Closed -Assigned To: +Assigned To: kvz
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. OK. took a hard look at http://gcov.php.net/PHP_5_3/lcov_html/pcntl/pcntl.c.gcov.php And added all SIGNALS to the System_Daemon script: http://github.com/kvz/system_daemon/commit/63eac05433375ad140636b8256c091d6f5cb62b9 Some are added conditionally (converted by ::start() ) as they may not be available in all PHP versions. Will be in published in 0.9.3. Until then you could replace the Daemon.php file by the raw version in GitHub.
 [2009-09-24 17:12 UTC] leening (Michiel van Leening)
Thanks!