Use of deprecated dbus-glib function

Question

Use of deprecated dbus-glib function

PureTryOut opened this issue 3 years ago · comments

While porting this application to CMake I found out that it uses dbus_connection_setup_with_g_main (https://github.com/AsteroidOS/asteroid-settings/blob/master/volumecontrol.cpp#L98) which is deprecated according to https://dbus.freedesktop.org/doc/dbus-glib/dbus-glib-DBus-GLib-low-level.html#dbus-connection-setup-with-g-main.

It should probably be ported away from. It's also the only dependency this application has on dbus-glib from what I can see.

Bart Ribbers · Answer 1 · Fri May 07 2021 21:12:13 GMT+0800 (China Standard Time)

Actually, seems the entirety of dbus-glib is deprecated, according to the repo description on https://gitlab.freedesktop.org/dbus/dbus-glib.

beroset · Answer 2 · Fri Jul 15 2022 01:22:15 GMT+0800 (China Standard Time)

I've started working on this and believe I can finish it in a few days. My plan is to use the Qt D-Bus functions instead of these low level ones. There doesn't appear to be any reason we can't and preliminary testing shows it should also be shorter and easier to understand.

beroset · Answer 3 · Sun Jul 17 2022 05:15:39 GMT+0800 (China Standard Time)

I have something that works, but the code never gets a callback from the StepsUpdated D-Bus interface. Same problem as this: https://devel.sailfishos.narkive.com/iA2ByWoC/sailfish-listening-for-volume-change. Meanwhile, I'll check in draft code for your viewing pleasure. It has a workaround, but I'd like to get the callback working properly.

beroset · Answer 4 · Mon Jul 18 2022 09:27:32 GMT+0800 (China Standard Time)

I've done some experiments with pulseaudio-modules-nemo which is supposed to send this signal. I changed it to send only the currentStep value and then changed it to send no parameters at all (with the corresponding changes to the asteroid-settings code) but never once is the callback invoked. I'm stumped.