Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-07 Thread Sascha Silbe
Excerpts from Marco Pesenti Gritti's message of Tue Sep 07 01:29:30 +0200 2010: So many reasons to move to introspection :) Then someone should get started on packaging the introspection stuff for Debian, Ubuntu and if necessary Fedora (the distributions supported by sugar-jhbuild). :-P ;)

Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-07 Thread Tomeu Vizoso
On Tue, Sep 7, 2010 at 13:53, Sascha Silbe sascha-ml-reply-to-201...@silbe.org wrote: Excerpts from Marco Pesenti Gritti's message of Tue Sep 07 01:29:30 +0200 2010: So many reasons to move to introspection :) Then someone should get started on packaging the introspection stuff for Debian,

Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-06 Thread Marco Pesenti Gritti
On Sun, Sep 5, 2010 at 3:44 PM, Dipankar Patro dipan...@seeta.in wrote: Hi, In reference to ticket #2290 (http://bugs.sugarlabs.org/ticket/2290) I have uploaded a patch (have attached it too). Problem: in the register procedure the time out of connection was not implemented, for an

Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-06 Thread Tomeu Vizoso
On Mon, Sep 6, 2010 at 11:51, Marco Pesenti Gritti ma...@marcopg.org wrote: On Sun, Sep 5, 2010 at 3:44 PM, Dipankar Patro dipan...@seeta.in wrote: Hi, In reference to ticket #2290 (http://bugs.sugarlabs.org/ticket/2290) I have uploaded a patch (have attached it too). Problem: in the

Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-06 Thread Dipankar Patro
Sorry for that duplicate bug. Missed out that lfaraone already filed the bug. I have attached the revised patch. (uploaded at bugs.sl.o too) * changed things (subject, description, etc) according to Sascha Silbe's suggestions. @ Marco : The default timeout is way too long (unable to find out

Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-06 Thread Sascha Silbe
Excerpts from Dipankar Patro's message of Mon Sep 06 19:04:04 +0200 2010: I have attached the revised patch. (uploaded at bugs.sl.o too) Please use git send-email to send patches so they appear as inline text rather than attachments. With many email clients it's hard to reply to text

Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-06 Thread Marco Pesenti Gritti
On Mon, Sep 6, 2010 at 6:04 PM, Dipankar Patro dipan...@seeta.in wrote: Sorry for that duplicate bug. Missed out that lfaraone already filed the bug. I have attached the revised patch. (uploaded at bugs.sl.o too) * changed things (subject, description, etc) according to Sascha Silbe's

Re: [Sugar-devel] Patch review request for ticket #2290

2010-09-06 Thread Marco Pesenti Gritti
On Mon, Sep 6, 2010 at 11:42 AM, Tomeu Vizoso to...@sugarlabs.org wrote: Previous discussion is actually in http://bugs.sugarlabs.org/ticket/2289 and not in #2290. Looks like we cannot do XML-RPC with GIO because it doesn't support POST for http requests. The right component in our stack

[Sugar-devel] Patch review request for ticket #2290

2010-09-05 Thread Dipankar Patro
Hi, In reference to ticket #2290 (http://bugs.sugarlabs.org/ticket/2290) I have uploaded a patch (have attached it too). Problem: in the register procedure the time out of connection was not implemented, for an unavailable server. Solution in Patch: - Have implemented a TimedOut connection