pikers
/
piker
3
0
Fork 0
Code Issues 5 Pull Requests 7 Packages Projects Releases Wiki Activity

wrap notification process in try catch and capture stderr data

cz_post_ftx
algorandpa 2022-12-04 21:13:33 +00:00
parent 209e1085ae
commit 6fa266e3e0
1 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
piker/ui/_notify.py
View File

@ -82,7 +82,6 @@ async def notify_from_ems_status_msg(
os.environ['DBUS_SESSION_BUS_ADDRESS'] = ( os.environ['DBUS_SESSION_BUS_ADDRESS'] = (
f'unix:path=/run/user/{_dbus_uid}/bus' f'unix:path=/run/user/{_dbus_uid}/bus'
) )
result = await trio.run_process( result = await trio.run_process(
[ [
'notify-send', 'notify-send',
@ -93,6 +92,17 @@ async def notify_from_ems_status_msg(
# TODO: add in standard fill/exec info that maybe we # TODO: add in standard fill/exec info that maybe we
# pack in a broker independent way? # pack in a broker independent way?
f"'{msg.pformat()}'", 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)