Summary: | remove tp_channel_manager_emit_new_channels | ||
---|---|---|---|
Product: | Telepathy | Reporter: | Jonny Lamb <jonny.lamb> |
Component: | tp-glib | Assignee: | Telepathy bugs list <telepathy-bugs> |
Status: | RESOLVED FIXED | QA Contact: | Telepathy bugs list <telepathy-bugs> |
Severity: | normal | ||
Priority: | medium | CC: | xclaesse |
Version: | git master | Keywords: | patch |
Hardware: | Other | ||
OS: | All | ||
URL: | http://cgit.freedesktop.org/~jonny/telepathy-glib/commit/?h=next-new-channels-47942 | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Bug Depends on: | 47943 | ||
Bug Blocks: | 31668 |
Description
Jonny Lamb
2012-03-27 07:50:20 UTC
Here's a patch to deprecate it in master. Once merged I'll merge master into next and remove it there. What do you think? +1 Great, thanks. I pushed this master, then merged master into next. Here's the branch for next to remove the function. +1 Please fix the CMs too, I see gabble at least still use it Merged, thanks. gabble and salut still use this for announcing Text MUC channels & Tube & Tubes channels together. I have a gabble branch to remove Tubes and fix this case so emit_new_channels isn't called. This should probably be the start of the gabble 'next' branch. I've yet to do the same thing in salut. We will need to add some kind of workaround to gabble and salut master so it doesn't call the deprecated emit_new_channels. |
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.