After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 699571 - Remove PyGBindingWeakRef
Remove PyGBindingWeakRef
Status: RESOLVED FIXED
Product: pygobject
Classification: Bindings
Component: introspection
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Nobody's working on this now (help wanted and appreciated)
Python bindings maintainers
Depends on: 698018
Blocks: 690397
 
 
Reported: 2013-05-03 11:05 UTC by Simon Feltman
Modified: 2013-10-14 22:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Remove PyGObjectWeakRef now that g_binding_unbind exists (9.61 KB, patch)
2013-05-03 19:13 UTC, Simon Feltman
none Details | Review
Remove PyGObjectWeakRef now that g_binding_unbind exists (9.26 KB, patch)
2013-05-04 01:31 UTC, Simon Feltman
needs-work Details | Review
Remove PyGObjectWeakRef now that g_binding_unbind exists (9.26 KB, patch)
2013-07-03 15:37 UTC, Simon Feltman
committed Details | Review

Description Simon Feltman 2013-05-03 11:05:05 UTC
The addition of the "unbind" method on GBinding (g_binding_unbind) should allow us to remove this and rely on introspection to supply it.
Comment 1 Simon Feltman 2013-05-03 19:13:33 UTC
Created attachment 243241 [details] [review]
Remove PyGObjectWeakRef now that g_binding_unbind exists

Remove the static code for managing GBinding references now that
GLib has a method for clearing a binding.
Comment 2 Simon Feltman 2013-05-04 01:31:24 UTC
Created attachment 243266 [details] [review]
Remove PyGObjectWeakRef now that g_binding_unbind exists

Remove the static code for managing GBinding references now that
GLib has a method for clearing a binding.
Comment 3 Simon Feltman 2013-05-05 00:12:43 UTC
Comment on attachment 243266 [details] [review]
Remove PyGObjectWeakRef now that g_binding_unbind exists

Might need work depending on the outcome of bug 698018
Comment 4 Simon Feltman 2013-07-03 15:37:06 UTC
Created attachment 248324 [details] [review]
Remove PyGObjectWeakRef now that g_binding_unbind exists

Remove the static code for managing GBinding references now that
GLib has a method for clearing a binding.
Add safe backwards compatible Binding class in which "unbind"
raises when called more than once.
Comment 5 Martin Pitt 2013-07-25 13:24:00 UTC
Comment on attachment 248324 [details] [review]
Remove PyGObjectWeakRef now that g_binding_unbind exists

I understand this would bump the glib requirement to 2.37.x? Traditionally we have kept the development version of pygobject working on the stable glib/g-i (except new tests, which we skip with old g-i).

I won't veto if you want to push it now, but please consider the above.

Also, this needs to go with bumping the glib required version in configure.ac either way.
Comment 6 Simon Feltman 2013-10-14 22:23:39 UTC
Pushed now that we depend on GLib > 2.37.

Attachment 248324 [details] pushed as 799989a - Remove PyGObjectWeakRef now that g_binding_unbind exists