Merge branch 'master' of git+ssh://dev.laptop.org/git/sugar

NEWS

@@ -1,3 +1,5 @@
+Snapshot 040c94d181
+
 * #2099 Tweak a color pair as requested by Walter. (marco)
 * Draw an invoker that is connected with the palette for toolbuttons. (benzea)
 * Fix traceback when reading in saved WPA2 network configs (dcbw)

sugar/activity/activity.py

@@ -305,6 +305,18 @@ class Activity(Window, gtk.Container):
     def _internal_jobject_error_cb(self, err):
         logging.debug("Error creating activity datastore object: %s" % err)
 
+    def get_activity_root(self):
+        """
+        Return the appropriate location in the fs where to store activity related
+        data that doesn't pertain to the current execution of the activity and
+        thus cannot go into the DataStore.
+        """
+        if os.environ.has_key('SUGAR_ACTIVITY_ROOT') and \
+           os.environ['SUGAR_ACTIVITY_ROOT']:
+            return os.environ['SUGAR_ACTIVITY_ROOT']
+        else:
+            return '/'
+
     def read_file(self, file_path):
         """
         Subclasses implement this method if they support resuming objects from

sugar/activity/activityfactoryservice.py

@@ -30,6 +30,7 @@ from sugar.activity.bundle import Bundle
 from sugar.activity import activityhandle
 from sugar import logger
 from sugar import _sugarext
+from sugar import env
 
 # Work around for dbus mutex locking issue
 gobject.threads_init()
@@ -156,6 +157,7 @@ def run(bundle_path):
     gtk.icon_theme_get_default().append_search_path(bundle.get_icons_path())
 
     os.environ['SUGAR_BUNDLE_PATH'] = bundle_path
+    os.environ['SUGAR_ACTIVITY_ROOT'] = env.get_profile_path(bundle.get_service_name())
 
     _sugarext.set_prgname(bundle.get_service_name())
     _sugarext.set_application_name(bundle.get_name())

sugar/datastore/datastore.py

@@ -140,14 +140,12 @@ class DSObject(object):
                 activityfactory.create_with_object_id(service_name, object_id)
 
     def destroy(self):
-        logging.debug('DSObject.destroy() file_path: %r.' % self._file_path)
         if self._destroyed:
             logging.warning('This DSObject has already been destroyed!.')
             import pdb;pdb.set_trace()
             return
         self._destroyed = True
         if self._file_path and self._owns_file:
-            logging.debug('Removing temp file: %r' % self._file_path)
             if os.path.isfile(self._file_path):
                 os.remove(self._file_path)
             self._owns_file = False