When you run Sugar with no activities installed, UpdateModel._bundles_to_check
is empty. Attempting to unconditionally pop this list results in an IndexError.
Instead, the updater should stop trying to update bundles when it determines
that it has no more bundles to check.

Signed-off-by: Michael Stone <mich...@laptop.org>
---
  extensions/cpsection/updater/model.py |    2 ++
  1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/extensions/cpsection/updater/model.py 
b/extensions/cpsection/updater/model.py
index f2de65b..c45dcd3 100755
--- a/extensions/cpsection/updater/model.py
+++ b/extensions/cpsection/updater/model.py
@@ -71,6 +71,8 @@ class UpdateModel(gobject.GObject):
          total = len(bundleregistry.get_registry())
          current = total - len(self._bundles_to_check)
  
+        if len(self._bundles_to_check) == 0:
+            return False
          bundle = self._bundles_to_check.pop()
          self.emit('progress', UpdateModel.ACTION_CHECKING, bundle.get_name(),
                    current, total)
-- 
1.7.1
_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to