43/756b5a91d5b4db9a9a959446ff1217f92e3c53

   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 BDB3C431FD0\r
   6         for <notmuch@notmuchmail.org>; Thu, 23 Jun 2011 15:22:58 -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: 0.01\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.01 tagged_above=-999 required=5\r
  12         tests=[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 6kQ9Oq5cNocl for <notmuch@notmuchmail.org>;\r
  16         Thu, 23 Jun 2011 15:22:58 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 229F4431FB6\r
  19         for <notmuch@notmuchmail.org>; Thu, 23 Jun 2011 15:22:58 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id 9EF4629A505;\r
  22         Thu, 23 Jun 2011 15:22:56 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id 8CA0A254157; Thu, 23 Jun 2011 15:22:56 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>,\r
  27         Notmuch Mail <notmuch@notmuchmail.org>\r
  28 Subject: Re: [PATCH 14/25] Fix old style notmuch-fcc-dirs configuration check.\r
  29 In-Reply-To: <87ipsmpswj.fsf@gmail.com>\r
  30 References: <1306619520-25730-1-git-send-email-jrollins@finestructure.net>\r
  31         <1306619520-25730-2-git-send-email-jrollins@finestructure.net>\r
  32         <1306619520-25730-3-git-send-email-jrollins@finestructure.net>\r
  33         <1306619520-25730-4-git-send-email-jrollins@finestructure.net>\r
  34         <1306619520-25730-5-git-send-email-jrollins@finestructure.net>\r
  35         <1306619520-25730-6-git-send-email-jrollins@finestructure.net>\r
  36         <1306619520-25730-7-git-send-email-jrollins@finestructure.net>\r
  37         <1306619520-25730-8-git-send-email-jrollins@finestructure.net>\r
  38         <1306619520-25730-9-git-send-email-jrollins@finestructure.net>\r
  39         <1306619520-25730-10-git-send-email-jrollins@finestructure.net>\r
  40         <1306619520-25730-11-git-send-email-jrollins@finestructure.net>\r
  41         <1306619520-25730-12-git-send-email-jrollins@finestructure.net>\r
  42         <1306619520-25730-13-git-send-email-jrollins@finestructure.net>\r
  43         <1306619520-25730-14-git-send-email-jrollins@finestructure.net>\r
  44         <1306619520-25730-15-git-send-email-jrollins@finestructure.net>\r
  45         <874o48yg28.fsf@yoom.home.c worth.org> <87lixkpw16.fsf@gmail.com>\r
  46         <87oc2ellzn.fsf@yoom.home.cworth.org> <87ipsmpswj.fsf@gmail.com>\r
  47 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  48         (i486-pc-linux-gnu)\r
  49 Date: Thu, 23 Jun 2011 15:22:46 -0700\r
  50 Message-ID: <87hb7gdw7d.fsf@yoom.home.cworth.org>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: multipart/signed; boundary="=-=-=";\r
  53         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  54 Cc: David Bremner <bremner@debian.org>\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Thu, 23 Jun 2011 22:22:58 -0000\r
  68 \r
  69 --=-=-=\r
  70 Content-Transfer-Encoding: quoted-printable\r
  71 \r
  72 On Sat, 04 Jun 2011 00:22:04 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmai=\r
  73 l.com> wrote:\r
  74 > On Fri, 03 Jun 2011 13:05:00 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  75 > > > I do not think we need a test for this fix.  What we need are tests f=\r
  76 or\r
  77 > > > FCC functionality when notmuch-fcc-dirs is a list.\r
  78 > >=20\r
  79 > > Yes!\r
  80 \r
  81 I've written these now. And they do test this fix. What they show is\r
  82 that a legitimate setting (of notmuch-fcc-dirs as a list) was resulting\r
  83 in an error rather than working. That's a nasty little bug, (and poor\r
  84 coverage from our test suite before.\r
  85 \r
  86 >   Fix wrong-type-argument lisp error in `notmuch-fcc-header-setup' when\r
  87 >   `notmuch-fcc-dirs' is set to a list.  The error was in the\r
  88 >   `notmuch-fcc-dirs' format check which was changed in an incompatible\r
  89 >   way from 0.4 to 0.5.\r
  90 \r
  91 Thanks for the fixed wording. I've now pushed out the fix (along with\r
  92 the tests).\r
  93 \r
  94 With all the talk of "old style" vs. "new style" I was thinking that the\r
  95 bug only affected people with the old-style FCC setting. The bug is much\r
  96 worse than that, (preventing people from using the new list-based\r
  97 style).\r
  98 \r
  99 Anyway, thanks for the patch, Dmitry. And thanks for pushing me to take\r
 100 another look.\r
 101 \r
 102 David, I suggest including this fix (and its test) in the release\r
 103 branch.\r
 104 \r
 105 =2DCarl\r
 106 \r
 107 =2D-=20\r
 108 carl.d.worth@intel.com\r
 109 \r
 110 --=-=-=\r
 111 Content-Type: application/pgp-signature\r
 112 \r
 113 -----BEGIN PGP SIGNATURE-----\r
 114 Version: GnuPG v1.4.11 (GNU/Linux)\r
 115 \r
 116 iEYEARECAAYFAk4DvLYACgkQ6JDdNq8qSWiGZgCfbUBGxnMUytax1MpM1QG8H/2w\r
 117 t7sAn3S40trkxVf1Uh3Y0u0Mr5Mj4MJg\r
 118 =pHzP\r
 119 -----END PGP SIGNATURE-----\r
 120 --=-=-=--\r