8c/7c11b383f44185bcafa7548698761ae5cf18f8

   1 Return-Path: <jrollins@finestructure.net>\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 C4A9A431FAE\r
   6         for <notmuch@notmuchmail.org>; Fri,  6 Apr 2012 09:20:10 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id xH1P7IqTWB6S for <notmuch@notmuchmail.org>;\r
  16         Fri,  6 Apr 2012 09:20:07 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id D4BE4431FB6\r
  20         for <notmuch@notmuchmail.org>; Fri,  6 Apr 2012 09:20:04 -0700 (PDT)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id 7C38D66E0107;\r
  23         Fri,  6 Apr 2012 09:20:02 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
  25 Received: from finestructure.net (DHCP-123-229.caltech.edu [131.215.123.229])\r
  26         (Authenticated sender: jrollins)\r
  27         by earth-doxen-submit (Postfix) with ESMTP id 69CFC66E010E;\r
  28         Fri,  6 Apr 2012 09:19:55 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id 522A23BC; Fri,  6 Apr 2012 09:19:55 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH v3 0/6] Move --no-exclude to --exclude=(true|false|flag)\r
  34 In-Reply-To: <87ty0w50r7.fsf@qmul.ac.uk>\r
  35 References: <1333716551-29153-1-git-send-email-markwalters1009@gmail.com>\r
  36         <87bon47v0o.fsf@servo.finestructure.net>\r
  37         <87ty0w50r7.fsf@qmul.ac.uk>\r
  38 User-Agent: Notmuch/0.12+93~gd54240d (http://notmuchmail.org) Emacs/23.4.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Fri, 06 Apr 2012 09:19:52 -0700\r
  41 Message-ID: <877gxs7ryv.fsf@servo.finestructure.net>\r
  42 MIME-Version: 1.0\r
  43 Content-Type: multipart/signed; boundary="=-=-=";\r
  44         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  45 X-BeenThere: notmuch@notmuchmail.org\r
  46 X-Mailman-Version: 2.1.13\r
  47 Precedence: list\r
  48 List-Id: "Use and development of the notmuch mail system."\r
  49         <notmuch.notmuchmail.org>\r
  50 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  52 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  53 List-Post: <mailto:notmuch@notmuchmail.org>\r
  54 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  55 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  56         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  57 X-List-Received-Date: Fri, 06 Apr 2012 16:20:11 -0000\r
  58 \r
  59 --=-=-=\r
  60 \r
  61 On Fri, Apr 06 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  62 > There are some exclude tests for count in test/count. Do you think it\r
  63 > needs to be a full test everything like in test/search-exclude?\r
  64 \r
  65 Ah yes.  Somehow I missed that.\r
  66 \r
  67 I might prefer to keep all exclude tests in one test script.  In which\r
  68 case it would be good to move the count exclude tests there, and maybe\r
  69 round it out by testing both exclude options.\r
  70 \r
  71 > There aren't any tests for show exclude, but there aren't very many\r
  72 > tests explicitly for show anyway. My preference would be to add\r
  73 > something like\r
  74 > id:"1332171061-27983-1-git-send-email-markwalters1009@gmail.com" and add\r
  75 > some tests to that.\r
  76 \r
  77 Oh wow I wonder why that hasn't been pushed yet.  That's a bunch of new\r
  78 good tests.  It's a long changeset, which is probably why there aren't\r
  79 any reviews, but it's all test additions so we should definitely\r
  80 fast-track it.  I'll try to look at it this weekend.\r
  81 \r
  82 > I could add some tests for show-excludes in the same format as the\r
  83 > search-exclude tests though if you think that is better.\r
  84 \r
  85 As I mentioned above, I personally think it would nice to test all the\r
  86 exclude functionality in one test script.  So I would vote for adding\r
  87 some show exclude tests there.  Definitely doesn't need to be as\r
  88 exhaustive as the search tests, but at least covering both cli options.\r
  89 \r
  90 jamie.\r
  91 \r
  92 --=-=-=\r
  93 Content-Type: application/pgp-signature\r
  94 \r
  95 -----BEGIN PGP SIGNATURE-----\r
  96 Version: GnuPG v1.4.12 (GNU/Linux)\r
  97 \r
  98 iQIcBAEBCAAGBQJPfxepAAoJEO00zqvie6q8q9AP/jaBXI13C8YX/GQzxa7YPgdH\r
  99 yE0tmUVct6IkghiA4mxYDh8Kw/qYuA5MIdAoqphfJF9kQlfOyHHOMaaPTn3fj/Ij\r
 100 /SXCH11EyDhoTlJKkAJ303/uHCbtbK2JIqBKXFXpQiTbrrb5+6IJw0lWk2y4+4pO\r
 101 BRVo3zy0Zx2NYrX9hxGmF2G7Fhn6sJNTcbWJTtoP3Oh+TDsd+28R4b0ujJVIm3N4\r
 102 QjW3QTS3vIAOpyFt1lsvnosRRwPD39fN/Yf5gMR/VobF0v021F/uk+Z16VxgTaWu\r
 103 Higy4nivYCGn+7IXV8BmmzgYJOyxIIBO0D3cfFsgECnJjOUBFbYtauAm2kGotXrV\r
 104 8+PgL9Bt3yDR7zTNsVUP6Il8FRNn6n2YOuWXRSKcFfvK8wPP98FOvO/2AeBsHNVW\r
 105 9/a2Wtor03DRzxPoZhFDvTpqRsIayIauCDvKh19AUT1gdVJod+oJEa+/pzAGbmHP\r
 106 jxPiQPePeYdvDqym0RWXAMUFdKRXWbTFU0qKKvD3zcwuzNHJiOzo7uV2Sfvzz4xD\r
 107 Ke8TVuWJxskXyzI/rqMY+c189xH++5uhDqsZy7AY+3WyuMkbDvFkX4Bx48XBEm2O\r
 108 FKUJ7JunseCYRUEktUvJlawYn/PY95UICoOeCBA8kgFTZievWI7h9JPpowhq+O6z\r
 109 P6l+uhFFdgNz8/j6BCdB\r
 110 =pLRB\r
 111 -----END PGP SIGNATURE-----\r
 112 --=-=-=--\r