Use start_new_session in the webbrowser module rather than a python os.setsid preeexec_fn.
This commit is contained in:
parent
8121898ec5
commit
5aaa498bad
|
@ -228,15 +228,9 @@ class UnixBrowser(BaseBrowser):
|
||||||
else:
|
else:
|
||||||
# for TTY browsers, we need stdin/out
|
# for TTY browsers, we need stdin/out
|
||||||
inout = None
|
inout = None
|
||||||
# if possible, put browser in separate process group, so
|
|
||||||
# keyboard interrupts don't affect browser as well as Python
|
|
||||||
setsid = getattr(os, 'setsid', None)
|
|
||||||
if not setsid:
|
|
||||||
setsid = getattr(os, 'setpgrp', None)
|
|
||||||
|
|
||||||
p = subprocess.Popen(cmdline, close_fds=True, stdin=inout,
|
p = subprocess.Popen(cmdline, close_fds=True, stdin=inout,
|
||||||
stdout=(self.redirect_stdout and inout or None),
|
stdout=(self.redirect_stdout and inout or None),
|
||||||
stderr=inout, preexec_fn=setsid)
|
stderr=inout, start_new_session=True)
|
||||||
if remote:
|
if remote:
|
||||||
# wait five secons. If the subprocess is not finished, the
|
# wait five secons. If the subprocess is not finished, the
|
||||||
# remote invocation has (hopefully) started a new instance.
|
# remote invocation has (hopefully) started a new instance.
|
||||||
|
|
Loading…
Reference in New Issue