[In Progress] Port to TelepathyGlib

[tonadev]

Explanation

Fixes #13. This PR ports to TelepathyGLib.

Reason

The telepathy library does not have its bindings for Python 3, and porting Telepathy to its PyGObject binding is a pre requisite for the Port to Python 3 Project.

Test result

No error related to the port to TelepathyGLib.

tonadev@TDPC:~/Documents/Work/OpenSource/code_in/sugarlabs/cookie-search-activity$ sugar-activity

(sugar-activity:9813): Gtk-WARNING **: 13:31:59.407: Theme parsing error: gtk-widgets.css:16:32: The style property GtkExpander:expander-size is deprecated and shouldn't be used anymore. It will be removed in a future version

(sugar-activity:9813): Gtk-WARNING **: 13:31:59.407: Theme parsing error: gtk-widgets.css:17:35: The style property GtkExpander:expander-spacing is deprecated and shouldn't be used anymore. It will be removed in a future version

[rhl-bthr]

Thanks. Reviewed.
Looks good to me

[quozl]

Thanks. Reviewed. Looks fine. I'll be able to test on Monday (Australia).

[quozl]

Tested on Fedora 18. No new errors in logs. Game playing board state is mostly shared, but not fully; differences are;

• when a guess for empty is made, proximity indicators are only shown to the player that made the guess for empty,
• when a guess for not-empty is made, the cookie only appears on the player that made the guess for non-empty,
• when a win occurs, the array of smiley faces and the new game prompt is shown to the player that made the last move, but not the other player,
• after a new game begins, the proximity indicators from the previous game may still be present,
• if both players complete the game, and pick new game at different times, and one player makes a guess before the second player picks new game, the results are inconsistent.

Hope that helps!

[tonadev]

I am not working on this currently. Anyone who wishes to take this up, please feel free to do so.

[Aniket21mathur]

Tried to test it using rdesktop. Successfully made two instances,but not able to connect the two activities. Using Ubuntu 18.04. In the neighbourhood view, after hovering on other user I am not getting an option of "Invite to Cookie-search", but I am getting it for Chat. And if any error results,are they seen through the log activity?
@quozl ?

[Aniket21mathur]

Is this change necessary? Can this change might be a reason for the port not to work properly? @quozl @tonadev .
I have seen other ports and didn't noticed it in any.

[quozl]

self._shared_activity is in the GTK 2 toolkit sugar-toolkit, and is not in the GTK 3 toolkit sugar-toolkit-gtk3.

self.get_shared_activity is in both toolkits.

My guess is that the code hadn't been changed during the port to GTK 3, before the port to TelepathyGLib. If we're not sure if collaboration worked for the GTK 2 release of the activity, the way to be sure is to test that release.

[Aniket21mathur]

Yes. I also think that collaboration should be tested for GTK 2 first, It seems that there are not much changes from telepathy to TelepathyGlib for this activity, and the differences @quozl pointed out might be seen before telepathy port.

Game playing board state is mostly shared, but not fully; differences are;

Now seen a couple of instances where collaboration code is not properly ported to GTK 3 .

[quozl]

Yes, it is endemic. Root cause is missing-in-action activity maintainers, and well-meaning volunteers during GCI or students in GSoC who have concentrated on a GTK 3 port without fully understanding the collaboration feature, and reviewers with the same limited focus. Another cause is activities copied from other activities that do collaboration, without implementing collaboration in the new activity. Is why we re-worded the project idea this time, and @nswarup14 is doing "Maintain 25 activities" rather than "Port to GTK 3".

[quozl]

I don't think it is worth testing collaboration with this activity, given that there is so little shared state. I suggest testing Write and Chat first.

[Aniket21mathur]

Reviewed it again. Comments -

• telepathy is used in only SearchActivity.py as per the pull request.
• Only constants are imported from telepathy. I again compared the constants of TelepathyGLib and telepathy, their values match.
• I don't think the Port to TelepathyGLib provided by this pr breaks collaboration, their should be some thing broken before the port.
  Thanks!

[quozl]

Yes, the port is not yet needed because the activity does not yet work with collaboration. See also #18. If that issue can't be fixed, then the fragmentary collaboration should be stripped out.