diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2020-06-08 07:37:21 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2020-06-08 08:50:54 +0200 |
| commit | 7b379661e6b99d103411a21b256a38fdc5357961 (patch) | |
| tree | 395114306ca501f66288ab7e5e2027bca935df86 /uitest | |
| parent | 1e5cf1b3ebf39aee01ed2a3bfd5be7ee6f44d18e (diff) | |
Reliably terminate soffice.bin when terminating soffice
d58a3e5f85d44efc9d21dd56329a3ef20cb3692f "Reliably terminate soffice when
connect throws an exception" had been a partial thinko: When kill()'ing (i.e.,
sending SIGKILL on POSIX) soffice (aka oosplash exec'ed from it), the
soffice.bin forked from it can still keep on running.
So terminate() (i.e., send SIGTERM on POSIX) soffice aka ooslpash, and in its
sigterm_handler actually wait for the soffice.bin process. (Where the latter
should just be a saftey measure. At least on POSIX, osl_terminateProcess as
called from the oosplash sigterm_handler sends a SIGKILL, so should swiftly and
reliably kill the soffice.bin process.)
Change-Id: Ifd6930691dce34894e76c417e152111983d62b8c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95779
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'uitest')
| -rw-r--r-- | uitest/libreoffice/connection.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/uitest/libreoffice/connection.py b/uitest/libreoffice/connection.py index 9de92e77312f..68330b47d6cf 100644 --- a/uitest/libreoffice/connection.py +++ b/uitest/libreoffice/connection.py @@ -57,7 +57,7 @@ class OfficeConnection: success = True finally: if not success and self.soffice: - self.soffice.kill() + self.soffice.terminate() self.soffice.wait() self.soffice = None |