[PATCH v2 0/5] Use Xapian query syntax for batch-tag dump/restore
David Bremner
david at tethera.net
Wed Dec 26 06:55:06 PST 2012
Austin Clements <amdragon at MIT.EDU> writes:
> This obsoletes
>
> id:1356415076-5692-1-git-send-email-amdragon at mit.edu
>
> In addition to incorporating all of David's suggestions, this reworks
> the boolean term parsing so it only handles the subset of quoting
> syntax used by make_boolean_term (which also happens to be all that we
> described in the man page for the format). The diff from v1 is below.
Implementation wise this series now looks OK to me; I think the dequote
routine could be slightly clearer, but it's a bit subjective at this
point.
IMHO the real question is if we like (or can live with) the resulting UI
for batch tagging. Mark ( id:87ehid5h64.fsf at qmul.ac.uk ) brings up the
question of whether tags with newlines should be somehow supported in a
batch-tagging context. The other UI issue I'm aware of is the fact that
tags with spaces needed to be quoted in two different ways for tag
operations and for queries. A relatively smooth upgrade path would be
to support Xapian quoting for tag operations in the future.
d
More information about the notmuch
mailing list