From: Todd <todd@electricoding.com>
To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org
Cc: Austin Clements <aclements@csail.mit.edu>
Subject: Re: [PATCH] Index Content-Type of attachments with a contenttype prefix
Date: Sat, 10 Jan 2015 08:22:25 -0600 [thread overview]
Message-ID: <87k30u90nv.fsf@electricoding.com> (raw)
In-Reply-To: <87ppanj9i9.fsf@maritornes.cs.unb.ca>
>>>>> "DB" == David Bremner <david@tethera.net> writes:
DB> Todd <todd@electricoding.com> writes:
>> I wanted to tag messages with calendar invitations, but couldn't as
>> the information wasn't indexed.
>>
>> This patch allows for queries for like:
>>
>> Find calendar invites
>> - contenttype:text/calendar or contenttype:applicaton/ics
>>
>> Find any image attachments
>> - contenttype:image
>>
>> Find all patches
>> - contenttype:text/x-patch
DB> The main issue, that I won't really address in this message (because I'm
DB> hoping Austin finds time to comment) is upgrading the database. The short
DB> version is that a new "database feature" needs to be created.
I just discovered and starting using notmuch this week, so I'm not
entirely familiar with the project yet. I had suspected there might
be more work :) I'll look into the "database feature".
My other thought was to just index the content-type with the
attachment prefix. This would have made the first search that I
attempted (without reading the docs/source) work
(e.g. attachment:text/calendar or attachment:application/ics).
Is this preferred to adding a new search term?
>> --- a/NEWS
>> +++ b/NEWS
>> @@ -15,6 +15,12 @@ keyboard shortcuts to saved searches.
>> Command-Line Interface
>> ----------------------
DB> Minor point, 0.19 was released, you should start a new NEWS section for
DB> 0.20 with date UNRELEASED
I'll make that change.
>> --- a/completion/notmuch-completion.bash
>> +++ b/completion/notmuch-completion.bash
DB> If you can without too much suffering, it would be nice to update the zsh completion at the same
DB> time.
I don't use zsh, but I'll look into what's needed.
>> +The **contenttype:** prefix can be used to search for specific
>> +content-types of attachments to email messages (as specified by the
>> +sender).
>> +
DB> I'm not 100% sure, but I did wonder if the docs should mention MIME
DB> somewhere, for people searching.
Agreed.
>> { "attachment", "XATTACHMENT" },
>> + { "contenttype", "XCONTENTTYPE"},
>> { "subject", "XSUBJECT"},
DB> I didn't work through all the details, but I did wonder if it was in
DB> some sense redundant to be indexing contenttype and also the existing
DB> attachement and encrypted pseudotags. I guess this might be one of
DB> those cases where we are stuck with the extra indexing for now, until we
DB> sort out some query parsing issues.
DB> d
- Todd
next prev parent reply other threads:[~2015-01-10 14:22 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-10 0:29 [PATCH] Index Content-Type of attachments with a contenttype prefix Todd
2015-01-10 9:00 ` David Bremner
2015-01-10 14:22 ` Todd [this message]
2015-01-10 12:13 ` Jani Nikula
2015-01-10 14:38 ` Todd
2015-01-14 0:00 ` [PATCH v2 0/5] Index the content-type of MIME parts Todd
2015-01-14 0:00 ` [PATCH v2 1/5] Add a failing unit test for indexed mime types Todd
2015-01-15 17:09 ` Jani Nikula
2015-01-15 17:19 ` Jani Nikula
2015-01-14 0:00 ` [PATCH v2 2/5] Add the NOTMUCH_FEATURE_INDEXED_MIMETYPES database feature Todd
2015-01-15 17:20 ` Jani Nikula
2015-01-15 18:57 ` Austin Clements
2015-01-14 0:00 ` [PATCH v2 3/5] Add indexing for the mimetype term Todd
2015-01-15 17:16 ` Jani Nikula
2015-01-14 0:00 ` [PATCH v2 4/5] Update completions for Emacs and bash Todd
2015-01-15 17:17 ` Jani Nikula
2015-01-14 0:00 ` [PATCH v2 5/5] Update documentation Todd
2015-01-16 0:30 ` [PATCH v3 1/5] Add failing unit tests for indexed mime types Todd
2015-01-17 10:44 ` David Bremner
2015-01-16 0:30 ` [PATCH v3 2/5] Add the NOTMUCH_FEATURE_INDEXED_MIMETYPES database feature Todd
2015-01-16 0:30 ` [PATCH v3 3/5] Add indexing for the mimetype term Todd
2015-01-17 15:21 ` David Bremner
2015-01-17 16:41 ` Todd
2015-01-16 0:30 ` [PATCH v3 4/5] Update completions for Emacs and bash Todd
2015-01-16 0:30 ` [PATCH v3 5/5] Update documentation Todd
2015-01-17 15:28 ` David Bremner
2015-01-22 23:43 ` [PATCH v4 0/5] Index the content-type of MIME parts Todd
2015-01-24 15:57 ` David Bremner
2015-01-22 23:43 ` [PATCH v4 1/5] test: Add failing unit tests for indexed mime types Todd
2015-01-22 23:43 ` [PATCH v4 2/5] Add the NOTMUCH_FEATURE_INDEXED_MIMETYPES database feature Todd
2015-01-22 23:43 ` [PATCH v4 3/5] Add indexing for the mimetype term Todd
2015-01-22 23:43 ` [PATCH v4 4/5] Update completions for Emacs and bash Todd
2015-01-22 23:43 ` [PATCH v4 5/5] Update documentation Todd
2015-01-23 19:11 ` Jani Nikula
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k30u90nv.fsf@electricoding.com \
--to=todd@electricoding.com \
--cc=aclements@csail.mit.edu \
--cc=david@tethera.net \
--cc=notmuch@notmuchmail.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).