dasbus icon indicating copy to clipboard operation
dasbus copied to clipboard

Decrement the reference count on the invocation object in _method_callback

Open wdouglass opened this issue 3 years ago • 6 comments

The caller is responsible for doing this according to: https://docs.gtk.org/gio/callback.DBusInterfaceMethodCallFunc.html

this should fix #80

wdouglass avatar Feb 09 '22 17:02 wdouglass

Ok, fair enough. I've filed a bug against pygobject here (i've also referenced this patch, i hope that's OK) https://gitlab.gnome.org/GNOME/pygobject/-/issues/519

wdouglass avatar Feb 14 '22 13:02 wdouglass

unfortunately, this bug seems to just get kicked back and forth between glib and pygobject.

I'm gonna keep running with the workaround from this PR; i don't think either of those projects are going to bend

wdouglass avatar Feb 25 '22 13:02 wdouglass

As an update, i've written a PR for the pygobject library that also addresses this issue https://gitlab.gnome.org/GNOME/pygobject/-/merge_requests/219

wdouglass avatar Dec 20 '22 14:12 wdouglass

As an update, i've written a PR for the pygobject library that also addresses this issue https://gitlab.gnome.org/GNOME/pygobject/-/merge_requests/219

That is great! Hopefully, it will work out this time. If it gets blocked again please let me know. In that case, I would merge your hotfix for dasbus as a temporary workaround, because I would like to see this issue fixed.

poncovka avatar Feb 03 '23 11:02 poncovka

careful merging this though, because if/when the pygobject fix gets merged, you'll have a ref underflow for systems that have an updated pygobject...

wdouglass avatar Feb 03 '23 13:02 wdouglass

careful merging this though, because if/when the pygobject fix gets merged, you'll have a ref underflow for systems that have an updated pygobject...

Yes: if merged, a version of PyGObject known to have this bug should be pinned, say PyGObject<=3.42.2 (for which I have ample experimental evidence that this patch is working) or PyGObject<=3.43.1 (which I have not yet tested).

By the way, is there a reason for which PyGObject is not listed as a explicit dependency for dasbus?

miccoli avatar Feb 03 '23 16:02 miccoli