[CalendarServer-changes] [7481] CalendarServer/branches/users/glyph/new-export/calendarserver/tools/ export.py

source_changes at macosforge.org source_changes at macosforge.org
Mon May 16 07:47:57 PDT 2011


Revision: 7481
          http://trac.macosforge.org/projects/calendarserver/changeset/7481
Author:   glyph at apple.com
Date:     2011-05-16 07:47:57 -0700 (Mon, 16 May 2011)
Log Message:
-----------
move commit/close into the main body, just in case; failure to exit is always a bother

Modified Paths:
--------------
    CalendarServer/branches/users/glyph/new-export/calendarserver/tools/export.py

Modified: CalendarServer/branches/users/glyph/new-export/calendarserver/tools/export.py
===================================================================
--- CalendarServer/branches/users/glyph/new-export/calendarserver/tools/export.py	2011-05-16 14:47:44 UTC (rev 7480)
+++ CalendarServer/branches/users/glyph/new-export/calendarserver/tools/export.py	2011-05-16 14:47:57 UTC (rev 7481)
@@ -248,14 +248,14 @@
                   self.options.exporters]
             )
             yield exportToFile(allCalendars, self.output)
+            yield txn.commit()
+            # TODO: should be read-only, so commit/abort shouldn't make a
+            # difference.  commit() for now, in case any transparent cache /
+            # update stuff needed to happen, don't want to undo it.
+            self.output.close()
         except:
             log.err()
 
-        yield txn.commit()
-        # TODO: should be read-only, so commit/abort shouldn't make a
-        # difference.  commit() for now, in case any transparent cache / update
-        # stuff needed to happen, don't want to undo it.
-        self.output.close()
         self.reactor.stop()
 
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20110516/445fe063/attachment.html>


More information about the calendarserver-changes mailing list