Re: [PATCH] remove prefixes from `--output={threads, messages}' results

Subject: Re: [PATCH] remove prefixes from `--output={threads, messages}' results

Date: Thu, 30 Jun 2011 09:24:12 -0700

To: Pieter Praet, notmuch@notmuchmail.org

Cc:

From: Carl Worth

On Thu, 30 Jun 2011 10:19:49 +0200, Pieter Praet <pieter@praet.org> wrote:
> Alter `do_search_threads()' and `do_search_messages()'
> to not prepend each result with `thread:' respectively `id:'.

My one concern here is that I've sometimes had a message-id without the
id prefix and run a command like this:

    notmuch search 1309421989-22410-1-git-send-email-pieter@praet.org

And I've gotten confused when I've received no output, (didn't I receive
that mail? what happened?).

But I think this is a separate bug where the right fix is to make any
search terms with no prefix search through *all* prefixed terms
generated from email content. This would allow us to also avoid indexing
some content twice, (currently we store subject, from, and to both with
a prefix and without a prefix).

-Carl

-- 
carl.d.worth@intel.com

• Previous message (by thread): Re: [PATCH] add `tag:' prefix to `--output=tags' results

Thread:

• Carl Worth—Inconsistent output from "notmuch search --output=<foo>" [inbox, unread]
  • Jameson Rollins—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
  • Sebastian Spaeth—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
    • David Edmondson—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
    • Carl Worth—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
      • Jameson Rollins—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
  • Michal Sojka—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, unread]
  • Sebastian Spaeth—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
    • Pieter Praet—[PATCH] remove prefixes from `--output={threads,messages}' results [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Carl Worth—Re: [PATCH] remove prefixes from `--output={threads, messages}' results [inbox, signed, unread]
        • Pieter Praet—Re: [PATCH] remove prefixes from `--output={threads, messages}' results [inbox, unread]
    • Pieter Praet—[PATCH] add `tag:' prefix to `--output=tags' results [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Pieter Praet—[PATCH] fix breakage in `notmuch-hello-generate-tag-alist' due to `tag:' prefix [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Pieter Praet—[PATCH] fix breakage in `notmuch-select-tag-with-completion' due to `tag:' prefix [inbox, notmuch::obsolete, notmuch::patch, unread]
      • David Bremner—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
        • Jani Nikula—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
          • Pieter Praet—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
        • Jameson Graef Rollins—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, signed, unread]
          • Dmitry Kurochkin—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
          • Pieter Praet—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
      • David Edmondson—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, signed, unread]
        • Pieter Praet—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
    • Pieter Praet—[PATCH] emacs: add keybind and function to stash Message-ID without prefix [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
      • Pieter Praet—[PATCH] test: stashing in notmuch-{show,search} [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • David Bremner—Re: [PATCH] test: stashing in notmuch-{show,search} [inbox, unread]
          • Pieter Praet—[PATCH] test: emacs: tidy up "Stashing in notmuch-show" test [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
            • David Bremner—Re: [PATCH] test: emacs: tidy up "Stashing in notmuch-show" test [inbox, unread]