You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by jo...@apache.org on 2012/03/11 02:22:34 UTC
svn commit: r1299319 - in /subversion/trunk/tools/server-side/svnpubsub:
daemonize.py svnwcsub.py
Author: joes
Date: Sun Mar 11 01:22:34 2012
New Revision: 1299319
URL: http://svn.apache.org/viewvc?rev=1299319&view=rev
Log:
* trunk/tools/server-side/svnpubsub svnwcsub.py daemonize.py
need to reinstall default sighandlers in the second child (the actual daemon process)
Modified:
subversion/trunk/tools/server-side/svnpubsub/daemonize.py
subversion/trunk/tools/server-side/svnpubsub/svnwcsub.py
Modified: subversion/trunk/tools/server-side/svnpubsub/daemonize.py
URL: http://svn.apache.org/viewvc/subversion/trunk/tools/server-side/svnpubsub/daemonize.py?rev=1299319&r1=1299318&r2=1299319&view=diff
==============================================================================
--- subversion/trunk/tools/server-side/svnpubsub/daemonize.py (original)
+++ subversion/trunk/tools/server-side/svnpubsub/daemonize.py Sun Mar 11 01:22:34 2012
@@ -151,6 +151,9 @@ class Daemon(object):
raise ChildResumedIncorrectly()
# we're a deamon now. get rid of the final remnants of the parent.
+ # start by restoring default signal handlers
+ signal.signal(signal.SIGUSR1, signal.SIG_DFL)
+ signal.signal(signal.SIGCHLD, signal.SIG_DFL)
sys.stdout.flush()
sys.stderr.flush()
si = open('/dev/null', 'r')
Modified: subversion/trunk/tools/server-side/svnpubsub/svnwcsub.py
URL: http://svn.apache.org/viewvc/subversion/trunk/tools/server-side/svnpubsub/svnwcsub.py?rev=1299319&r1=1299318&r2=1299319&view=diff
==============================================================================
--- subversion/trunk/tools/server-side/svnpubsub/svnwcsub.py (original)
+++ subversion/trunk/tools/server-side/svnpubsub/svnwcsub.py Sun Mar 11 01:22:34 2012
@@ -29,7 +29,6 @@
# See svnwcsub.conf for more information on its contents.
#
-import signal
import subprocess
import threading
import sys
@@ -140,7 +139,6 @@ class BigDoEverythingClasss(object):
self.hostports.append((parsed.hostname, parsed.port))
def start(self):
- signal.signal(signal.SIGCHLD, signal.SIG_DFL)
for path, url in self.tracking.items():
# working copies auto-register with the BDEC when they are ready.
WorkingCopy(self, path, url)
Re: svn commit: r1299319 - in /subversion/trunk/tools/server-side/svnpubsub:
daemonize.py svnwcsub.py
Posted by Greg Stein <gs...@gmail.com>.
On Sat, Mar 10, 2012 at 20:22, <jo...@apache.org> wrote:
>...
> +++ subversion/trunk/tools/server-side/svnpubsub/daemonize.py Sun Mar 11 01:22:34 2012
> @@ -151,6 +151,9 @@ class Daemon(object):
> raise ChildResumedIncorrectly()
>
> # we're a deamon now. get rid of the final remnants of the parent.
> + # start by restoring default signal handlers
> + signal.signal(signal.SIGUSR1, signal.SIG_DFL)
> + signal.signal(signal.SIGCHLD, signal.SIG_DFL)
Thanks, Joe! Committed upstream:
http://code.google.com/p/gstein/source/detail?r=16
Cheers,
-g