Merge lp:~om26er/ubuntu/maverick/telepathy-haze/telepathy-haze-fix-652944 into lp:ubuntu/maverick/telepathy-haze

Proposed by Omer Akram
Status: Merged
Merged at revision: 15
Proposed branch: lp:~om26er/ubuntu/maverick/telepathy-haze/telepathy-haze-fix-652944
Merge into: lp:ubuntu/maverick/telepathy-haze
Diff against target: 59 lines (+34/-1)
3 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/patches/do_not_show_webcam_for_haze.patch (+25/-0)
To merge this branch: bzr merge lp:~om26er/ubuntu/maverick/telepathy-haze/telepathy-haze-fix-652944
Reviewer Review Type Date Requested Status
Martin Pitt Approve
Artur Rona (community) Needs Fixing
Review via email: mp+41597@code.launchpad.net

Description of the change

backported from upstream git http://git.collabora.co.uk/?p=telepathy-haze.git;a=commitdiff;h=924fa3d90eb1a4949e431b0bd6284071f0d5b38b so that people dont get tricked into believing yahoo, icq etc support a/v calls

To post a comment you must log in.
17. By Omer Akram

debian/patches/do_not_show_webcam_for_haze.patch:

Revision history for this message
Artur Rona (ari-tczew) wrote :

Could you complete your patch with DEP3 tags? Follow with https://wiki.ubuntu.com/UbuntuDevelopment/PatchTaggingGuidelines.

review: Needs Fixing
18. By Omer Akram

added DEP3 tags to the patch.

Revision history for this message
Omer Akram (om26er) wrote :

> Could you complete your patch with DEP3 tags? Follow with
> https://wiki.ubuntu.com/UbuntuDevelopment/PatchTaggingGuidelines.

yes sir done ;)

Revision history for this message
Artur Rona (ari-tczew) wrote :

Please use short URLs, so:
Bug: https://bugs.freedesktop.org/30594
Bug-Ubuntu: https://launchpad.net/bugs/652944

and full sentences, so:
Subject: Instantiate the media manager only if the protocol support calls.

review: Needs Fixing
19. By Omer Akram

more corrections

Revision history for this message
Omer Akram (om26er) wrote :

> Please use short URLs, so:
> Bug: https://bugs.freedesktop.org/30594
> Bug-Ubuntu: https://launchpad.net/bugs/652944
>
> and full sentences, so:
> Subject: Instantiate the media manager only if the protocol support calls.

sorry for the inconvenience i hope this time things are right.

20. By Omer Akram

put a '.' at the end of Subject just in case

Revision history for this message
Omer Akram (om26er) wrote :

now i think this branch is ready :)

Revision history for this message
Martin Pitt (pitti) wrote :

This looks good now, thanks Omer! I'll merge into the maverick-proposed branch and upload.

review: Approve
Revision history for this message
Martin Pitt (pitti) wrote :

OK, nevermind about the maverick-proposed branch; it doesn't exist, and apparently I can't create it either, so just using the maverick branch.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-08-05 16:22:55 +0000
3+++ debian/changelog 2010-11-25 17:12:46 +0000
4@@ -1,3 +1,10 @@
5+telepathy-haze (0.4.0-1ubuntu1) maverick-proposed; urgency=low
6+
7+ * debian/patches/do_not_show_webcam_for_haze.patch:
8+ - instantiate the media manager only if the protocol support calls (LP: #652944)
9+
10+ -- Omer Akram <om26er@ubuntu.com> Tue, 23 Nov 2010 18:37:55 +0500
11+
12 telepathy-haze (0.4.0-1) unstable; urgency=low
13
14 * New upstream release
15
16=== modified file 'debian/control'
17--- debian/control 2010-08-05 16:22:55 +0000
18+++ debian/control 2010-11-25 17:12:46 +0000
19@@ -1,7 +1,8 @@
20 Source: telepathy-haze
21 Section: net
22 Priority: optional
23-Maintainer: Debian Telepathy maintainers <pkg-telepathy-maintainers@lists.alioth.debian.org>
24+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
25+XSBC-Original-Maintainer: Debian Telepathy maintainers <pkg-telepathy-maintainers@lists.alioth.debian.org>
26 Uploaders: Laurent Bigonville <bigon@debian.org>,
27 Simon McVittie <smcv@debian.org>,
28 Jonny Lamb <jonny@debian.org>
29
30=== added directory 'debian/patches'
31=== added file 'debian/patches/do_not_show_webcam_for_haze.patch'
32--- debian/patches/do_not_show_webcam_for_haze.patch 1970-01-01 00:00:00 +0000
33+++ debian/patches/do_not_show_webcam_for_haze.patch 2010-11-25 17:12:46 +0000
34@@ -0,0 +1,25 @@
35+From: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
36+Subject: Instantiate the media manager only if the protocol support calls.
37+Origin: http://git.collabora.co.uk/?p=telepathy-haze.git;a=commitdiff;h=924fa3d90eb1a4949e431b0bd6284071f0d5b38b
38+Bug: https://bugs.freedesktop.org/30594
39+Bug-Ubuntu: https://launchpad.net/bugs/652944
40+
41+--- a/src/connection.c
42++++ b/src/connection.c
43+@@ -446,9 +446,13 @@ _haze_connection_create_channel_managers (TpBaseConnection *base)
44+ g_ptr_array_add (channel_managers, self->im_factory);
45+
46+ #ifdef ENABLE_MEDIA
47+- self->media_manager = HAZE_MEDIA_MANAGER (
48+- g_object_new (HAZE_TYPE_MEDIA_MANAGER, "connection", self, NULL));
49+- g_ptr_array_add (channel_managers, self->media_manager);
50++ /* Instantiate the media manager only if the protocol support calls */
51++ if (PURPLE_PROTOCOL_PLUGIN_HAS_FUNC (self->priv->prpl_info, initiate_media))
52++ {
53++ self->media_manager = HAZE_MEDIA_MANAGER (
54++ g_object_new (HAZE_TYPE_MEDIA_MANAGER, "connection", self, NULL));
55++ g_ptr_array_add (channel_managers, self->media_manager);
56++ }
57+ #endif
58+
59+ self->contact_list = HAZE_CONTACT_LIST (

Subscribers

People subscribed via source and target branches

to all changes: