Re: [PATCH 5/6] cli: add global option "--db-revision"

Subject: Re: [PATCH 5/6] cli: add global option "--db-revision"

Date: Fri, 07 Aug 2015 10:05:51 +0200

To: David Bremner, notmuch@notmuchmail.org

Cc:

On Fri, 05 Jun 2015 19:28 +0200, David Bremner wrote:
> +    ``--db-revision=UUID``
> +       Enforce that the database revision is UUID. This is useful to
> +       detect rollover in modification counts on messages. You can
> +       find this UUID in the first column of output from
> +       ``notmuch count --output=modifications``

I'm not sure if the wording here is entirely clear. When reading this
the first time, to me it sounded like it might give you a view of the
database from when the revision was UUID. Maybe we can clarify here that
this will result in notmuch exiting if the revision is not matched.

Best regards,
Daniel

Previous message (by thread): Re: [PATCH 2/6] lib: Add per-message last modification tracking

Thread:

David Bremner—Revision tracking, round 2 [inbox, unread]
- David Bremner—[WIP2 01/12] lib: Only sync modified message documents [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 02/12] lib: Add per-message last modification tracking [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 03/12] lib: API to retrieve database revision and UUID [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 04/12] cli: add type introspection to sprinter type [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 05/12] cli: add two sprinter utility functions [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 06/12] cli/show: add extra element to structured output for metadata [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 07/12] lib: Add "lastmod:" queries for filtering by last modification [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 08/12] cli/show: add lastmod to structured output [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 09/12] cli/search: add metadata element to structured output [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 10/12] emacs: convert notmuch-search to format-version 3 [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 11/12] emacs: convert notmuch-tree to format-version 3 [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- David Bremner—[WIP2 12/12] cli: add global option "--db-revision" [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
- Gaute Hope—Revision tracking, round 2 [inbox, unread]
  - David Bremner—revision tracking patches, round 1 [inbox, unread]
    - David Bremner—[PATCH 1/6] lib: Only sync modified message documents [inbox, notmuch::patch, notmuch::pushed, unread]
      - David Bremner—Re: [PATCH 1/6] lib: Only sync modified message documents [inbox, unread]
    - David Bremner—[PATCH 2/6] lib: Add per-message last modification tracking [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Daniel Schoepe—Re: [PATCH 2/6] lib: Add per-message last modification tracking [inbox, unread]
        David Bremner—Re: [PATCH 2/6] lib: Add per-message last modification tracking [inbox, unread]
        Austin Clements—Re: [PATCH 2/6] lib: Add per-message last modification tracking [inbox, unread]
        David Bremner—[PATCH] fixup! lib: Add per-message last modification tracking [inbox, unread]
    - David Bremner—[PATCH 3/6] lib: API to retrieve database revision and UUID [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Tomi Ollila—Re: [PATCH 3/6] lib: API to retrieve database revision and UUID [inbox, unread]
    - David Bremner—[PATCH 4/6] cli/count: add --output=modifications [inbox, notmuch::obsolete, notmuch::patch, unread]
    - David Bremner—[PATCH 5/6] cli: add global option "--db-revision" [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Daniel Schoepe—Re: [PATCH 5/6] cli: add global option "--db-revision" [inbox, unread]
        David Bremner—Re: [PATCH 5/6] cli: add global option "--db-revision" [inbox, unread]
    - David Bremner—[PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Daniel Schoepe—Re: [PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification [inbox, unread]
        David Bremner—Re: [PATCH 6/6] lib: Add "lastmod:" queries for filtering by last modification [inbox, unread]