56/d850288ea4c78deb2bb97173f2c433e5747799

   1 Return-Path: <cworth@cworth.org>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id AE8FA431FBC;\r
   6         Fri, 27 Nov 2009 21:48:49 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id iUJ3uomrOp6N; Fri, 27 Nov 2009 21:48:48 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id A4F6C431FAE;\r
  13         Fri, 27 Nov 2009 21:48:48 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: Jed Brown <jed@59A2.org>\r
  16 In-Reply-To: <87skc0domr.fsf@59A2.org>\r
  17 References: <87pr75q9va.fsf@yoom.home.cworth.org>\r
  18         <1259271410-15336-1-git-send-email-jed@59A2.org>\r
  19         <87fx80m5fe.fsf@yoom.home.cworth.org> <87skc0domr.fsf@59A2.org>\r
  20 Date: Fri, 27 Nov 2009 21:48:33 -0800\r
  21 Message-ID: <877htbtd1q.fsf@yoom.home.cworth.org>\r
  22 MIME-Version: 1.0\r
  23 Content-Type: multipart/signed; boundary="=-=-=";\r
  24         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  25 Cc: notmuch@notmuchmail.org\r
  26 Subject: Re: [notmuch] [PATCH 1/2] New function notmuch-search-operate-all:\r
  27  operate on all messages in the current query.\r
  28 X-BeenThere: notmuch@notmuchmail.org\r
  29 X-Mailman-Version: 2.1.12\r
  30 Precedence: list\r
  31 List-Id: "Use and development of the notmuch mail system."\r
  32         <notmuch.notmuchmail.org>\r
  33 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  34         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  35 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  36 List-Post: <mailto:notmuch@notmuchmail.org>\r
  37 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  38 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  40 X-List-Received-Date: Sat, 28 Nov 2009 05:48:49 -0000\r
  41 \r
  42 --=-=-=\r
  43 Content-Transfer-Encoding: quoted-printable\r
  44 \r
  45 On Fri, 27 Nov 2009 15:32:44 +0100, Jed Brown <jed@59A2.org> wrote:\r
  46 > On Fri, 27 Nov 2009 06:02:45 -0800, Carl Worth <cworth@cworth.org> wrote:\r
  47 > > Since this operates via a single call to "notmuch tag" you might mention\r
  48 > > here that all tag removals occur before any tag additions.\r
  49 >=20\r
  50 > I was unaware of this point, if I do\r
  51 >=20\r
  52 >   notmuch tag -inbox +star tag:inbox some-expression\r
  53 >=20\r
  54 > I will have starred nothing?\r
  55 \r
  56 Oh no, it's not that.\r
  57 \r
  58 It does one search and then does a single pass over all the messages\r
  59 returned. So the above will star things.\r
  60 \r
  61 The clarification about removing and adding really only matters if you\r
  62 have the same tag appearing as both -foo and +foo in the same command\r
  63 line, (which doesn't seem that useful, but there you have it).\r
  64 \r
  65 I don't recall now why I made the special effort to handle tag removal\r
  66 before tag addition rather than just applying tag changes in order. But\r
  67 since that's what the current "notmuch tag" code does, I thought I\r
  68 should point it out.\r
  69 \r
  70 > > > + (unless (string-match-p "^[\+\-][_\+\-\\w]+$" (car words))\r
  71 > > > +   (error "Action must be of the form `+thistag -that_tag'"))\r
  72 > >=20\r
  73 > > The error message has inconsistent "thistag" and "that_tag".\r
  74 >=20\r
  75 > That was somewhat intentional to illustrate that non-alphanumeric\r
  76 > characters could be used in tags.  Should the alphabet for tags be based\r
  77 > on a whitelist or blacklist?  It would be rather hard to validate a tag\r
  78 > operation when there is no assumption/restriction on the alphabet.\r
  79 \r
  80 Yeah, that's something we need to document. The current interface does\r
  81 use space-separated tag lists without any escaping. But the command line\r
  82 doesn't yet impose any restriction on tag names, and I did accidentally\r
  83 add a tag once of just " " (oops!).\r
  84 \r
  85 We should nail this down so that interface authors can know how to\r
  86 validate things consistently. Any suggestions?\r
  87 \r
  88 =2DCarl\r
  89 \r
  90 --=-=-=\r
  91 Content-Type: application/pgp-signature\r
  92 \r
  93 -----BEGIN PGP SIGNATURE-----\r
  94 Version: GnuPG v1.4.10 (GNU/Linux)\r
  95 \r
  96 iD8DBQFLELmy6JDdNq8qSWgRAqYTAJ9TUEBaw4AnaM6e501vABitworrAQCfR3sp\r
  97 KICLK+YlzhoIscT4ipXpcEA=\r
  98 =iq+/\r
  99 -----END PGP SIGNATURE-----\r
 100 --=-=-=--\r