1 Return-Path: <tomi.ollila@iki.fi>
\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 4C0E1431FAF
\r
6 for <notmuch@notmuchmail.org>; Mon, 18 Feb 2013 08:25:40 -0800 (PST)
\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org
\r
11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]
\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 cEtLCRe14QAU for <notmuch@notmuchmail.org>;
\r
16 Mon, 18 Feb 2013 08:25:39 -0800 (PST)
\r
17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])
\r
18 by olra.theworths.org (Postfix) with ESMTP id A2B45431FAE
\r
19 for <notmuch@notmuchmail.org>; Mon, 18 Feb 2013 08:25:39 -0800 (PST)
\r
20 Received: from guru.guru-group.fi (localhost [IPv6:::1])
\r
21 by guru.guru-group.fi (Postfix) with ESMTP id EE66410014F;
\r
22 Mon, 18 Feb 2013 18:25:27 +0200 (EET)
\r
23 From: Tomi Ollila <tomi.ollila@iki.fi>
\r
24 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org
\r
25 Subject: Re: [PATCH 1/1] devel/release-checks.sh: added check that 1st NEWS
\r
27 In-Reply-To: <87ehgg7aar.fsf@zancas.localnet>
\r
28 References: <1359558095-22923-1-git-send-email-tomi.ollila@iki.fi>
\r
29 <87ehgg7aar.fsf@zancas.localnet>
\r
30 User-Agent: Notmuch/0.15+20~g7e29ae9 (http://notmuchmail.org) Emacs/24.2.1
\r
31 (x86_64-unknown-linux-gnu)
\r
32 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL
\r
33 $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F
\r
34 !)g;OY^,BjTbr)Np:%c_o'jj,Z
\r
35 Date: Mon, 18 Feb 2013 18:25:27 +0200
\r
36 Message-ID: <m24nh9si6g.fsf@guru.guru-group.fi>
\r
38 Content-Type: text/plain
\r
39 X-BeenThere: notmuch@notmuchmail.org
\r
40 X-Mailman-Version: 2.1.13
\r
42 List-Id: "Use and development of the notmuch mail system."
\r
43 <notmuch.notmuchmail.org>
\r
44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,
\r
45 <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>
\r
46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>
\r
47 List-Post: <mailto:notmuch@notmuchmail.org>
\r
48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>
\r
49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,
\r
50 <mailto:notmuch-request@notmuchmail.org?subject=subscribe>
\r
51 X-List-Received-Date: Mon, 18 Feb 2013 16:25:40 -0000
\r
53 On Sat, Feb 16 2013, David Bremner <david@tethera.net> wrote:
\r
55 > Tomi Ollila <tomi.ollila@iki.fi> writes:
\r
57 >> +echo -n "Checking that NEWS header is tidy... "
\r
58 >> +if [ "`exec sed 's/./=/g; 1q' NEWS`" = "`exec sed '1d; 2q' NEWS`" ]
\r
60 > This patch seems OK to me, although the execs might be a bit overkill
\r
61 > for a script that is run once ever few months.
\r
63 You're right. The 2 extra fork(2):s(*) here doesn't matter...
\r
65 In case you don't amend my patch in following weeks I'll send
\r
70 (*) FYI: It is only bash(1) which forks process to execute the external
\r
71 command in such a trivial cases. Most (if not all) other shells have
\r
72 (conservative) heuristics to avoid the "extra" fork there. If you're
\r
73 interested, my simple way to test this is 'strace -ff -o pidc cmd args...'
\r