Re: [PATCH v4 01/16] add util/search-path.{c, h} to test for executables in $PATH
[notmuch-archives.git] / 1d / cbc5c994c5a51a2c357821504d7b92c5968b77
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 55D30429E25\r
6         for <notmuch@notmuchmail.org>; Mon, 21 Nov 2011 03:04:27 -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 KAiDbbTf+8FU for <notmuch@notmuchmail.org>;\r
16         Mon, 21 Nov 2011 03:04:22 -0800 (PST)\r
17 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\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 7A76B431FD0\r
21         for <notmuch@notmuchmail.org>; Mon, 21 Nov 2011 03:04:22 -0800 (PST)\r
22 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
23         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
24         pALB4IG7018294; Mon, 21 Nov 2011 13:04:19 +0200\r
25 From: Tomi Ollila <tomi.ollila@iki.fi>\r
26 To: Thomas Schwinge <thomas@schwinge.name>, notmuch@notmuchmail.org\r
27 Subject: Re: [PATCH] dump: Don't sort.\r
28 In-Reply-To: <87r51ab36z.fsf@zancas.localnet>\r
29 References: <1319884657-5574-1-git-send-email-thomas@schwinge.name>\r
30         <87r51ab36z.fsf@zancas.localnet>\r
31 User-Agent: Notmuch/0.10~rc1+16~gc832dad (http://notmuchmail.org) Emacs/23.3.1\r
32         (i686-pc-linux-gnu)\r
33 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
34         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
35         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
36 Date: Mon, 21 Nov 2011 13:04:18 +0200\r
37 Message-ID: <yf6sjlhn3d9.fsf@taco2.nixu.fi>\r
38 MIME-Version: 1.0\r
39 Content-Type: text/plain; charset=us-ascii\r
40 X-BeenThere: notmuch@notmuchmail.org\r
41 X-Mailman-Version: 2.1.13\r
42 Precedence: list\r
43 List-Id: "Use and development of the notmuch mail system."\r
44         <notmuch.notmuchmail.org>\r
45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
46         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
47 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
48 List-Post: <mailto:notmuch@notmuchmail.org>\r
49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
51         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
52 X-List-Received-Date: Mon, 21 Nov 2011 11:04:27 -0000\r
53 \r
54 On Mon, 14 Nov 2011 21:10:44 -0400, David Bremner <david@tethera.net> wrote:\r
55 > On Sat, 29 Oct 2011 12:37:37 +0200, Thomas Schwinge <thomas@schwinge.name> wrote:\r
56 > > From: Thomas Schwinge <thomas@schwinge.name>\r
57 > > \r
58 > > This improves usage experience considerably in the given scenario.\r
59 > > \r
60\r
61 > I'm not sure if I mentioned this only in IRC, so let me recap.\r
62\r
63 > Personally, I think this needs more information in the commit message,\r
64 > and (optionally) less in the comment. I would prefer to be able to\r
65 > understand what is going on just from the output of git log.\r
66 \r
67 I agree; the git commit message:\r
68 \r
69 --8<----8<----8<----8<----8<----8<----8<----8<----8<--\r
70 dump: Don't sort.\r
71 \r
72 This improves usage experience considerably in the given scenario.\r
73 --8<----8<----8<----8<----8<----8<----8<----8<----8<--\r
74 \r
75 Is not very informative. Better to describe the given scenario in commit\r
76 message and (perhaps!) tune the comment in the code.\r
77 \r
78 We've seen that the patch is very useful -- So, Thomas, please provide a \r
79 patch in a format it can be comfortably added to the repository :).\r
80 \r
81\r
82 > d\r
83 \r
84 Tomi\r