[Sugar-devel] [write PATCH] sl#4436: Making collaboration work again.

Ajay Garg ajay at activitycentral.com
Wed Feb 27 04:15:23 EST 2013


Well, I compared the logs, and they appeared to be same :)


I re-tested on build 30, with the same setup ::

   Write-86.2 (after applying this patch) on

    * build 30 for XO-4      (as the "inviter")
    * build 30 for XO-1.75 (as the  "joiner")

and if one waits for about 5 seconds before starting to write/do-anything
in either of the collaborated canvas, collaboration "does" work.


Gonzalo,
I  will be grateful if you try once again :)


Again, I am extremely sorry to Gonzalo to others, for the minor hiccup.



On Wed, Feb 27, 2013 at 10:58 AM, Ajay Garg <ajay at activitycentral.com>wrote:

>
>
> On Wed, Feb 27, 2013 at 5:05 AM, Gonzalo Odiard <gonzalo at laptop.org>wrote:
>
>> Thanks Ajay.
>> I have a doubt. Testing after applying the patch,
>> can start the activity and join other instance, but can't see the changes
>> from one xo in the other. Can you?
>>
>
> a)
> I tested with bundle-86.2 (after applying this patch) on
>
>     * build 30 for XO-4      (as the "inviter")
>     * build 30 for XO-1.75 (as the  "joiner")
>
> Collaboration didn't work :(
>
>
>
> b)
> I then re-tested with the SAME bundle, on
>
>     * DX4 build 14 on XO-4      (as the "inviter")
>     * DX4 build 14 on XO-1.75 (as the "joiner")
>
> Collaboration worked fine (I tested with sharing text, and image).
>
>
> There seem to be some platform-exceptions on build 30 in the failure
> cases, most probably as a result of http://bugs.sugarlabs.org/ticket/2955.
> I will try and confirm, and see if I can make collaboration work on the
> build 30.
>
>
>
>
>
>
>>
>> Gonzalo
>>
>> On Thu, Feb 21, 2013 at 8:53 AM, Ajay Garg <ajay at activitycentral.com>wrote:
>>
>>> This stopped working (for "Write"), since the time
>>>
>>> http://git.sugarlabs.org/sugar-toolkit-gtk3/sugar-toolkit-gtk3/commit/70cee44717d46129419992cd6a7e15472dc1f568
>>> was pushed.
>>>
>>> Signed-off-by: Ajay Garg <ajay at activitycentral.com>
>>> ---
>>>  AbiWordActivity.py     | 21 ++++++++++-----------
>>>  1 files changed, 10 insertions(+), 11 deletions(-)
>>>
>>> diff --git a/AbiWordActivity.py b/AbiWordActivity.py
>>> index 35b248c..e95718e 100644
>>> --- a/AbiWordActivity.py
>>> +++ b/AbiWordActivity.py
>>> @@ -67,7 +67,6 @@ class AbiWordActivity(activity.Activity):
>>>
>>>          # create our main abiword canvas
>>>          self.abiword_canvas = DocumentView()
>>> -        self._shared_activity = None
>>>          self._new_instance = True
>>>          toolbar_box = ToolbarBox()
>>>
>>> @@ -182,13 +181,13 @@ class AbiWordActivity(activity.Activity):
>>>
>>>          self.connect('shared', self._shared_cb)
>>>
>>> -        if self._shared_activity:
>>> +        if self.shared_activity:
>>>              # we are joining the activity
>>>              logger.error('We are joining an activity')
>>>              self.connect('joined', self._joined_cb)
>>> -            self._shared_activity.connect('buddy-joined',
>>> +            self.shared_activity.connect('buddy-joined',
>>>                      self._buddy_joined_cb)
>>> -            self._shared_activity.connect('buddy-left',
>>> self._buddy_left_cb)
>>> +            self.shared_activity.connect('buddy-left',
>>> self._buddy_left_cb)
>>>              if self.get_shared():
>>>  #                # oh, OK, we've already joined
>>>                  self._joined_cb()
>>> @@ -236,8 +235,8 @@ class AbiWordActivity(activity.Activity):
>>>          logger.error('My Write activity was shared')
>>>          self._sharing_setup()
>>>
>>> -        self._shared_activity.connect('buddy-joined',
>>> self._buddy_joined_cb)
>>> -        self._shared_activity.connect('buddy-left', self._buddy_left_cb)
>>> +        self.shared_activity.connect('buddy-joined',
>>> self._buddy_joined_cb)
>>> +        self.shared_activity.connect('buddy-left', self._buddy_left_cb)
>>>
>>>          channel = self.tubes_chan[telepathy.CHANNEL_TYPE_TUBES]
>>>          logger.error('This is my activity: offering a tube...')
>>> @@ -247,13 +246,13 @@ class AbiWordActivity(activity.Activity):
>>>      def _sharing_setup(self):
>>>          logger.debug("_sharing_setup()")
>>>
>>> -        if self._shared_activity is None:
>>> +        if self.shared_activity is None:
>>>              logger.error('Failed to share or join activity')
>>>              return
>>>
>>> -        self.conn = self._shared_activity.telepathy_conn
>>> -        self.tubes_chan = self._shared_activity.telepathy_tubes_chan
>>> -        self.text_chan = self._shared_activity.telepathy_text_chan
>>> +        self.conn = self.shared_activity.telepathy_conn
>>> +        self.tubes_chan = self.shared_activity.telepathy_tubes_chan
>>> +        self.text_chan = self.shared_activity.telepathy_text_chan
>>>          self.tube_id = None
>>>          self.tubes_chan[telepathy.CHANNEL_TYPE_TUBES].connect_to_signal(
>>>                  'NewTube', self._new_tube_cb)
>>> @@ -267,7 +266,7 @@ class AbiWordActivity(activity.Activity):
>>>
>>>      def _joined_cb(self, activity):
>>>          logger.error("_joined_cb()")
>>> -        if not self._shared_activity:
>>> +        if not self.shared_activity:
>>>              return
>>>
>>>          self.joined = True
>>> --
>>> 1.7.11.7
>>>
>>> _______________________________________________
>>> Sugar-devel mailing list
>>> Sugar-devel at lists.sugarlabs.org
>>> http://lists.sugarlabs.org/listinfo/sugar-devel
>>>
>>
>>
>
>
> --
> Regards,
>
> Ajay Garg
> Dextrose Developer
> Activity Central: http://activitycentral.com




-- 
Regards,

Ajay Garg
Dextrose Developer
Activity Central: http://activitycentral.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20130227/2d5a052d/attachment-0001.html>


More information about the Sugar-devel mailing list