Re: [PATCH] configure: only install bash completion if supported
[notmuch-archives.git] / 53 / 0d2a54d9d68ab9a062de91460761e1c8fa8e0e
1 Return-Path: <m.walters@qmul.ac.uk>\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 2080D431FBD\r
6         for <notmuch@notmuchmail.org>; Sun,  2 Feb 2014 09:43:16 -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: -1.098\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-1.098 tagged_above=-999 required=5\r
12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
13         NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
14 Received: from olra.theworths.org ([127.0.0.1])\r
15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
16         with ESMTP id jWQIeXdHS31S for <notmuch@notmuchmail.org>;\r
17         Sun,  2 Feb 2014 09:43:08 -0800 (PST)\r
18 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
19         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
20         (No client certificate requested)\r
21         by olra.theworths.org (Postfix) with ESMTPS id D846B431FBC\r
22         for <notmuch@notmuchmail.org>; Sun,  2 Feb 2014 09:43:07 -0800 (PST)\r
23 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
24         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
25         (envelope-from <m.walters@qmul.ac.uk>)\r
26         id 1WA14O-0001NQ-CG; Sun, 02 Feb 2014 17:43:02 +0000\r
27 Received: from 93-97-24-31.zone5.bethere.co.uk ([93.97.24.31] helo=localhost)\r
28         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.71)\r
29         (envelope-from <m.walters@qmul.ac.uk>)\r
30         id 1WA13U-0007dP-Vh; Sun, 02 Feb 2014 17:42:05 +0000\r
31 From: Mark Walters <markwalters1009@gmail.com>\r
32 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
33 Subject: Re: [PATCH] configure: only install bash completion if supported\r
34 In-Reply-To: <1391359677-12497-1-git-send-email-jani@nikula.org>\r
35 References: <1391359677-12497-1-git-send-email-jani@nikula.org>\r
36 User-Agent: Notmuch/0.15.2+484~gfb59956 (http://notmuchmail.org) Emacs/23.4.1\r
37         (x86_64-pc-linux-gnu)\r
38 Date: Sun, 02 Feb 2014 17:40:38 +0000\r
39 Message-ID: <877g9dqup5.fsf@qmul.ac.uk>\r
40 MIME-Version: 1.0\r
41 Content-Type: text/plain; charset=us-ascii\r
42 X-Sender-Host-Address: 93.97.24.31\r
43 X-QM-Geographic: According to ripencc,\r
44         this message was delivered by a machine in Britain (UK) (GB).\r
45 X-QM-SPAM-Info: Sender has good ham record.  :)\r
46 X-QM-Body-MD5: bc4a3a4d2c7913277a1edf16efa1e1a5 (of first 20000 bytes)\r
47 X-SpamAssassin-Score: 0.0\r
48 X-SpamAssassin-SpamBar: /\r
49 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
50         determine if it is\r
51         spam. We require at least 5.0 points to mark a message as spam.\r
52         This message scored 0.0 points. Summary of the scoring: \r
53         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
54         provider *      (markwalters1009[at]gmail.com)\r
55         *  0.0 AWL AWL: From: address is in the auto white-list\r
56 X-QM-Scan-Virus: ClamAV says the message is clean\r
57 X-BeenThere: notmuch@notmuchmail.org\r
58 X-Mailman-Version: 2.1.13\r
59 Precedence: list\r
60 List-Id: "Use and development of the notmuch mail system."\r
61         <notmuch.notmuchmail.org>\r
62 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
63         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
64 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
65 List-Post: <mailto:notmuch@notmuchmail.org>\r
66 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
67 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
68         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
69 X-List-Received-Date: Sun, 02 Feb 2014 17:43:16 -0000\r
70 \r
71 \r
72 This LGTM (untested)\r
73 \r
74 I think this is the right way to do this: I don't think we should\r
75 spam a user with error messages for attempting tab completion with\r
76 an older version of bash-completion (which I think you said was the case\r
77 with the other version)\r
78 \r
79 Best wishes\r
80 \r
81 Mark\r
82 \r
83 \r
84 On Sun, 02 Feb 2014, Jani Nikula <jani@nikula.org> wrote:\r
85 > Our bash completion depends on bash-completion 1.90 or later. Only\r
86 > install where available.\r
87 > ---\r
88 >  configure | 8 ++++++++\r
89 >  1 file changed, 8 insertions(+)\r
90 >\r
91 > diff --git a/configure b/configure\r
92 > index 13b6062..66aaedb 100755\r
93 > --- a/configure\r
94 > +++ b/configure\r
95 > @@ -360,6 +360,14 @@ else\r
96 >      have_valgrind=0\r
97 >  fi\r
98 >  \r
99 > +printf "Checking for bash-completion (>= 1.90)... "\r
100 > +if pkg-config --atleast-version=1.90 bash-completion; then\r
101 > +    printf "Yes.\n"\r
102 > +else\r
103 > +    printf "No (will not install bash completion).\n"\r
104 > +    WITH_BASH=0\r
105 > +fi\r
106 > +\r
107 >  if [ -z "${EMACSLISPDIR}" ]; then\r
108 >      if pkg-config --exists emacs; then\r
109 >       EMACSLISPDIR=$(pkg-config emacs --variable sitepkglispdir)\r
110 > -- \r
111 > 1.8.5.2\r
112 >\r
113 > _______________________________________________\r
114 > notmuch mailing list\r
115 > notmuch@notmuchmail.org\r
116 > http://notmuchmail.org/mailman/listinfo/notmuch\r