mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-11-21 18:38:08 +01:00
cli: reset db directory mtime upon directory removal
The library does not have a function to remove a directory document for a path. Usually this doesn't matter except for a slight waste of space. However, if the same directory gets added to the filesystem again, the old directory document is found with the old mtime. Reset the directory mtime on removal to avoid problems. The corner case that can hit this problem is renaming directories back and forth. Renaming does not change the mtime of the directory in the filesystem, and thus the old db directory document mtime may match the fs mtime of the directory. The long term fix might be to add a library function to remove a directory document, however this is a much simpler and faster fix for the time being.
This commit is contained in:
parent
9a5143abbc
commit
acd66cdec0
1 changed files with 9 additions and 0 deletions
|
@ -878,6 +878,15 @@ _remove_directory (void *ctx,
|
|||
goto DONE;
|
||||
}
|
||||
|
||||
/*
|
||||
* XXX: The library does not have a function to remove a directory
|
||||
* document for a path. Usually this doesn't matter except for a
|
||||
* slight waste of space. However, if the directory gets added to
|
||||
* the filesystem again, the old directory document is found with
|
||||
* the old mtime. Reset the directory mtime to avoid problems.
|
||||
*/
|
||||
notmuch_directory_set_mtime (directory, 0);
|
||||
|
||||
DONE:
|
||||
notmuch_directory_destroy (directory);
|
||||
return status;
|
||||
|
|
Loading…
Reference in a new issue