Bug 25416 - Port the 'callable' example CM to Call.DRAFT
Summary: Port the 'callable' example CM to Call.DRAFT
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-glib (show other bugs)
Version: unspecified
Hardware: Other All
: high enhancement
Assignee: Simon McVittie
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/sm...
Whiteboard: trivial instareview needed
Keywords: patch
Depends on:
Reported: 2009-12-03 07:03 UTC by Simon McVittie
Modified: 2010-02-22 10:46 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Description Simon McVittie 2009-12-03 07:03:11 UTC
Bug #24936 introduces a new channel type, Call, which replaces StreamedMedia.

To do automated regression tests on telepathy-qt4 and telepathy-glib, it would be very useful to port this CM to implement the current draft of Call, well before it's considered stable enough to go in telepathy-spec.

Git branch smcv/from-the-future implements an extension library (libfuture-extensions.la), and copies the callable example CM in examples/future/call-cm for porting to Call. It doesn't yet actually *implement* Call (it's still in terms of StreamedMedia), but it does have the right generated-code boilerplate, and it compiles.

It might be useful to merge a prefix of my branch, so we have libfuture-extensions.la even before the example CM has been ported; setting the patch keyword accordingly.
Comment 1 Simon McVittie 2009-12-04 04:30:00 UTC
I like examples, and also code-generation voodoo, and also new spec. Taking this bug.
Comment 2 Simon McVittie 2009-12-14 09:51:13 UTC
The test coverage isn't wonderful, but I think this is ready for review, and if approved, ready for use in testing telepathy-qt4 Call API.
Comment 3 Simon McVittie 2010-02-22 02:03:33 UTC
This has been here for a while and I know people have been basing code/tests on it. Any chance of a review? Andre? Andres?
Comment 4 Simon McVittie 2010-02-22 10:11:06 UTC
Andre is happy with this so I'm merging it. It'll be in 0.11.0.
Comment 5 Simon McVittie 2010-02-22 10:39:31 UTC
Some trivial changes were needed to fix distcheck. Could someone insta-review http://git.collabora.co.uk/?p=user/smcv/telepathy-glib-smcv.git please?

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.