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 DB801431FAF
\r
6 for <notmuch@notmuchmail.org>; Wed, 1 Feb 2012 12:37:25 -0800 (PST)
\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org
\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 nb2ESdGolk3p for <notmuch@notmuchmail.org>;
\r
16 Wed, 1 Feb 2012 12:37:25 -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 03574431FAE
\r
21 for <notmuch@notmuchmail.org>; Wed, 1 Feb 2012 12:37:24 -0800 (PST)
\r
22 Received: by wibhi8 with SMTP id hi8so1320789wib.26
\r
23 for <notmuch@notmuchmail.org>; Wed, 01 Feb 2012 12:37:22 -0800 (PST)
\r
24 Received: by 10.180.24.105 with SMTP id t9mr99507wif.19.1328128642555;
\r
25 Wed, 01 Feb 2012 12:37:22 -0800 (PST)
\r
26 Received: from localhost ([109.131.39.11])
\r
27 by mx.google.com with ESMTPS id q2sm35342124wiy.7.2012.02.01.12.37.21
\r
28 (version=TLSv1/SSLv3 cipher=OTHER);
\r
29 Wed, 01 Feb 2012 12:37:21 -0800 (PST)
\r
30 From: Pieter Praet <pieter@praet.org>
\r
31 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org
\r
32 Subject: Re: emacs: quote MML tags in replies
\r
33 In-Reply-To: <87d39ymyb4.fsf@gmail.com>
\r
34 References: <1326998589-37187-1-git-send-email-aaronecay@gmail.com>
\r
35 <1328064581-13949-1-git-send-email-dmitry.kurochkin@gmail.com>
\r
36 <87vcnq64ra.fsf@praet.org> <87d39ymyb4.fsf@gmail.com>
\r
37 User-Agent: Notmuch/0.11+132~ga81001f (http://notmuchmail.org) Emacs/23.3.1
\r
38 (x86_64-unknown-linux-gnu)
\r
39 Date: Wed, 01 Feb 2012 21:35:26 +0100
\r
40 Message-ID: <87ipjqb8c1.fsf@praet.org>
\r
42 Content-Type: text/plain; charset=us-ascii
\r
43 X-BeenThere: notmuch@notmuchmail.org
\r
44 X-Mailman-Version: 2.1.13
\r
46 List-Id: "Use and development of the notmuch mail system."
\r
47 <notmuch.notmuchmail.org>
\r
48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,
\r
49 <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>
\r
50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>
\r
51 List-Post: <mailto:notmuch@notmuchmail.org>
\r
52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>
\r
53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,
\r
54 <mailto:notmuch-request@notmuchmail.org?subject=subscribe>
\r
55 X-List-Received-Date: Wed, 01 Feb 2012 20:37:26 -0000
\r
57 On Wed, 01 Feb 2012 18:18:55 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
\r
58 > On Wed, 01 Feb 2012 14:51:37 +0100, Pieter Praet <pieter@praet.org> wrote:
\r
59 > > On Wed, 1 Feb 2012 06:49:39 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
\r
62 > > > Thanks for your work! I took the liberty to do some cleanups for your
\r
63 > > > patch. Below is a detailed list of changes.
\r
66 > > Thanks to the both of you!
\r
69 > > > Hope this helps.
\r
71 > > > Changes since v2:
\r
73 > > > * change patch names to be consistent with others:
\r
75 > > > - s/emacs:/test:/ for the test patch
\r
77 > > > - lower case the first word after colon in the patch title
\r
79 > > > * polish NEWS wording, move it to 0.12 section
\r
81 > > > * add comment to `mml-quote-region' call, as suggested by Tomi [1]
\r
83 > > > * fix and clean up the test:
\r
85 > > > - set `notmuch-fcc-dirs' to nil to avoid adding the Fcc header,
\r
86 > > > otherwise it breaks the test on other systems as pointed by
\r
90 > > Could also have been avoided by adding the expected result inline,
\r
91 > > and using "Fcc: $(pwd)/mail/sent". I'll send an updated patch to
\r
95 > I do not understand how this is better. [...]
\r
97 Well, the expected output is only 7 lines, so we might as well keep it
\r
98 inside the test itself (of which the advantages need no reiteration),
\r
99 and doing so would allow us to get the Fcc path in a more future-proof way;
\r
100 `notmuch-fcc-dirs' is more susceptible to deprecation than $MAIL_DIR et al.
\r
103 > [...] But please do not use $(pwd)
\r
104 > here. I know that other tests do that in the same case, but it is
\r
105 > wrong. There is $MAIL_DIR variable for this. See "notmuch-fcc-dirs set
\r
106 > to a string" test for an example.
\r
109 Good point! Patch follows.
\r
116 > > > - use default values for add_message parameters where possible
\r
118 > > > - use a sane subject value in add_message
\r
120 > > > - use shorter MML tag as produced by (mml-insert-part)
\r
122 > > > - indenting and other minor cleanups
\r
127 > > > [1] id:"m2wr89ioos.fsf@guru.guru-group.fi"
\r
128 > > > [2] id:"87ehugzycb.fsf@zancas.localnet"
\r
130 > > > _______________________________________________
\r
131 > > > notmuch mailing list
\r
132 > > > notmuch@notmuchmail.org
\r
133 > > > http://notmuchmail.org/mailman/listinfo/notmuch
\r
135 > > Despite my comment re Fcc, both LGTM.
\r