Mission Control has a bunch of bonus interfaces. Each of them should either be part of telepathy-spec, or not exist. A good starting point would be "sync them with the corresponding draft in telepathy-spec"...
Created attachment 86177 [details] [review] [MC master] Update extension interfaces to their telepathy-spec 0.27.1 versions No functional changes. The remaining interfaces (A.I.Conditions, CD.I.Messages) are still not in telepathy-spec at all. :-(
A.I.Hidden, AM.I.Hidden: Bug #33101 CD.I.Messages: Bug #37380 A.I.ExternalPasswordStorage, CM.I.AccountStorage: Bug #33485 (but should be undrafted, unless we're just going to delete it) A.I.Conditions: Bug #24896
Comment on attachment 86177 [details] [review] [MC master] Update extension interfaces to their telepathy-spec 0.27.1 versions I merged this.
Created attachment 87472 [details] [review] [MC master] Remove Conditions interface I was never sure about the design, and the supporting code was removed when I refactored connectivity for 5.16. If it's needed in future, we can bring it back, hopefully with a simpler design. --- Bringing it back is Bug #24896, which already has a simpler design (that was never implemented).
Comment on attachment 87472 [details] [review] [MC master] Remove Conditions interface Review of attachment 87472 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 87472 [details] [review] [MC master] Remove Conditions interface Deleted for 5.17.
Please review this first: https://bugs.freedesktop.org/show_bug.cgi?id=69446#c3
Created attachment 93056 [details] [review] Remove A.I.ExternalPasswordStorage, CM.I.AccountStorage Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33485 Bug: https://bugs.freedesktop.org/show_bug.cgi?id=44512 Bug: https://bugs.freedesktop.org/show_bug.cgi?id=69600
Created attachment 93057 [details] [review] Remove code-generation machinery --- This seems as good a way as any to assert that we don't have any extension interfaces :-P
Comment on attachment 93056 [details] [review] Remove A.I.ExternalPasswordStorage, CM.I.AccountStorage Review of attachment 93056 [details] [review]: ----------------------------------------------------------------- ++
Comment on attachment 93057 [details] [review] Remove code-generation machinery Review of attachment 93057 [details] [review]: ----------------------------------------------------------------- ++
No more extension interfaces! Fixed in git for 5.17.0
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.