Summary: | Haze: update 'next' branch to spec/tp-glib 0.99.1 | ||
---|---|---|---|
Product: | Telepathy | Reporter: | Simon McVittie <smcv> |
Component: | haze | Assignee: | Simon McVittie <smcv> |
Status: | RESOLVED FIXED | QA Contact: | Telepathy bugs list <telepathy-bugs> |
Severity: | normal | ||
Priority: | medium | Keywords: | patch |
Version: | git master | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Bug Depends on: | |||
Bug Blocks: | 69431 | ||
Attachments: | Text tests: use the Messages interface |
Description
Simon McVittie
2013-09-24 18:34:46 UTC
(In reply to comment #0) > The branch says it all. > > > Use the "undrafted" version of MailNotification > > This should happen in master too (the only mechanical difference seems to be > the nonexistence of Subscribe(), which is a no-op in Haze anyway) but since > Haze already has an outdated 'next' branch which doesn't compile, I think > making it compile again > perfection. I'll do the change on master too, and > merge it or something. That's bug #69468 btw. Patches look good, including the gabbletest change for Gabble as well. All merged, thanks. One more... Created attachment 86560 [details] [review] Text tests: use the Messages interface This makes them more like the ones in 'next', which use the newly unified Text interface (whose API is mostly based on Messages). Comment on attachment 86560 [details] [review] Text tests: use the Messages interface Review of attachment 86560 [details] [review]: ----------------------------------------------------------------- ++ Fixed, thanks. |
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.