Bug 69604 - [1.0] Idle: port to tp-glib 0.99.1
Summary: [1.0] Idle: port to tp-glib 0.99.1
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: idle (show other bugs)
Version: unspecified
Hardware: Other All
: medium enhancement
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL: http://cgit.collabora.com/git/user/ca...
Whiteboard: r+
Keywords: patch
Depends on: 69703
Blocks: 69431
  Show dependency treegraph
 
Reported: 2013-09-20 13:20 UTC by Guillaume Desmottes
Modified: 2013-09-23 13:04 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Guillaume Desmottes 2013-09-20 13:20:47 UTC
http://cgit.collabora.com/git/user/cassidy/telepathy-idle/log/?h=next

I split the commits to make reviewing easier but we may want to merge it as a big single commit as "make check" only passes with the top one.
Comment 1 Simon McVittie 2013-09-20 17:01:21 UTC
(In reply to comment #0)
> http://cgit.collabora.com/git/user/cassidy/telepathy-idle/log/?h=next
> 
> I split the commits to make reviewing easier but we may want to merge it as
> a big single commit as "make check" only passes with the top one.

To be honest, bisecting is doomed in this situation. I'd be inclined to leave them split.

> constants: port to Telepathy 1.0

Any chance you could sync from <https://bugs.freedesktop.org/show_bug.cgi?id=69618#c4> first? It'd be a clearer diff afterwards, I think.

All the other commits look fine, but I'd like some of them in master so merges are less horrible:

> add missing telepathy-glib-dbus.h includes

This could go to master, with a comment noting that they're not all needed yet, but will be needed in Telepathy 1.0.

> extensions: include telepathy-glib.h in generated code

via master, please

> TpBaseConnectionClass->create_channel_factories has been removed

Could be done on master, with a comment about future-proofing

> s/NUM_TP_*/TP_NUM_*

master

> 	use GetContactByID instead of RequestHandles

master

> 	stop using RequestHandles when joining channels

master

> stop using GetInterfaces()

master

> NewChannel signal has been removed

could go via master with a different commit message

> stop using GetSelfHandle()

master

> stop using InspectHandles()

master

> room-list-channels: use CHANNEL_TYPE_ROOM_LIST

master
Comment 2 Guillaume Desmottes 2013-09-23 08:52:54 UTC
(In reply to comment #1)
> Any chance you could sync from
> <https://bugs.freedesktop.org/show_bug.cgi?id=69618#c4> first? It'd be a
> clearer diff afterwards, I think.

Sure. Can you please push the branch so I can easily steal the new file.

> All the other commits look fine, but I'd like some of them in master so
> merges are less horrible:

Ok, I'll do some rebasing.
Comment 3 Guillaume Desmottes 2013-09-23 12:02:57 UTC
http://cgit.collabora.com/git/user/cassidy/telepathy-idle/log/?h=next rebased on top of master and updated.
Comment 4 Simon McVittie 2013-09-23 12:55:40 UTC
Ship it!
Comment 5 Guillaume Desmottes 2013-09-23 13:04:51 UTC
Pushed to a 'next' branch. thanks for the reviews.


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.