Re: [PATCH v6 03/12] cli: add insert command

Subject: Re: [PATCH v6 03/12] cli: add insert command

Date: Sun, 23 Jun 2013 13:57:27 +1000

To: Mark Walters

Cc: notmuch@notmuchmail.org

From: Peter Wang

On Sat, 22 Jun 2013 08:48:56 +0100, Mark Walters <markwalters1009@gmail.com> wrote:
> 
> I think I would like the option to have the command fail if it doesn't
> get added to the database. If, for example, the command is used to
> postpone a message it may be very difficult/impossible to find it
> without the postponed tag that I asked for.
> 
> Could we have a --fail-on-index-fail cli option which says which
> behaviour we want? It is a little ugly but I think different users will
> want different behaviour and there is a risk of dataloss in either case.

I would like that option, too (not sure about the name).  It can be
added after at least the first few patches in the series are pushed.

Peter

• Previous message (by thread): Re: [PATCH v6 03/12] cli: add insert command

Thread:

• Peter Wang—[PATCH v6 00/12] insert command [inbox, unread]
  • Peter Wang—[PATCH v6 01/12] tag-util: move out 'tag' command-line checks [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Mark Walters—Re: [PATCH v6 01/12] tag-util: move out 'tag' command-line checks [inbox, unread]
  • Peter Wang—[PATCH v6 02/12] tag-util: do not reset list in parse_tag_command_line [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Mark Walters—Re: [PATCH v6 02/12] tag-util: do not reset list in parse_tag_command_line [inbox, unread]
  • Peter Wang—[PATCH v6 03/12] cli: add insert command [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Mark Walters—Re: [PATCH v6 03/12] cli: add insert command [inbox, unread]
    • Mark Walters—Re: [PATCH v6 03/12] cli: add insert command [inbox, unread]
      • Peter Wang—Re: [PATCH v6 03/12] cli: add insert command [inbox, unread]
  • Peter Wang—[PATCH v6 04/12] man: document 'insert' command [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 05/12] man: reference notmuch-insert.1 [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 06/12] test: add tests for insert [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 07/12] insert: add --folder option [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 08/12] man: document insert --folder option [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 09/12] test: test insert --folder option [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 10/12] insert: add --create-folder option [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 11/12] man: document insert --create-folder [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v6 12/12] test: test insert --create-folder option [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Mark Walters—Re: [PATCH v6 00/12] insert command [inbox, unread]