43/c8c83a554d06a00914b6bdd31f252f7d79051c

   1 Return-Path: <dmitry.kurochkin@gmail.com>\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 AE74D431FD0\r
   6         for <notmuch@notmuchmail.org>; Wed,  1 Jun 2011 23:49:39 -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.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id nD2U6J9Ns6DU for <notmuch@notmuchmail.org>;\r
  17         Wed,  1 Jun 2011 23:49:38 -0700 (PDT)\r
  18 Received: from mail-bw0-f53.google.com (mail-bw0-f53.google.com\r
  19         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id 15C19431FB6\r
  22         for <notmuch@notmuchmail.org>; Wed,  1 Jun 2011 23:49:37 -0700 (PDT)\r
  23 Received: by bwg12 with SMTP id 12so754261bwg.26\r
  24         for <notmuch@notmuchmail.org>; Wed, 01 Jun 2011 23:49:36 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=domainkey-signature:from:to:subject:in-reply-to:references\r
  27         :user-agent:date:message-id:mime-version:content-type;\r
  28         bh=eZQyBgGuIZ17ZiyO0LrCiwL0JRQ7eJAA46Wzx1O1vRM=;\r
  29         b=qdoOPvfT7Cn5uw/E9prst56VLnNVgTYV+RPxXeWiL4vN7wHbuihasUkcgi2xN4/qnm\r
  30         kG7+71zmU4e03c9bTVrNeNkVXnpznNjguOQO6EqrmkXkXNT5To4LP4hIuSFFSA49Itf6\r
  31         ivQaadi83OQ0FT2MTR3uGYSFwkgaVs9MhJAf8=\r
  32 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;\r
  33         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  34         :mime-version:content-type;\r
  35         b=m1HliZv+1Vz9LKQSjuks1XH0Qtjc2u98Tf4EoCjCRKvJ++Nd0p4nwDAsHnVFQx3kEq\r
  36         lF5+MW6IFVfqIkEBKO2wj5LX1CXN65egoqWADFvD/K5d8qVscOoxzSepSFsyZfXLpMVL\r
  37         Jq195LY94lmEvCX6HeRfpcIjrJywt2JeAuM6k=\r
  38 Received: by 10.204.3.196 with SMTP id 4mr350749bko.188.1306997376283;\r
  39         Wed, 01 Jun 2011 23:49:36 -0700 (PDT)\r
  40 Received: from localhost ([91.144.186.21])\r
  41         by mx.google.com with ESMTPS id t23sm220130bkf.4.2011.06.01.23.49.33\r
  42         (version=TLSv1/SSLv3 cipher=OTHER);\r
  43         Wed, 01 Jun 2011 23:49:34 -0700 (PDT)\r
  44 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  45 To: Carl Worth <cworth@cworth.org>, Notmuch Mail <notmuch@notmuchmail.org>\r
  46 Subject: Re: [PATCH 14/25] Fix old style notmuch-fcc-dirs configuration check.\r
  47 In-Reply-To: <874o48yg28.fsf@yoom.home.cworth.org>\r
  48 References: <1306619520-25730-1-git-send-email-jrollins@finestructure.net>\r
  49         <1306619520-25730-2-git-send-email-jrollins@finestructure.net>\r
  50         <1306619520-25730-3-git-send-email-jrollins@finestructure.net>\r
  51         <1306619520-25730-4-git-send-email-jrollins@finestructure.net>\r
  52         <1306619520-25730-5-git-send-email-jrollins@finestructure.net>\r
  53         <1306619520-25730-6-git-send-email-jrollins@finestructure.net>\r
  54         <1306619520-25730-7-git-send-email-jrollins@finestructure.net>\r
  55         <1306619520-25730-8-git-send-email-jrollins@finestructure.net>\r
  56         <1306619520-25730-9-git-send-email-jrollins@finestructure.net>\r
  57         <1306619520-25730-10-git-send-email-jrollins@finestructure.net>\r
  58         <1306619520-25730-11-git-send-email-jrollins@finestructure.net>\r
  59         <1306619520-25730-12-git-send-email-jrollins@finestructure.net>\r
  60         <1306619520-25730-13-git-send-email-jrollins@finestructure.net>\r
  61         <1306619520-25730-14-git-send-email-jrollins@finestructure.net>\r
  62         <1306619520-25730-15-git-send-email-jrollins@finestructure.net>\r
  63         <874o48yg28.fsf@yoom.home.c worth.org>\r
  64 User-Agent: Notmuch/0.5-208-g109ec1a (http://notmuchmail.org) Emacs/23.3.1\r
  65         (x86_64-pc-linux-gnu)\r
  66 Date: Thu, 02 Jun 2011 10:49:57 +0400\r
  67 Message-ID: <87lixkpw16.fsf@gmail.com>\r
  68 MIME-Version: 1.0\r
  69 Content-Type: text/plain; charset=us-ascii\r
  70 X-BeenThere: notmuch@notmuchmail.org\r
  71 X-Mailman-Version: 2.1.13\r
  72 Precedence: list\r
  73 List-Id: "Use and development of the notmuch mail system."\r
  74         <notmuch.notmuchmail.org>\r
  75 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  76         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  77 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  78 List-Post: <mailto:notmuch@notmuchmail.org>\r
  79 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  80 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  81         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  82 X-List-Received-Date: Thu, 02 Jun 2011 06:49:40 -0000\r
  83 \r
  84 Hi Carl.\r
  85 \r
  86 On Wed, 01 Jun 2011 22:10:07 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  87 > On Sat, 28 May 2011 14:51:49 -0700, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  88 > Hi Jamie,\r
  89 > \r
  90 > I've pushed the next few patches up to this point, (with only one\r
  91 > functional change---I fixed a new test case to correctly use\r
  92 > notmuch_search_sanitize to avoid spurious failures unmatching thread ID\r
  93 > values).\r
  94 > \r
  95 > This patch, however, isn't ready. The big problem is in this commit\r
  96 > message:\r
  97 > \r
  98 > > From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  99 > > \r
 100 > > In notmuch 0.5 notmuch-fcc-dirs style changed.  The previous code\r
 101 > > did not correctly identify an old configuration and, as a\r
 102 > > consequence, broke new configurations.\r
 103 > \r
 104 > There are several things there that are too vague, ("previous code",\r
 105 > "old configuration", "new configurations").\r
 106 \r
 107 Well, it says that changes are in notmuch 0.5.  So "old" and "previous"\r
 108 refer to pre-0.5 (i.e. 0.4) and "new" refers to 0.5.\r
 109 \r
 110 > What kind of configuration\r
 111 > is broken?\r
 112 \r
 113 Any configuration when `notmuch-fcc-dirs' is a list.  That variable has\r
 114 a nice documentation.\r
 115 \r
 116 > How does the change here help?\r
 117 > \r
 118 \r
 119 It fixes detection of old-style pre-0.5 setting for `notmuch-fcc-dirs'.\r
 120 \r
 121 > It would be easier to understand the code if there were a corresponding\r
 122 > test case for it. I'd even be willing to help write a test case, but\r
 123 > there are not enough specifics in that commit message to even tell me\r
 124 > what to test.\r
 125 > \r
 126 \r
 127 I do not think we need a test for this fix.  What we need are tests for\r
 128 FCC functionality when notmuch-fcc-dirs is a list.\r
 129 \r
 130 > >       ((and (listp notmuch-fcc-dirs)\r
 131 > > -           (= 1 (length (car notmuch-fcc-dirs))))\r
 132 > > +           (stringp (car notmuch-fcc-dirs)))\r
 133 > \r
 134 > So the "old configuration" was a single string?\r
 135 \r
 136 No, it is a list with a string as the first element.  May I refer you to\r
 137 "git show 0.4:emacs/notmuch-maildir-fcc.el" and current\r
 138 emacs/notmuch-maildir-fcc.el?  It has good documentation with examples\r
 139 for notmuch-fcc-dirs.  Also NEWS for 0.5 describe this change.\r
 140 \r
 141 > And this has been\r
 142 > inadvertently broken since 0.5?\r
 143 > \r
 144 \r
 145 Old configuration format was changed in 0.5 in an incompatible way.\r
 146 There is a check for the unsupported old-style configuration.  But the\r
 147 check is broken and results in an error when running with a valid\r
 148 new-style configuration.\r
 149 \r
 150 > >        ;; Old style - no longer works.\r
 151 > >        (error "Invalid `notmuch-fcc-dirs' setting (old style)"))\r
 152 > \r
 153 > Yikes. That vague phrasing ("old style") is already in the code in error\r
 154 > messages as well.\r
 155 > \r
 156 \r
 157 Yes.\r
 158 \r
 159 > Dmitry, can you help me know what's going on here?\r
 160 \r
 161 I hope above helps.\r
 162 \r
 163 > (Preferably by\r
 164 > sending a newer commit with a more thorough commit message.)\r
 165 > \r
 166 \r
 167 I am not sure what you expect from the commit message here.  IMO it is\r
 168 enough for this small bugfix and those who interested can always refer\r
 169 to documentation for details.\r
 170 \r
 171 Regards,\r
 172   Dmitry\r
 173 \r
 174 > Thanks,\r
 175 > \r
 176 > -Carl\r
 177 Non-text part: application/pgp-signature\r
 178 > _______________________________________________\r
 179 > notmuch mailing list\r
 180 > notmuch@notmuchmail.org\r
 181 > http://notmuchmail.org/mailman/listinfo/notmuch\r