[PATCH] nmbug: allow excluded messages

W. Trevor King wking at tremily.us
Sat Oct 22 14:39:37 PDT 2016


On Sat, Oct 22, 2016 at 08:59:40AM +0100, Mark Walters wrote:
> This makes nmbug work even if the notmuch mailing list messages are
> excluded (i.e., have a tag in the excluded tags list).
>> (I think this is the only place it is needed, but I don't use nmbug
> much so I haven't tested thoroughly.)
>> --- a/devel/nmbug/nmbug
> +++ b/devel/nmbug/nmbug
> @@ -580,7 +580,7 @@ def get_status():
>      maybe_deleted = _diff_index(index=index, filter='D')
>      for id, tags in maybe_deleted.items():
>          (_, stdout, stderr) = _spawn(
> -            args=['notmuch', 'search', '--output=files', 'id:{0}'.format(id)],
> +            args=['notmuch', 'search', '--output=files', '--exclude=false', 'id:{0}'.format(id)],
>              stdout=_subprocess.PIPE,
>              wait=True)

Besides this call, we also use ‘notmuch search …’ in get_tags.  We may
need an --exclude=false there too to cover folks who exclude a tag
used by nmbug.

And ‘notmuch dump …’ doesn't appear to take an --exclude flag,
although it's based on searching so I'm not sure why not.

Anyhow, the idea looks good to me.  I'm just not clear enough on
exclude handling to know if this patch sufficiently covers it.

Cheers,
Trevor

-- 
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20161022/921a367b/attachment.sig>


More information about the notmuch mailing list