Re: [PATCH] notmuch dump: default to batch-tag format.

[Tomi Ollila <tomi.ollila@iki.fi>]

On Tue, Jan 21 2014, david@tethera.net wrote:

> From: David Bremner <david@tethera.net>
>
> Although we didn't formally deprecate the old format, the new one has
> been available for a year.
> ---
>  NEWS           | 10 ++++++++++
>  notmuch-dump.c |  2 +-
>  2 files changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/NEWS b/NEWS
> index 28788d8..bdb248e 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -1,3 +1,13 @@
> +Notmuch 0.18 (2014-xx-xx)
> +=========================

I tried to advocate Notmuch 0.18 (UNRELEASED) during last development
cycle -- in followu ========================= p patch (if any -- 
with manpage change) this change could also be made.

> +
> +Command-Line Interface
> +----------------------
> +
> +`notmuch dump` now defaults to `batch-tag` format.
> +
> +  The old format is still available with `--format=sup`.
> +
>  Notmuch 0.17 (2013-12-30)
>  =========================
>  
> diff --git a/notmuch-dump.c b/notmuch-dump.c
> index f8edda7..158142f 100644
> --- a/notmuch-dump.c
> +++ b/notmuch-dump.c
> @@ -40,7 +40,7 @@ notmuch_dump_command (notmuch_config_t *config, int argc, char *argv[])
>      char *output_file_name = NULL;
>      int opt_index;
>  
> -    int output_format = DUMP_FORMAT_SUP;
> +    int output_format = DUMP_FORMAT_BATCH_TAG;

This part looks good.

>  
>      notmuch_opt_desc_t options[] = {
>  	{ NOTMUCH_OPT_KEYWORD, &output_format, "format", 'f',
> -- 

Tomi