Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

oslayer/log_dbus: use libdbus directly #1496

Merged

Conversation

benoit-pierre
Copy link
Member

@benoit-pierre benoit-pierre commented Mar 1, 2022

Summary of changes

  • remove dependency on the dbus-python
  • notification messages are now sent asynchronously
  • work even if the notifications service does not support type introspection
  • more robust: if the notification service goes down, calls to the handler won't raise an exception

Close #1490.

Pull Request Checklist

- remove dependency on the `dbus-python`
- notification messages are now sent asynchronously
- work even if the notifications service does not support type introspection
@benoit-pierre benoit-pierre force-pushed the log_dbus_with_libdbus branch from 10bf360 to e0a0378 Compare May 14, 2022 22:32
@benoit-pierre benoit-pierre merged commit ca5cf77 into openstenoproject:master May 14, 2022
@benoit-pierre benoit-pierre deleted the log_dbus_with_libdbus branch May 14, 2022 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

As non-root user, serial port is not found and is None
1 participant