Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering

Subject: Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering

Date: Wed, 31 Dec 2014 22:03:29 +0100

To: Michal Sojka, notmuch@notmuchmail.org

Cc: Tomi Ollila

From: David Bremner

Michal Sojka <sojkam1@fel.cvut.cz> writes:

> This option allows to configure the criterion for duplicate address
> filtering. Without this option, all unique combinations of name and
> address parts are printed. This option allows to filter the output
> more, for example to only contain unique address parts.

I had the feeling there was some "controversy" about the UI here, but
following back the 3 versions of the series I didn't see it. Does that
mean we just need to sanity check the code, or are there outstanding
bikes to shed?

d

• Previous message (by thread): Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering

Thread:

• Michal Sojka—[PATCH v3 00/10] "notmuch address" command [inbox, unread]
  • Michal Sojka—[PATCH v3 01/10] cli: search: Rename options to context [inbox, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
  • Michal Sojka—[PATCH v3 02/10] cli: search: Move more variables into search_context_t [inbox, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
  • Michal Sojka—[PATCH v3 03/10] cli: search: Convert ctx. to ctx-> [inbox, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
  • Michal Sojka—[PATCH v3 04/10] cli: search: Split notmuch_search_command to smaller functions [inbox, notmuch::patch, notmuch::pushed, unread]
  • Michal Sojka—[PATCH v3 05/10] cli: add support for hierarchical command line option arrays [inbox, notmuch::patch, notmuch::pushed, unread]
  • Michal Sojka—[PATCH v3 06/10] cli: Introduce "notmuch address" command [inbox, notmuch::patch, notmuch::pushed, unread]
    • Tomi Ollila—Re: [PATCH v3 06/10] cli: Introduce "notmuch address" command [inbox, unread]
  • Michal Sojka—[PATCH v3 07/10] cli: search: Convert --output to keyword argument [inbox, notmuch::patch, notmuch::pushed, unread]
  • Michal Sojka—[PATCH v3 08/10] cli: address: Do not output duplicate addresses [inbox, notmuch::patch, notmuch::pushed, unread]
  • Michal Sojka—[PATCH v3 09/10] cli: address: Add --output=count [inbox, notmuch::patch, notmuch::pushed, unread]
    • Tomi Ollila—Re: [PATCH v3 09/10] cli: address: Add --output=count [inbox, unread]
    • David Bremner—Re: [PATCH v3 09/10] cli: address: Add --output=count [inbox, unread]
  • Michal Sojka—[PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, notmuch::obsolete, notmuch::patch, unread]
    • David Bremner—Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, unread]
      • Tomi Ollila—Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, unread]
        • Michal Sojka—Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, unread]
          • Tomi Ollila—Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, unread]
            • Michal Sojka—Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, unread]
              • Michal Sojka—Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, unread]
    • David Bremner—Re: [PATCH v3 10/10] cli: address: Add --filter-by option to configure address filtering [inbox, unread]