Merge lp:~gary-lasker/software-center/fix-crash-lp960742 into lp:software-center

[Gary Lasker]

Proposed fix for crasher bug 960742. Please see the bug report for details and the stacktrace.

Thanks!

[Michael Vogt (mvo)]

Thanks, looks good. I guess this is caused by a race condition when multiple expunge-cache processes run, but that should be fine, i.e. I don't think we need locking here.

review: Approve

=== modified file 'utils/expunge-cache.py'
--- utils/expunge-cache.py	2012-03-15 10:43:13 +0000
+++ utils/expunge-cache.py	2012-03-22 01:33:30 +0000
@@ -25,7 +25,11 @@
             print "Would delete: %s" % f
         else:
             logging.debug("Deleting: %s" % f)
-            os.unlink(f)
+            try:
+                os.unlink(f)
+            except OSError as e:
+                logging.warn("When expunging the cache, could not unlink "
+                             "file '%s' (%s)'" % (f, e))
 
     def clean(self):
         # go over the directories