d3/d29be85d318da3b10be8101eaff23db4c2b78c

   1 Return-Path: <tomi.ollila@nixu.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 3E7074074C9\r
   6         for <notmuch@notmuchmail.org>; Fri,  6 Jan 2012 14:56:24 -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 x79x299Cefvj for <notmuch@notmuchmail.org>;\r
  16         Fri,  6 Jan 2012 14:56:23 -0800 (PST)\r
  17 Received: from mail-gw3.nixu.fi (mail-gw3.nixu.fi [193.209.237.7])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 5FF104074C7\r
  21         for <notmuch@notmuchmail.org>; Fri,  6 Jan 2012 14:56:23 -0800 (PST)\r
  22 Received: from pps.filterd (mail-gw3 [127.0.0.1])\r
  23         by mail-gw3.nixu.fi (8.14.4/8.14.4) with SMTP id q06MsCwY028833;\r
  24         Sat, 7 Jan 2012 00:56:20 +0200\r
  25 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  26         by mail-gw3.nixu.fi with ESMTP id 114cs0xu8b-1\r
  27         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  28         Sat, 07 Jan 2012 00:56:20 +0200\r
  29 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  30         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
  31         q06MuJBe011453; Sat, 7 Jan 2012 00:56:19 +0200\r
  32 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  33 To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH] emacs: Helpers for notmuch developers.\r
  35 In-Reply-To: <1325685678-12710-1-git-send-email-dme@dme.org>\r
  36 References: <1325685678-12710-1-git-send-email-dme@dme.org>\r
  37 User-Agent: Notmuch/0.10.2+157~g442d405 (http://notmuchmail.org) Emacs/23.3.1\r
  38         (i686-pc-linux-gnu)\r
  39 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  40         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  41         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  42 Date: Sat, 07 Jan 2012 00:56:19 +0200\r
  43 Message-ID: <yf6sjjsigoc.fsf@taco2.nixu.fi>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: text/plain; charset=us-ascii\r
  46 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.5.7110,\r
  47  1.0.211,       0.0.0000        definitions=2012-01-05_05:2012-01-05, 2012-01-05,\r
  48         1970-01-01 signatures=0\r
  49 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0\r
  50         ipscore=0 suspectscore=0\r
  51         phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0\r
  52         reason=mlx\r
  53         scancount=1 engine=6.0.2-1012030000 definitions=main-1201060286\r
  54 X-BeenThere: notmuch@notmuchmail.org\r
  55 X-Mailman-Version: 2.1.13\r
  56 Precedence: list\r
  57 List-Id: "Use and development of the notmuch mail system."\r
  58         <notmuch.notmuchmail.org>\r
  59 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  61 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  62 List-Post: <mailto:notmuch@notmuchmail.org>\r
  63 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  64 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  66 X-List-Received-Date: Fri, 06 Jan 2012 22:56:24 -0000\r
  67 \r
  68 On Wed,  4 Jan 2012 14:01:18 +0000, David Edmondson <dme@dme.org> wrote:\r
  69 > ---\r
  70 > \r
  71 > I've been using this for a few days and decided to share it to get\r
  72 > feedback. Reviewing patches can be tedious, so I tried to make things\r
  73 > a little simpler.\r
  74 > \r
  75 > To use this, load the file and then from `notmuch-show-mode' invoke\r
  76 > `notmuch-dev-show-review-patch'. It assumes that any open messages\r
  77 > contain patches and attempts to build a repository with the patches\r
  78 > applied.\r
  79 > \r
  80 > General management (i.e. keeping up to date) of the repository it uses\r
  81 > is your responsibility, as is cleaning out old branches. You can, of\r
  82 > course, just delete the temporary repository after using it - the code\r
  83 > will re-create it next time.\r
  84 > \r
  85 > If you have a slow network connection then a local copy of the main\r
  86 > repository can be specified by changing\r
  87 > `notmuch-dev-master-repository'.\r
  88 > \r
  89 > Comments?\r
  90 \r
  91 I think this is great. I've worked on something related but the emacs MUA\r
  92 "integration" makes patching simpler (My thing worked on shell and would\r
  93 require a 'recipe' file to describe HEAD commit and patch message id:s).\r
  94 \r
  95 What I've done differently is that I already have cloned notmuch repository\r
  96 somewhere which I attempt to keep up-to-date. For repository which I'm\r
  97 going to patch I first do:\r
  98 \r
  99 git clone --local --shared --no-checkout /path/to/notmuch/repo notmuch-dev-$USER\r
 100 git --git-dir notmuch-dev-$USER/.git' config --unset remote.origin.url\r
 101 cd notmuch-dev-$USER\r
 102 git checkout -b patched $commit\r
 103 \r
 104 I.e.\r
 105 \r
 106 1) I created a local repository clone, sharing objects without initial checkout.\r
 107 2) removed "remote" origin so accidental pushes are not possible\r
 108 3) checkout out tree from some commit (possible not remote HEAD) \r
 109 \r
 110 And this is done every time before new patchset is applied -- always on top\r
 111 of clean working directory.\r
 112 \r
 113 What do you think of this approach related to your way cloning the repo and\r
 114 then deleting/creating the branch. Just that developer may mess with the\r
 115 repository contents and then there is tedious working tree cleanup to be\r
 116 done (especially those who are not so fluent using git).\r
 117 \r
 118 \r
 119 Tomi\r
 120 \r