39/078c7ba8245b275f75de3f52263115a21fce3f

   1 Return-Path: <bremner@unb.ca>\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 F1EA6431FB6\r
   6         for <notmuch@notmuchmail.org>; Mon,  5 Nov 2012 18:17:04 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 70ZpCNs1j3X2 for <notmuch@notmuchmail.org>;\r
  16         Mon,  5 Nov 2012 18:17:04 -0800 (PST)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
  18         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 57E9B431FAE\r
  21         for <notmuch@notmuchmail.org>; Mon,  5 Nov 2012 18:17:04 -0800 (PST)\r
  22 Received: from fctnnbsc30w-156034089108.dhcp-dynamic.fibreop.nb.bellaliant.net\r
  23         ([156.34.89.108] helo=zancas.localnet)\r
  24         by tesseract.cs.unb.ca with esmtpsa\r
  25         (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.72)\r
  26         (envelope-from <bremner@unb.ca>)\r
  27         id 1TVYio-0004wt-6m; Mon, 05 Nov 2012 22:17:00 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
  29         (envelope-from <bremner@unb.ca>)\r
  30         id 1TVYiS-0004ns-NR; Mon, 05 Nov 2012 22:16:36 -0400\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: Daniel <quite@hack.org>, Louis Rilling <l.rilling@av7.net>\r
  33 Subject: Re: [PATCH 0/4 v4] lib: Maildir flags synchronization fixes\r
  34 In-Reply-To: <20121106001633.GA14918@kelebek.lublin.se>\r
  35 References: <20121106001633.GA14918@kelebek.lublin.se>\r
  36 User-Agent: Notmuch/0.14+75~g984212d (http://notmuchmail.org) Emacs/24.1.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Mon, 05 Nov 2012 22:16:36 -0400\r
  39 Message-ID: <87d2zro4qj.fsf@zancas.localnet>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: text/plain\r
  42 X-Spam_bar: -\r
  43 Cc: notmuch@notmuchmail.org\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Tue, 06 Nov 2012 02:17:05 -0000\r
  57 \r
  58 Daniel <quite@hack.org> writes:\r
  59 \r
  60 > On Mon, Nov 21, 2011 at 08:41:42PM -0400, David Bremner wrote:\r
  61 >> I pushed the first two patches in this series; the second two need to be\r
  62 >> updated for the new "broken test" framework, and reviewed.\r
  63 >\r
  64 > And old one... What came of this patch?\r
  65 >\r
  66 > I came across it, having been annoyed by the fact that notmuch broke my reading\r
  67 > habits by marking messages (in the mutt sense) as seen, which I hadn't really\r
  68 > seen yet in mutt. I tried out the patch; it applied and seems to work. What can\r
  69 > I do to make it be applied? Other than giving it some attention. I must say\r
  70 > that I have zero insight in the test framework, but if that's what's needed\r
  71 > from me...\r
  72 \r
  73 The patches need to be reviewed for correctness and style. Typically any\r
  74 non-trivial patch (or series) needs two reviews. You can see examples in\r
  75 the archives of how this works.  They are currently marked stale, which\r
  76 is supposed to mean they no long apply. On the other hand, if they apply\r
  77 for you, maybe that tagging is wrong (see http://notmuchmail.org/nmbug/\r
  78 for more information about patch tagging).\r
  79 \r
  80 By the way, it seems like commit b88030b might be relevant.\r
  81 \r
  82 d\r