Re: [PATCH 1/2] Add Google Inc. to AUTHORS as a contributor.
[notmuch-archives.git] / 7f / 29d1ff8e009c8da53910ae3382ee22f12e9571
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 1738E431FC2\r
6         for <notmuch@notmuchmail.org>; Sun, 19 Jan 2014 05:58:47 -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 xWP1HmbItaen for <notmuch@notmuchmail.org>;\r
16         Sun, 19 Jan 2014 05:58:35 -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 70C1C431FBD\r
19         for <notmuch@notmuchmail.org>; Sun, 19 Jan 2014 05:58:35 -0800 (PST)\r
20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
21         by guru.guru-group.fi (Postfix) with ESMTP id 41E2C10005D;\r
22         Sun, 19 Jan 2014 15:58:29 +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: test: (gnu) tar(1) portability\r
26 In-Reply-To: <87ppny7bdq.fsf@zancas.localnet>\r
27 References: <m2mwj2aj9w.fsf@guru.guru-group.fi>\r
28         <87ppny7bdq.fsf@zancas.localnet>\r
29 User-Agent: Notmuch/0.17+34~g98b959f (http://notmuchmail.org) Emacs/24.3.1\r
30         (x86_64-unknown-linux-gnu)\r
31 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
32         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
33         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
34 Date: Sun, 19 Jan 2014 15:58:29 +0200\r
35 Message-ID: <m2ha906pkq.fsf@guru.guru-group.fi>\r
36 MIME-Version: 1.0\r
37 Content-Type: text/plain\r
38 X-BeenThere: notmuch@notmuchmail.org\r
39 X-Mailman-Version: 2.1.13\r
40 Precedence: list\r
41 List-Id: "Use and development of the notmuch mail system."\r
42         <notmuch.notmuchmail.org>\r
43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
46 List-Post: <mailto:notmuch@notmuchmail.org>\r
47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
50 X-List-Received-Date: Sun, 19 Jan 2014 13:58:47 -0000\r
51 \r
52 On Sat, Jan 11 2014, David Bremner <david@tethera.net> wrote:\r
53 \r
54 > Tomi Ollila <tomi.ollila@iki.fi> writes:\r
55 >>\r
56 >> 2) Change calls to 'tar' to format ${GNU_TAR:-tar} so testers can set \r
57 >>    environment variable GNU_TAR to the name/path of gnu tar when running\r
58 >>    tests (like GNU_TAR=/usr/local/bin/gtar make test or GNU_TAR=gtar ...)\r
59 >\r
60 > I don't really see any downside to this option; there's only a few calls\r
61 > to tar to replace.  Of course we could bikeshed about whether to write \r
62 >\r
63 > TAR=${GNU_TAR:-tar}\r
64 > ...\r
65 >\r
66 > ${TAR} --frub --blah \r
67 >\r
68 > instead of testing GNU_TAR everywhere.\r
69 \r
70 I also thought of this option, exactly in the same format, too.. \r
71 \r
72 >\r
73 > But, shrug...\r
74 \r
75 Yes... \r
76 \r
77 and this leads to probably more problematic issue: Mac OS X has\r
78 bash version 3 -- which cannot be used to run test scripts. For\r
79 that we'd also need a variabe (and some test script trickiness).\r
80 \r
81 This bash issue is easier to solve by tuning the PATH instead of\r
82 introducing another variable (or check for 'bash4' existence (ehhe;))\r
83 There for we could (that's probably me, then ;/) just document that\r
84 in test/README Prerequisites section..\r
85 \r
86 >\r
87 > d\r
88 \r
89 Tomi\r
90 \r