From 6fa266e3e090cdfba6dc26cf35ee392614d350cd Mon Sep 17 00:00:00 2001 From: algorandpa Date: Sun, 4 Dec 2022 21:13:33 +0000 Subject: [PATCH] wrap notification process in try catch and capture stderr data --- piker/ui/_notify.py | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/piker/ui/_notify.py b/piker/ui/_notify.py index c4804663..ee79c75d 100644 --- a/piker/ui/_notify.py +++ b/piker/ui/_notify.py @@ -82,7 +82,6 @@ async def notify_from_ems_status_msg( os.environ['DBUS_SESSION_BUS_ADDRESS'] = ( f'unix:path=/run/user/{_dbus_uid}/bus' ) - result = await trio.run_process( [ 'notify-send', @@ -93,6 +92,17 @@ async def notify_from_ems_status_msg( # TODO: add in standard fill/exec info that maybe we # pack in a broker independent way? f"'{msg.pformat()}'", - ], + ], + capture_stdout=True, + capture_stderr=True, + check=False, ) - log.runtime(result) + if result.returncode != 0: + log.warn(f'No notification daemon installed stderr: {result.stderr}') + + + log.runtime(result) + + + +