0e/b38eff31d10735a1cadb25810a16012914d6b3

   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 117BF429E3E\r
   6         for <notmuch@notmuchmail.org>; Sat, 14 Jan 2012 01:01:38 -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 kOp+WwSK7gWO for <notmuch@notmuchmail.org>;\r
  16         Sat, 14 Jan 2012 01:01:37 -0800 (PST)\r
  17 Received: from mail-wi0-f181.google.com (mail-wi0-f181.google.com\r
  18         [209.85.212.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 58BD3429E38\r
  21         for <notmuch@notmuchmail.org>; Sat, 14 Jan 2012 01:01:37 -0800 (PST)\r
  22 Received: by wibhr12 with SMTP id hr12so1043007wib.26\r
  23         for <notmuch@notmuchmail.org>; Sat, 14 Jan 2012 01:01:36 -0800 (PST)\r
  24 Received: by 10.180.99.225 with SMTP id et1mr7397466wib.2.1326531696197;\r
  25         Sat, 14 Jan 2012 01:01:36 -0800 (PST)\r
  26 Received: from localhost ([109.131.75.86])\r
  27         by mx.google.com with ESMTPS id dr5sm20826036wib.0.2012.01.14.01.01.35\r
  28         (version=TLSv1/SSLv3 cipher=OTHER);\r
  29         Sat, 14 Jan 2012 01:01:35 -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: [PATCH v2] Output unmodified Content-Type header value for JSON\r
  33         format.\r
  34 In-Reply-To: <87vcogs35e.fsf@zancas.localnet>\r
  35 References: <1321659905-24367-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  36         <1321676321-27745-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  37         <87sjlktgi6.fsf@zancas.localnet> <87obw6prpd.fsf@gmail.com>\r
  38         <87fwfkluh3.fsf@praet.org> <87vcogs35e.fsf@zancas.localnet>\r
  39 User-Agent: Notmuch/0.10.2+115~gadd29f6 (http://notmuchmail.org) Emacs/23.3.1\r
  40         (x86_64-unknown-linux-gnu)\r
  41 Date: Sat, 14 Jan 2012 09:59:49 +0100\r
  42 Message-ID: <87zkdqpsl6.fsf@praet.org>\r
  43 MIME-Version: 1.0\r
  44 Content-Type: text/plain; charset=us-ascii\r
  45 X-BeenThere: notmuch@notmuchmail.org\r
  46 X-Mailman-Version: 2.1.13\r
  47 Precedence: list\r
  48 List-Id: "Use and development of the notmuch mail system."\r
  49         <notmuch.notmuchmail.org>\r
  50 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  52 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  53 List-Post: <mailto:notmuch@notmuchmail.org>\r
  54 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  55 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  56         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  57 X-List-Received-Date: Sat, 14 Jan 2012 09:01:38 -0000\r
  58 \r
  59 On Thu, 12 Jan 2012 23:16:29 -0400, David Bremner <david@tethera.net> wrote:\r
  60 > On Thu, 12 Jan 2012 18:08:08 +0100, Pieter Praet <pieter@praet.org> wrote:\r
  61 > > > > - The previous point is a bit of a counterargument to this, but in\r
  62 > \r
  63 > > > I couls separate it.  I made is a single patch to avoid having a\r
  64 > > > revision with broken emacs UI (and tests).\r
  65 > > > \r
  66 > \r
  67 > > I'd like to propose to always apply patch series on a *topic* branch\r
  68 > > which would then be merged back into 'master', thus avoiding this issue\r
  69 > > altogether whilst making it more obvious which patches belong together\r
  70 > > (eg. for easier cross-referencing with the ML).\r
  71 > \r
  72 > Hi Pieter;\r
  73 > \r
  74 > Sorry, I must have lost the thread somewhere.  What application are you\r
  75 > thinking about, and what issue do you think that topic branches avoid?\r
  76 > \r
  77 \r
  78 No specific application;\r
  79 \r
  80 In the case of patch series, it may occur that an individual commit\r
  81 leaves the code in a broken state.  One might argue that devs need to\r
  82 ensure that the project builds cleanly at every commit, but this can be\r
  83 prohibitively time-consuming or even simply impossible due to the need\r
  84 to keep changes atomic and area-specific (eg. changes to the binary and\r
  85 the Emacs UI should be separate commits irrespective of their relation).\r
  86 \r
  87 So it would make sense to consistently apply (certain, or perhaps all?)\r
  88 patch series on an ad-hoc branch which would then be merged back into\r
  89 'master', instead of being directly applied on 'master'.\r
  90 \r
  91 Aside from preventing broken builds (as was Dmitry's reason to consolidate\r
  92 a series touching on multiple distinct "spheres" of the code into a single\r
  93 patch), series of related commits would also be clearly delineated/clustered,\r
  94 facilitating source archaeology. (and all without making the commit log exude\r
  95 the odor of entropy incarnate)\r
  96 \r
  97 > d\r
  98 > \r
  99 \r
 100 \r
 101 Peace\r
 102 \r
 103 -- \r
 104 Pieter\r