04/9b98b6ffd91c33b75bd8bc6cf91b352ed42898

   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 5C3A1431FD0\r
   6         for <notmuch@notmuchmail.org>; Wed,  1 Jun 2011 22:10:18 -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 vqxhk2cWlMg2 for <notmuch@notmuchmail.org>;\r
  16         Wed,  1 Jun 2011 22:10:17 -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 73485431FB6\r
  19         for <notmuch@notmuchmail.org>; Wed,  1 Jun 2011 22:10:17 -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 23A8929A4E5;\r
  22         Wed,  1 Jun 2011 22:10:16 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id 090CE54C02C; Wed,  1 Jun 2011 22:10:16 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Jameson Graef Rollins <jrollins@finestructure.net>,\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: <1306619520-25730-15-git-send-email-jrollins@finestructure.net>\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 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  46         (i486-pc-linux-gnu)\r
  47 Date: Wed, 01 Jun 2011 22:10:07 -0700\r
  48 Message-ID: <874o48yg28.fsf@yoom.home.cworth.org>\r
  49 MIME-Version: 1.0\r
  50 Content-Type: multipart/signed; boundary="=-=-=";\r
  51         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  52 X-BeenThere: notmuch@notmuchmail.org\r
  53 X-Mailman-Version: 2.1.13\r
  54 Precedence: list\r
  55 List-Id: "Use and development of the notmuch mail system."\r
  56         <notmuch.notmuchmail.org>\r
  57 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  59 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  60 List-Post: <mailto:notmuch@notmuchmail.org>\r
  61 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  62 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  64 X-List-Received-Date: Thu, 02 Jun 2011 05:10:18 -0000\r
  65 \r
  66 --=-=-=\r
  67 Content-Transfer-Encoding: quoted-printable\r
  68 \r
  69 On Sat, 28 May 2011 14:51:49 -0700, Jameson Graef Rollins <jrollins@finestr=\r
  70 ucture.net> wrote:\r
  71 Hi Jamie,\r
  72 \r
  73 I've pushed the next few patches up to this point, (with only one\r
  74 functional change---I fixed a new test case to correctly use\r
  75 notmuch_search_sanitize to avoid spurious failures unmatching thread ID\r
  76 values).\r
  77 \r
  78 This patch, however, isn't ready. The big problem is in this commit\r
  79 message:\r
  80 \r
  81 > From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  82 >=20\r
  83 > In notmuch 0.5 notmuch-fcc-dirs style changed.  The previous code\r
  84 > did not correctly identify an old configuration and, as a\r
  85 > consequence, broke new configurations.\r
  86 \r
  87 There are several things there that are too vague, ("previous code",\r
  88 "old configuration", "new configurations"). What kind of configuration\r
  89 is broken? How does the change here help?\r
  90 \r
  91 It would be easier to understand the code if there were a corresponding\r
  92 test case for it. I'd even be willing to help write a test case, but\r
  93 there are not enough specifics in that commit message to even tell me\r
  94 what to test.\r
  95 \r
  96 >         ((and (listp notmuch-fcc-dirs)\r
  97 > -             (=3D 1 (length (car notmuch-fcc-dirs))))\r
  98 > +             (stringp (car notmuch-fcc-dirs)))\r
  99 \r
 100 So the "old configuration" was a single string? And this has been\r
 101 inadvertently broken since 0.5?\r
 102 \r
 103 >          ;; Old style - no longer works.\r
 104 >          (error "Invalid `notmuch-fcc-dirs' setting (old style)"))\r
 105 \r
 106 Yikes. That vague phrasing ("old style") is already in the code in error\r
 107 messages as well.\r
 108 \r
 109 Dmitry, can you help me know what's going on here? (Preferably by\r
 110 sending a newer commit with a more thorough commit message.)\r
 111 \r
 112 Thanks,\r
 113 \r
 114 =2DCarl\r
 115 \r
 116 --=-=-=\r
 117 Content-Type: application/pgp-signature\r
 118 \r
 119 -----BEGIN PGP SIGNATURE-----\r
 120 Version: GnuPG v1.4.11 (GNU/Linux)\r
 121 \r
 122 iEYEARECAAYFAk3nGy8ACgkQ6JDdNq8qSWhjtgCgqoIcakwlAb/jkrpdX/gldWqb\r
 123 uykAn1jqk3hyOh0Vq/Pje1ubAoF84P4S\r
 124 =1ZcD\r
 125 -----END PGP SIGNATURE-----\r
 126 --=-=-=--\r