Re: [PATCH v2 2/4] cli: Extend the search command for --output=addresses and similar
authorMichal Sojka <sojkam1@fel.cvut.cz>
Thu, 9 Oct 2014 10:55:38 +0000 (12:55 +0200)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 18:05:10 +0000 (10:05 -0800)
a0/a323efae2ecb3e8bd2a70e61bd6b9fc52abe77 [new file with mode: 0644]

diff --git a/a0/a323efae2ecb3e8bd2a70e61bd6b9fc52abe77 b/a0/a323efae2ecb3e8bd2a70e61bd6b9fc52abe77
new file mode 100644 (file)
index 0000000..4443199
--- /dev/null
@@ -0,0 +1,509 @@
+Return-Path: <sojkam1@fel.cvut.cz>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id BEED0431FB6\r
+       for <notmuch@notmuchmail.org>; Thu,  9 Oct 2014 03:55:56 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -2.3\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id V9osyjXzGVic for <notmuch@notmuchmail.org>;\r
+       Thu,  9 Oct 2014 03:55:47 -0700 (PDT)\r
+Received: from max.feld.cvut.cz (max.feld.cvut.cz [147.32.192.36])\r
+       by olra.theworths.org (Postfix) with ESMTP id 73D37431FAF\r
+       for <notmuch@notmuchmail.org>; Thu,  9 Oct 2014 03:55:47 -0700 (PDT)\r
+Received: from localhost (unknown [192.168.200.7])\r
+       by max.feld.cvut.cz (Postfix) with ESMTP id 8F02E5CCF60;\r
+       Thu,  9 Oct 2014 12:55:42 +0200 (CEST)\r
+X-Virus-Scanned: IMAP STYX AMAVIS\r
+Received: from max.feld.cvut.cz ([192.168.200.1])\r
+       by localhost (styx.feld.cvut.cz [192.168.200.7]) (amavisd-new,\r
+       port 10044)\r
+       with ESMTP id NbQiOYUUUiMu; Thu,  9 Oct 2014 12:55:39 +0200 (CEST)\r
+Received: from imap.feld.cvut.cz (imap.feld.cvut.cz [147.32.192.34])\r
+       by max.feld.cvut.cz (Postfix) with ESMTP id 06CAA5CCE7B;\r
+       Thu,  9 Oct 2014 12:55:38 +0200 (CEST)\r
+Received: from wsh by steelpick.2x.cz with local (Exim 4.84)\r
+       (envelope-from <sojkam1@fel.cvut.cz>)\r
+       id 1XcBNi-0000TV-MN; Thu, 09 Oct 2014 12:55:38 +0200\r
+From: Michal Sojka <sojkam1@fel.cvut.cz>\r
+To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
+Subject: Re: [PATCH v2 2/4] cli: Extend the search command for\r
+       --output=addresses      and similar\r
+In-Reply-To: <m24mvht4c4.fsf@guru.guru-group.fi>\r
+References: <874mvqxrnp.fsf@nikula.org>\r
+       <1412542319-20017-1-git-send-email-sojkam1@fel.cvut.cz>\r
+       <1412542319-20017-3-git-send-email-sojkam1@fel.cvut.cz>\r
+       <m24mvht4c4.fsf@guru.guru-group.fi>\r
+User-Agent: Notmuch/0.18.1+101~g56b0ff0 (http://notmuchmail.org) Emacs/24.3.1\r
+       (x86_64-pc-linux-gnu)\r
+Date: Thu, 09 Oct 2014 12:55:38 +0200\r
+Message-ID: <87zjd51phx.fsf@steelpick.2x.cz>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=utf-8\r
+Content-Transfer-Encoding: quoted-printable\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Thu, 09 Oct 2014 10:55:56 -0000\r
+\r
+On Mon, Oct 06 2014, Tomi Ollila wrote:\r
+> On Sun, Oct 05 2014, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
+>\r
+>> The new outputs allow printing senders, recipients or both of matching\r
+>> messages.\r
+>>\r
+>> This code based on a patch from Jani Nikula.\r
+>\r
+> OK, IMO...\r
+>\r
+> 1/4 OK\r
+>\r
+> Before 2/4 add support for 'flag' arguments, drop the --output=3Daddresses\r
+> option which is now done as --output=3Dsender --output=3Drecipients\r
+\r
+OK\r
+\r
+> In deduplication comment did not describe the deduplication at all...\r
+> so I looked a bit into the code now... the Default you described was\r
+> that with "John Doe" <john.doe@example.com> and "John Doe" <JOHN.DOE@EXAM=\r
+PLE.COM>\r
+> only one was printed (but not which one).\r
+\r
+I intentionally didn't want to define which one, but I agree that it\r
+might be useful in same cases. It would depend on --sort option and on\r
+the order of addresses in email headers.\r
+\r
+> Secondly, what happens with "Doe, John" <john.doe@example.com> and\r
+> "John Doe" <JOHN.DOE@EXAMPLE.COM>... ah, it is same as *addr* with\r
+> case-insensitive address.\r
+>\r
+> Sorry, but IMO these options are a bit strange.\r
+\r
+My impression is that I did bad job describing the deduplication\r
+algorithm, which is why you don't understand it. Maybe, we can also\r
+change the name of the option to --filter-by, or something like this.\r
+\r
+When thinking about how to best document such an option, it seems that\r
+the user must be aware that this is implemented as flags that are ORed.\r
+Which means that the default should be what was in the previous patch\r
+--unique=3Dnone.\r
+\r
+What about the following?\r
+\r
+    ``--filter-flag=3D``\ (**addr**\ \|\ **name**\ \|\ **addrfold**)\r
+\r
+        Can be used with ``--output=3Daddresses``, ``--output=3Dsender`` or\r
+        ``--output=3Drecipients`` to filter out duplicate addresses. The\r
+        filtering algorithm receives a sequence of email addresses and\r
+        outputs the same sequence without the addresses that are\r
+        considered a duplicate of a previously output address. What is\r
+        considered a duplicate depends on the flags given:\r
+\r
+       **addr** means that the address part is compared.\r
+       Case-sensitivity can be controlled by **addrfold** flag (see\r
+       below). For example, the addresses "John Doe <john@example.com>"\r
+       and "Dr. John Doe <john@example.com>" will be considered\r
+       duplicate.\r
+\r
+       **name** means that the name part is compared in case-sensitive\r
+       manner. For example, the addresses "John Doe <john@example.com>"\r
+       and "John Doe <john@doe.name>" will be considered duplicate.\r
+\r
+       **addrfold** when used with the **addr** flag, the address\r
+       comparison is performed in case-insensitive manner. For example,\r
+       the addresses "John Doe <john@example.com>" and "Dr. John Doe\r
+       <JOHN@EXAMPLE.COM>" will be considered duplicate.\r
+\r
+       To specify multiple flags, this option can be given multiple\r
+       times. For example, ``--filter-flag=3Dname --filter-flag=3Daddr``\r
+       will print unique case-sensitive combinations of both name and\r
+       address parts.\r
+\r
+With this, the previously default behavior would now has to be spelled\r
+as "--filter-flag=3Daddr --filter-flag=3Daddrfold".\r
+\r
+I'm not sure it is wise present such a low-level interface (flags) to\r
+command-line users, but it is hopefully more understandable now. What do\r
+you think?\r
+\r
+> Not to go to choose which one to choose (first, last, most common) instead\r
+> of the suggested options these should be the ones:\r
+>\r
+> 1) "John Doe" <john.doe@example.com> and "John Doe" <JOHN.DOE@EXAMPLE.COM=\r
+>:\r
+> only one printed, but if either were "Dr. John Doe", both of these are pr=\r
+inted\r
+> (this as default).\r
+\r
+According to the above, which could be achieved as --filter-flag=3Dname\r
+--filter-flag=3Daddr --filter-flag=3Daddrfold.\r
+\r
+> 2) same as above, but only make case-insensitive\r
+\r
+case-insensitive is already in 1), you probably mean case-sensitive.\r
+\r
+> address match -- i.e. in the 2 above cases in option 1, print only\r
+> one.\r
+\r
+This would be --filter-flag=3Dname --filter-flag=3Daddr.\r
+\r
+> (and same name but different address to perhaps never been an option...)\r
+>\r
+> I might like to have option that does case-sensitive address match,=20\r
+\r
+This would be just --filter-flag=3Daddr.\r
+\r
+As a side note, it is interesting, that you mentioned your options as an\r
+enumeration even though they are actually combinations of several on/off\r
+flags. I think that it is more natural for human brains to think in\r
+terms of simple lists than in terms of combinations of flags. That's why\r
+I originally implemented --output=3Daddresses as just another keyword,\r
+rather than requiring the user to specify both sender and receivers.\r
+\r
+Thanks for the review.\r
+-Michal\r
+\r
+> In those cases I don't know the recipient's culture and the email he\r
+> sent to me used format <Foo.Bar@example.org> (and not knowing which\r
+> one is the first and which last name (or whatever names these are) --\r
+> just to reply in same case format in respect...\r
+\r
+\r
+>\r
+>\r
+> Tomi\r
+>\r
+>\r
+>> ---\r
+>>  completion/notmuch-completion.bash |   2 +-\r
+>>  completion/notmuch-completion.zsh  |   3 +-\r
+>>  doc/man1/notmuch-search.rst        |  22 +++++++-\r
+>>  notmuch-search.c                   | 100 ++++++++++++++++++++++++++++++=\r
+++++---\r
+>>  test/T090-search-output.sh         |  64 ++++++++++++++++++++++++\r
+>>  5 files changed, 182 insertions(+), 9 deletions(-)\r
+>>\r
+>> diff --git a/completion/notmuch-completion.bash b/completion/notmuch-com=\r
+pletion.bash\r
+>> index 0571dc9..c37ddf5 100644\r
+>> --- a/completion/notmuch-completion.bash\r
+>> +++ b/completion/notmuch-completion.bash\r
+>> @@ -294,7 +294,7 @@ _notmuch_search()\r
+>>         return\r
+>>         ;;\r
+>>     --output)\r
+>> -       COMPREPLY=3D( $( compgen -W "summary threads messages files tags" =\r
+-- "${cur}" ) )\r
+>> +       COMPREPLY=3D( $( compgen -W "summary threads messages files tags s=\r
+ender recipients addresses" -- "${cur}" ) )\r
+>>         return\r
+>>         ;;\r
+>>     --sort)\r
+>> diff --git a/completion/notmuch-completion.zsh b/completion/notmuch-comp=\r
+letion.zsh\r
+>> index 67a9aba..bff8fd5 100644\r
+>> --- a/completion/notmuch-completion.zsh\r
+>> +++ b/completion/notmuch-completion.zsh\r
+>> @@ -52,7 +52,8 @@ _notmuch_search()\r
+>>    _arguments -s : \\r
+>>      '--max-threads=3D[display only the first x threads from the search =\r
+results]:number of threads to show: ' \\r
+>>      '--first=3D[omit the first x threads from the search results]:numbe=\r
+r of threads to omit: ' \\r
+>> -    '--sort=3D[sort results]:sorting:((newest-first\:"reverse chronolog=\r
+ical order" oldest-first\:"chronological order"))'\r
+>> +    '--sort=3D[sort results]:sorting:((newest-first\:"reverse chronolog=\r
+ical order" oldest-first\:"chronological order"))' \\r
+>> +    '--output=3D[select what to output]:output:((summary threads messag=\r
+es files tags sender recipients addresses))'\r
+>>  }\r
+>>\r
+>>  _notmuch()\r
+>> diff --git a/doc/man1/notmuch-search.rst b/doc/man1/notmuch-search.rst\r
+>> index 90160f2..3447820 100644\r
+>> --- a/doc/man1/notmuch-search.rst\r
+>> +++ b/doc/man1/notmuch-search.rst\r
+>> @@ -35,7 +35,7 @@ Supported options for **search** include\r
+>>          intended for programs that invoke **notmuch(1)** internally. If\r
+>>          omitted, the latest supported version will be used.\r
+>>\r
+>> -    ``--output=3D(summary|threads|messages|files|tags)``\r
+>> +    ``--output=3D(summary|threads|messages|files|tags|sender|recipients=\r
+|addresses)``\r
+>>\r
+>>          **summary**\r
+>>              Output a summary of each thread with any message matching\r
+>> @@ -78,6 +78,26 @@ Supported options for **search** include\r
+>>              by null characters (--format=3Dtext0), as a JSON array\r
+>>              (--format=3Djson), or as an S-Expression list (--format=3Ds=\r
+exp).\r
+>>\r
+>> +   **sender**\r
+>> +            Output all addresses from the *From* header that appear on\r
+>> +            any message matching the search terms, either one per line\r
+>> +            (--format=3Dtext), separated by null characters\r
+>> +            (--format=3Dtext0), as a JSON array (--format=3Djson), or as\r
+>> +            an S-Expression list (--format=3Dsexp).\r
+>> +\r
+>> +       Note: Searching for **sender** should be much faster than\r
+>> +       searching for **recipients** or **addresses**, because\r
+>> +       sender addresses are cached directly in the database\r
+>> +       whereas other addresses need to be fetched from message\r
+>> +       files.\r
+>> +\r
+>> +   **recipients**\r
+>> +            Like **sender** but for addresses from *To*, *Cc* and\r
+>> +       *Bcc* headers.\r
+>> +\r
+>> +   **addresses**\r
+>> +       Like **sender** and **recipients** together.\r
+>> +\r
+>>      ``--sort=3D``\ (**newest-first**\ \|\ **oldest-first**)\r
+>>          This option can be used to present results in either\r
+>>          chronological order (**oldest-first**) or reverse chronological\r
+>> diff --git a/notmuch-search.c b/notmuch-search.c\r
+>> index 5ac2a26..0614f10 100644\r
+>> --- a/notmuch-search.c\r
+>> +++ b/notmuch-search.c\r
+>> @@ -23,11 +23,14 @@\r
+>>  #include "string-util.h"\r
+>>\r
+>>  typedef enum {\r
+>> -    OUTPUT_SUMMARY,\r
+>> -    OUTPUT_THREADS,\r
+>> -    OUTPUT_MESSAGES,\r
+>> -    OUTPUT_FILES,\r
+>> -    OUTPUT_TAGS\r
+>> +    OUTPUT_SUMMARY =3D 1 << 0,\r
+>> +    OUTPUT_THREADS =3D 1 << 1,\r
+>> +    OUTPUT_MESSAGES        =3D 1 << 2,\r
+>> +    OUTPUT_FILES   =3D 1 << 3,\r
+>> +    OUTPUT_TAGS            =3D 1 << 4,\r
+>> +    OUTPUT_SENDER  =3D 1 << 5,\r
+>> +    OUTPUT_RECIPIENTS      =3D 1 << 6,\r
+>> +    OUTPUT_ADDRESSES       =3D OUTPUT_SENDER | OUTPUT_RECIPIENTS,\r
+>>  } output_t;\r
+>>\r
+>>  typedef struct {\r
+>> @@ -220,6 +223,67 @@ do_search_threads (search_options_t *o)\r
+>>      return 0;\r
+>>  }\r
+>>\r
+>> +static void\r
+>> +print_address_list (const search_options_t *o, InternetAddressList *lis=\r
+t)\r
+>> +{\r
+>> +    InternetAddress *address;\r
+>> +    int i;\r
+>> +\r
+>> +    for (i =3D 0; i < internet_address_list_length (list); i++) {\r
+>> +   address =3D internet_address_list_get_address (list, i);\r
+>> +   if (INTERNET_ADDRESS_IS_GROUP (address)) {\r
+>> +       InternetAddressGroup *group;\r
+>> +       InternetAddressList *group_list;\r
+>> +\r
+>> +       group =3D INTERNET_ADDRESS_GROUP (address);\r
+>> +       group_list =3D internet_address_group_get_members (group);\r
+>> +       if (group_list =3D=3D NULL)\r
+>> +           continue;\r
+>> +\r
+>> +       print_address_list (o, group_list);\r
+>> +   } else {\r
+>> +       InternetAddressMailbox *mailbox;\r
+>> +       const char *name;\r
+>> +       const char *addr;\r
+>> +       char *full_address;\r
+>> +\r
+>> +       mailbox =3D INTERNET_ADDRESS_MAILBOX (address);\r
+>> +\r
+>> +       name =3D internet_address_get_name (address);\r
+>> +       addr =3D internet_address_mailbox_get_addr (mailbox);\r
+>> +\r
+>> +       if (name && *name)\r
+>> +           full_address =3D talloc_asprintf (o->format, "%s <%s>", name, addr);\r
+>> +       else\r
+>> +           full_address =3D talloc_strdup (o->format, addr);\r
+>> +\r
+>> +       if (!full_address) {\r
+>> +           fprintf (stderr, "Error: out of memory\n");\r
+>> +           break;\r
+>> +       }\r
+>> +       o->format->string (o->format, full_address);\r
+>> +       o->format->separator (o->format);\r
+>> +\r
+>> +       talloc_free (full_address);\r
+>> +   }\r
+>> +    }\r
+>> +}\r
+>> +\r
+>> +static void\r
+>> +print_address_string (const search_options_t *o, const char *recipients)\r
+>> +{\r
+>> +    InternetAddressList *list;\r
+>> +\r
+>> +    if (recipients =3D=3D NULL)\r
+>> +   return;\r
+>> +\r
+>> +    list =3D internet_address_list_parse_string (recipients);\r
+>> +    if (list =3D=3D NULL)\r
+>> +   return;\r
+>> +\r
+>> +    print_address_list (o, list);\r
+>> +}\r
+>> +\r
+>>  static int\r
+>>  do_search_messages (search_options_t *o)\r
+>>  {\r
+>> @@ -266,11 +330,29 @@ do_search_messages (search_options_t *o)\r
+>>\r
+>>         notmuch_filenames_destroy( filenames );\r
+>>\r
+>> -   } else { /* output =3D=3D OUTPUT_MESSAGES */\r
+>> +   } else if (o->output =3D=3D OUTPUT_MESSAGES) {\r
+>>         format->set_prefix (format, "id");\r
+>>         format->string (format,\r
+>>                         notmuch_message_get_message_id (message));\r
+>>         format->separator (format);\r
+>> +   } else {\r
+>> +       if (o->output & OUTPUT_SENDER) {\r
+>> +           const char *addrs;\r
+>> +\r
+>> +           addrs =3D notmuch_message_get_header (message, "from");\r
+>> +           print_address_string (o, addrs);\r
+>> +       }\r
+>> +\r
+>> +       if (o->output & OUTPUT_RECIPIENTS) {\r
+>> +           const char *hdrs[] =3D { "to", "cc", "bcc" };\r
+>> +           const char *addrs;\r
+>> +           size_t j;\r
+>> +\r
+>> +           for (j =3D 0; j < ARRAY_SIZE (hdrs); j++) {\r
+>> +               addrs =3D notmuch_message_get_header (message, hdrs[j]);\r
+>> +               print_address_string (o, addrs);\r
+>> +           }\r
+>> +       }\r
+>>     }\r
+>>\r
+>>     notmuch_message_destroy (message);\r
+>> @@ -370,6 +452,9 @@ notmuch_search_command (notmuch_config_t *config, in=\r
+t argc, char *argv[])\r
+>>       (notmuch_keyword_t []){ { "summary", OUTPUT_SUMMARY },\r
+>>                               { "threads", OUTPUT_THREADS },\r
+>>                               { "messages", OUTPUT_MESSAGES },\r
+>> +                             { "sender", OUTPUT_SENDER },\r
+>> +                             { "recipients", OUTPUT_RECIPIENTS },\r
+>> +                             { "addresses", OUTPUT_ADDRESSES },\r
+>>                               { "files", OUTPUT_FILES },\r
+>>                               { "tags", OUTPUT_TAGS },\r
+>>                               { 0, 0 } } },\r
+>> @@ -461,6 +546,9 @@ notmuch_search_command (notmuch_config_t *config, in=\r
+t argc, char *argv[])\r
+>>     ret =3D do_search_threads (&o);\r
+>>     break;\r
+>>      case OUTPUT_MESSAGES:\r
+>> +    case OUTPUT_SENDER:\r
+>> +    case OUTPUT_RECIPIENTS:\r
+>> +    case OUTPUT_ADDRESSES:\r
+>>      case OUTPUT_FILES:\r
+>>     ret =3D do_search_messages (&o);\r
+>>     break;\r
+>> diff --git a/test/T090-search-output.sh b/test/T090-search-output.sh\r
+>> index 947d572..5458de1 100755\r
+>> --- a/test/T090-search-output.sh\r
+>> +++ b/test/T090-search-output.sh\r
+>> @@ -387,6 +387,70 @@ cat <<EOF >EXPECTED\r
+>>  EOF\r
+>>  test_expect_equal_file OUTPUT EXPECTED\r
+>>\r
+>> +test_begin_subtest "--output=3Dsender"\r
+>> +notmuch search --output=3Dsender '*' | sort | uniq --count >OUTPUT\r
+>> +cat <<EOF >EXPECTED\r
+>> +      1 Adrian Perez de Castro <aperez@igalia.com>\r
+>> +      2 Alex Botero-Lowry <alex.boterolowry@gmail.com>\r
+>> +      4 Alexander Botero-Lowry <alex.boterolowry@gmail.com>\r
+>> +      1 Aron Griffis <agriffis@n01se.net>\r
+>> +     12 Carl Worth <cworth@cworth.org>\r
+>> +      1 Chris Wilson <chris@chris-wilson.co.uk>\r
+>> +      1 Fran=C3=A7ois Boulogne <boulogne.f@gmail.com>\r
+>> +      1 Ingmar Vanhassel <ingmar@exherbo.org>\r
+>> +      1 Israel Herraiz <isra@herraiz.org>\r
+>> +      4 Jan Janak <jan@ryngle.com>\r
+>> +      2 Jjgod Jiang <gzjjgod@gmail.com>\r
+>> +      7 Keith Packard <keithp@keithp.com>\r
+>> +      5 Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
+>> +      5 Mikhail Gusarov <dottedmag@dottedmag.net>\r
+>> +      1 Olivier Berger <olivier.berger@it-sudparis.eu>\r
+>> +      1 Rolland Santimano <rollandsantimano@yahoo.com>\r
+>> +      3 Stewart Smith <stewart@flamingspork.com>\r
+>> +EOF\r
+>> +test_expect_equal_file OUTPUT EXPECTED\r
+>> +\r
+>> +test_begin_subtest "--output=3Drecipients"\r
+>> +notmuch search --output=3Drecipients '*' | sort | uniq --count >OUTPUT\r
+>> +cat <<EOF >EXPECTED\r
+>> +      1 Allan McRae <allan@archlinux.org>\r
+>> +      1 Discussion about the Arch User Repository (AUR) <aur-general@ar=\r
+chlinux.org>\r
+>> +      1 Keith Packard <keithp@keithp.com>\r
+>> +      1 Mikhail Gusarov <dottedmag@dottedmag.net>\r
+>> +      2 notmuch <notmuch@notmuchmail.org>\r
+>> +     48 notmuch@notmuchmail.org\r
+>> +      1 olivier.berger@it-sudparis.eu\r
+>> +EOF\r
+>> +test_expect_equal_file OUTPUT EXPECTED\r
+>> +\r
+>> +test_begin_subtest "--output=3Daddresses"\r
+>> +notmuch search --output=3Daddresses '*' | sort | uniq --count >OUTPUT\r
+>> +cat <<EOF >EXPECTED\r
+>> +      1 Adrian Perez de Castro <aperez@igalia.com>\r
+>> +      2 Alex Botero-Lowry <alex.boterolowry@gmail.com>\r
+>> +      4 Alexander Botero-Lowry <alex.boterolowry@gmail.com>\r
+>> +      1 Allan McRae <allan@archlinux.org>\r
+>> +      1 Aron Griffis <agriffis@n01se.net>\r
+>> +     12 Carl Worth <cworth@cworth.org>\r
+>> +      1 Chris Wilson <chris@chris-wilson.co.uk>\r
+>> +      1 Discussion about the Arch User Repository (AUR) <aur-general@ar=\r
+chlinux.org>\r
+>> +      1 Fran=C3=A7ois Boulogne <boulogne.f@gmail.com>\r
+>> +      1 Ingmar Vanhassel <ingmar@exherbo.org>\r
+>> +      1 Israel Herraiz <isra@herraiz.org>\r
+>> +      4 Jan Janak <jan@ryngle.com>\r
+>> +      2 Jjgod Jiang <gzjjgod@gmail.com>\r
+>> +      8 Keith Packard <keithp@keithp.com>\r
+>> +      5 Lars Kellogg-Stedman <lars@seas.harvard.edu>\r
+>> +      6 Mikhail Gusarov <dottedmag@dottedmag.net>\r
+>> +      1 Olivier Berger <olivier.berger@it-sudparis.eu>\r
+>> +      1 Rolland Santimano <rollandsantimano@yahoo.com>\r
+>> +      3 Stewart Smith <stewart@flamingspork.com>\r
+>> +      2 notmuch <notmuch@notmuchmail.org>\r
+>> +     48 notmuch@notmuchmail.org\r
+>> +      1 olivier.berger@it-sudparis.eu\r
+>> +EOF\r
+>> +test_expect_equal_file OUTPUT EXPECTED\r
+>> +\r
+>>  test_begin_subtest "sanitize output for quoted-printable line-breaks in=\r
+ author and subject"\r
+>>  add_message "[subject]=3D'two =3D?ISO-8859-1?Q?line=3D0A_subject?=3D\r
+>>     headers'"\r
+>> --\r
+>> 2.1.1\r
+>>\r
+>> _______________________________________________\r
+>> notmuch mailing list\r
+>> notmuch@notmuchmail.org\r
+>> http://notmuchmail.org/mailman/listinfo/notmuch\r