[Dextrose] Problem with "Software Update" feature

[Anish Mangal]

Hi,

For some reason, this mail skipped my inbox. Apologies for the delay
in response.

On Fri, Oct 22, 2010 at 7:51 PM, Bernie Innocenti <bernie at codewiz.org> wrote:
> On Thu, 2010-10-21 at 15:20 -0200, Daniel Castelo wrote:
>> Hi!
>>
>> We found some problems in the feature "Software Update" that uses the
>> microformat lenguage. I notice that now is using
>> 'http://wiki.paraguayeduca.org/index.php/Actividades_Dextrose_1', we
>> are planning to have our own version of this page.
>
> Oh, yeah... that was done in a real hurry. Of course we should make the
> URL configurable with a gconf key, and set it from a custom script in
> the Dextrose ini file.
>
> Can you or Esteban work on a patch? Even better, we could improve the
> original patch by Anish without creating a second patch on top of it.
>

Apart from the problems highlighted in this mail-chain, here are two more

1. Parts of it are sync, i.e. the display/ui may hang/appear-inactive
during long http requests.
2. Not tested thoroughly, atleast by me.

If its ok, I'd like to work on it. I had separately tested portions of
code, which fix the above problems, so the groundwork is laid,
compositing and cleaning remains.

>
> I'll let Anish comment on the following two problems:
>
>> 1) If the computer is not connected to Internet the user didn't
>> receive any messages and the window get stuck.
>
>> This is the log:
>>
>> 1287683991.599589 WARNING root: No gtk.AccelGroup in the top level
>> window.
>> 1287683991.611232 WARNING root: No gtk.AccelGroup in the top level
>> window.
>> 1287683991.622605 WARNING root: No gtk.AccelGroup in the top level
>> window.
>> 1287683991.635661 WARNING root: No gtk.AccelGroup in the top level
>> window.
>> /usr/lib/python2.6/site-packages/dbus/connection.py:242:
>> DeprecationWarning: object.__init__() takes no parameters
>>   super(Connection, self).__init__(*args, **kwargs)
>> Traceback (most recent call last):
>>   File
>> "/usr/share/sugar/extensions/cpsection/updater/backends/microformat.py", line 59, in __download_file_complete_cb
>>     content = self._file.load_contents_finish(result)[0]
>> glib.GError: Cannot resolve hostname
>>
>>
>> 2) If I try to update a set of activities I get this error:
>>
>> 1287684870.693166 WARNING root: No gtk.AccelGroup in the top level
>> window.
>> 1287684870.694456 WARNING root: No gtk.AccelGroup in the top level
>> window.
>> Traceback (most recent call last):
>>   File "/usr/share/sugar/extensions/cpsection/updater/model.py", line
>> 301, in __write_async_cb
>>     progress = self._downloaded_size / float(self.bundle_update.size)
>> ZeroDivisionError: float division
>> Traceback (most recent call last):
>

Hmm, I checked the patch in the Dextrose repo, and it is not _latest_
when compared with the patch/code I locally have. Some more digging
inside the inbox, and I only mailed the patch to Bernie (which he told
me to fwd to the list and I didn't; sorry). Both these problems are
fixed in the code I currently have (and mailed to bernie, tch).
Further, the new patch also installs those activities which were not
previously present in the system.

>
>> Seems like this feature has never worked in dextrose, could be?
>
> We have tested it in Paraguay with just 2 activities and it was working
> fine. Perhaps Jorge could repeat the test now and comment?
>
> --
>   // Bernie Innocenti - http://codewiz.org/
>  \X/  Sugar Labs       - http://sugarlabs.org/
>
>

-- 
Anish Mangal
anish at sugarlabs.org

P.S. Also, this might be a good place and time to suggest a minor
enhancement to the microformat definition, viz. the addition of an
olpc-activity-size tag. It would save us a lot of redundant http
requests, and time.