SIGTERM is broken in CherryPy 3.0.1

Anonymous avatarAnonymous created an issue

Since 3.0.1, SIGTERM causes a backtrace instead of shutting down CherryPy. It seems that the patch for ticket #637 installs SIGHUP and SIGTERM handlers as bound methods, but SIGTERM() is not defined that way. (Notice how SIGHUP() has a 'self' argument while SIGTERM() does not.)

Reverting to CherryPy 3.0.0 make the problem go away.

Here is an excerpt from the backtrace:

Traceback (most recent call last):
  ...
  File "/usr/lib/python2.4/site-packages/CherryPy-3.0.1-py2.4.egg/cherrypy/_cpengine.py", line 93, in start
    self.block()
  File "/usr/lib/python2.4/site-packages/CherryPy-3.0.1-py2.4.egg/cherrypy/_cpengine.py", line 101, in block
    time.sleep(self.autoreload_frequency)
TypeError: SIGTERM() takes at most 2 arguments (3 given)

Reported by hdbsfeq02@sneakemail.com

Comments (2)

  1. Log in to comment
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.