30/0cf8901064ce5db56d2687ff113cc25776b61c

   1 Return-Path: <ingmar@exherbo.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 2AFA7431FBF\r
   6         for <notmuch@notmuchmail.org>; Mon, 23 Nov 2009 07:41:25 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id dw8FEnv8tqmq for <notmuch@notmuchmail.org>;\r
  11         Mon, 23 Nov 2009 07:41:24 -0800 (PST)\r
  12 Received: from bach.exherbo.org (bach.exherbo.org [78.47.197.147])\r
  13         by olra.theworths.org (Postfix) with ESMTP id 7DAB3431FAE\r
  14         for <notmuch@notmuchmail.org>; Mon, 23 Nov 2009 07:41:24 -0800 (PST)\r
  15 Received: from [83.101.72.69] (helo=localhost)\r
  16         by bach.exherbo.org with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.69)\r
  17         (envelope-from <ingmar@exherbo.org>) id 1NCb2Z-0006J3-P1\r
  18         for notmuch@notmuchmail.org; Mon, 23 Nov 2009 15:41:23 +0000\r
  19 Content-Type: text/plain; charset=utf8\r
  20 From: Ingmar Vanhassel <ingmar@exherbo.org>\r
  21 To: notmuch <notmuch@notmuchmail.org>\r
  22 In-reply-to: <46263c600911230731i529f2db5rec11bed02f84ef13@mail.gmail.com>\r
  23 References: <46263c600911230731i529f2db5rec11bed02f84ef13@mail.gmail.com>\r
  24 Date: Mon, 23 Nov 2009 16:41:23 +0100\r
  25 Message-Id: <1258990831-sup-2789@cannonball>\r
  26 User-Agent: Sup/git\r
  27 Content-Transfer-Encoding: 8bit\r
  28 Subject: Re: [notmuch] [patch] Trivial fix for non-root install\r
  29 X-BeenThere: notmuch@notmuchmail.org\r
  30 X-Mailman-Version: 2.1.12\r
  31 Precedence: list\r
  32 List-Id: "Use and development of the notmuch mail system."\r
  33         <notmuch.notmuchmail.org>\r
  34 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  35         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  36 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  37 List-Post: <mailto:notmuch@notmuchmail.org>\r
  38 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  39 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  40         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  41 X-List-Received-Date: Mon, 23 Nov 2009 15:41:25 -0000\r
  42 \r
  43 Excerpts from Brett Viren's message of Mon Nov 23 16:31:47 +0100 2009:\r
  44 > Installing as a normal user fails because the bash completion config\r
  45 > files try to install into /etc.  This trivial patch fixes this.\r
  46 \r
  47 NAK\r
  48 \r
  49 Your patch breaks the more common case of installing as root user. It\r
  50 now installs into /usr/etc where bash completions most likely won't be\r
  51 found.\r
  52 \r
  53 > diff --git a/Makefile.config b/Makefile.config\r
  54 > index ddc7436..c04d57a 100644\r
  55 > --- a/Makefile.config\r
  56 > +++ b/Makefile.config\r
  57 > @@ -1,3 +1,2 @@\r
  58 > -prefix = /usr/local\r
  59 > -bash_completion_dir = /etc/bash_completion.d\r
  60 > +bash_completion_dir = $(prefix)/etc/bash_completion.d\r
  61 >  CFLAGS += -DHAVE_VALGRIND\r
  62 > \r
  63 > \r
  64 > -Brett.\r
  65 -- \r
  66 Exherbo KDE, X.org maintainer\r