compact: unconditionally remove old wip database compact directory
authorTomi Ollila <tomi.ollila@iki.fi>
Thu, 14 Nov 2013 22:03:26 +0000 (00:03 +0200)
committerDavid Bremner <david@tethera.net>
Wed, 20 Nov 2013 00:14:28 +0000 (20:14 -0400)
In case previous notmuch compact has been interrupted there is old
work-in-progress database compact directory partially filled. Remove
it just before starting to fill the directory with new files.

lib/database.cc

index d79cc3006d29a204aaa54a762aa30e0a4734ded7..d09ad99f505c63b060c6d78ed81bdc79d0b470fb 100644 (file)
@@ -922,6 +922,12 @@ notmuch_database_compact (const char *path,
        goto DONE;
     }
 
+    /* Unconditionally attempt to remove old work-in-progress database (if
+     * any). This is "protected" by database lock. If this fails due to write
+     * errors (etc), the following code will fail and provide error message.
+     */
+    (void) rmtree (compact_xapian_path);
+
     try {
        NotmuchCompactor compactor (status_cb, closure);