[PATCH 0/9] Fix directory lookup on read-only databases
Tomi Ollila
tomi.ollila at iki.fi
Tue May 22 13:48:00 PDT 2012
Austin Clements <amdragon at MIT.EDU> writes:
> This fixes notmuch_database_get_directory and
> notmuch_database_find_message_by_filename so that they don't attempt
> to create missing directory documents. This makes them work on
> read-only databases (and prevents n_d_f_m_b_f from crashing
> unceremoniously on read-only databases).
>
> Unfortunately, there are several functions involved in directory
> document lookup, so the first three patches simply add a creation flag
> at each necessary layer. The remaining patches then fix up the two
> API functions and their uses.
>
> If we do a 0.13.1 bug fix release, these patches could go in to
> complement the API changes made in 0.13 to support these fixes. David
> can make that call.
>
> There are several patches, but they're all short and incremental.
This patch series looks good to me...
Tomi
>
> bindings/python/notmuch/database.py | 11 -----------
> lib/database.cc | 40 +++++++++++++++++++++++++---------------
> lib/directory.cc | 41 +++++++++++++++++++++++++++++++++++------
> lib/message.cc | 11 +++++------
> lib/notmuch-private.h | 13 +++++++++++++
> lib/notmuch.h | 10 ++--------
> notmuch-new.c | 36 +++++++-----------------------------
> test/python | 8 ++++++++
> 8 files changed, 95 insertions(+), 75 deletions(-)
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list