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

Preferred interface #1

Open
rfjakob opened this issue Apr 4, 2020 · 4 comments
Open

Preferred interface #1

rfjakob opened this issue Apr 4, 2020 · 4 comments

Comments

@rfjakob
Copy link

rfjakob commented Apr 4, 2020

Hi! earlyoom is looking for a way out of mess that are gui notifications from a service. I found https://wiki.debianforum.de/Desktop-Notification_von_Systemservice_mittels_dbus and now this repo.

I have started a C implementation of basically the same thing you have done in Python: https://github.com/rfjakob/systembus-notify

However, I chose the D-Bus names independently (net.nuetzlich.SystemNotifications), which is stupid, as now the both are not compatible. Your project was here before, so I would try to make systembus-notify compatible to at.xundeenergie.notifications.

What do you think? Regards from Vienna.

@TiZ-HugLife
Copy link

Why not just handle both interfaces? I feel like I'll be needing system notifications for some of my scripts, so I was about to make a script that both handles earlyoom's notifications at net.nuetzlich.SystemNotifications, but also handles the more advanced functionality from at.xundeenergie.notifications.

@rfjakob
Copy link
Author

rfjakob commented Aug 28, 2021

You mean that systembus-notify could handle both?

Yes, that would be nice. However, it seems that this project already has three different interfaces, and I'm not sure which one is preferred:

  1. /at/xundeenergie/notifications
  2. /at/xundeenergie/notifications/simple
  3. /at/xundeenergie/notifications/advanced

@xundeenergie
Copy link
Owner

it depends, which Interfaces you define, on which the service should listen to...

@TiZ-HugLife
Copy link

TiZ-HugLife commented Aug 28, 2021

I couldn't figure out this project's code at all, so I just ignored this one's interfaces in favor of @rfjakob's interface and my own custom one: https://github.com/HugLifeTiZ/configs-scripts/blob/master/scripts/system-notify

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

No branches or pull requests

3 participants