ed/06a8084a180bd6e95c2fdb733521e2aea6acf1

   1 Return-Path: <pieter@praet.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 55E8C429E26\r
   6         for <notmuch@notmuchmail.org>; Thu, 12 Jan 2012 09:27:20 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 VIn3fO7UKVbu for <notmuch@notmuchmail.org>;\r
  16         Thu, 12 Jan 2012 09:27:19 -0800 (PST)\r
  17 Received: from mail-ww0-f45.google.com (mail-ww0-f45.google.com\r
  18  [74.125.82.45])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
  19  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
  20  278B2429E25    for <notmuch@notmuchmail.org>; Thu, 12 Jan 2012 09:27:19 -0800\r
  21  (PST)\r
  22 Received: by wgbds11 with SMTP id ds11so1964852wgb.2\r
  23         for <notmuch@notmuchmail.org>; Thu, 12 Jan 2012 09:27:18 -0800 (PST)\r
  24 Received: by 10.180.20.39 with SMTP id k7mr2054077wie.6.1326389237862;\r
  25         Thu, 12 Jan 2012 09:27:17 -0800 (PST)\r
  26 Received: from localhost ([109.131.126.209])\r
  27         by mx.google.com with ESMTPS id ej7sm6493398wbb.20.2012.01.12.09.27.17\r
  28         (version=TLSv1/SSLv3 cipher=OTHER);\r
  29         Thu, 12 Jan 2012 09:27:17 -0800 (PST)\r
  30 From: Pieter Praet <pieter@praet.org>\r
  31 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  32 Subject: Re: revised patch for gmime init, with test.\r
  33 In-Reply-To: <1325388169-8444-1-git-send-email-david@tethera.net>\r
  34 References: <1325306261-21444-2-git-send-email-kaz.rag@gmail.com>\r
  35         <1325388169-8444-1-git-send-email-david@tethera.net>\r
  36 User-Agent: Notmuch/0.10.2+115~gadd29f6 (http://notmuchmail.org) Emacs/23.3.1\r
  37         (x86_64-unknown-linux-gnu)\r
  38 Date: Thu, 12 Jan 2012 18:25:38 +0100\r
  39 Message-ID: <871ur4ltnx.fsf@praet.org>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: text/plain; charset=us-ascii\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Thu, 12 Jan 2012 17:27:20 -0000\r
  55 \r
  56 On Sat, 31 Dec 2011 23:22:46 -0400, David Bremner <david@tethera.net> wrote:\r
  57 > It turns out that our existing (trivial) python test is enough to\r
  58 > catch this bug, but the corpus needs to be augmented.  This\r
  59 > augmentation is a bit intrusive so I'm thinking of cherry-picking only\r
  60 > the actual fix to the release branch.\r
  61 > \r
  62 \r
  63 Due to the ~same change being applied in multiple places (and thus\r
  64 with differing hashes), this has the potential of causing confusion\r
  65 and/or quite some extra work when debugging using git-bisect(1), so\r
  66 I'd like to propose that bugfixes for (to-be-)released code are only\r
  67 applied on the 'maint' branch ('release' in the case of Notmuch),\r
  68 and then immediately merged back into 'master'.  In fact, this would\r
  69 preferrably happen after *every* (series of) commit(s) on the 'maint'\r
  70 branch, to prevent issues like [1].\r
  71 \r
  72 \r
  73 In this specific case (g_mime_init fix), it would have sufficed to\r
  74 apply the test suite augmentation patches on 'master' and the bugfix\r
  75 only on 'maint', merging 'maint' into 'master', and then removing\r
  76 the "test_subtest_known_broken" line in a followup commit on 'master'.\r
  77 \r
  78 Thus, maintainers would have fewer merge conflicts to resolve,\r
  79 developers would at all times benefit from release-specific fixes,\r
  80 and (pre-)release users would have clear indication of what was fixed\r
  81 due to the test suite reporting "FIXED" instead of "PASS".\r
  82 \r
  83 \r
  84 Thanks!\r
  85 \r
  86 > Unfortunately the test message is 8 bit, so it may be encoded in some\r
  87 > inconvenient way for patch application. The message is attached to an\r
  88 > earlier message in the thread if you want to double check.\r
  89 > \r
  90 > I also wondered about putting g_type_init inside the (!initialized)\r
  91 > test, but decided against it on the grounds of minimality.\r
  92 > \r
  93 > I think we want to in the medium term factor out all of the\r
  94 > initialization code into one (probably private) function; we can clean\r
  95 > things up a bit more then.\r
  96 > \r
  97 > _______________________________________________\r
  98 > notmuch mailing list\r
  99 > notmuch@notmuchmail.org\r
 100 > http://notmuchmail.org/mailman/listinfo/notmuch\r
 101 \r
 102 \r
 103 Peace\r
 104 \r
 105 -- \r
 106 Pieter\r
 107 \r
 108 [1] id:"87mxa95u3d.fsf@zancas.localnet"\r