2a/8aaa1183d0e76f43d6a25a1c276d0ed6dba0ef

   1 Return-Path: <cworth@cworth.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 39A4A431FBD;\r
   6         Fri,  4 Dec 2009 16:12:49 -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 oifTl9k2mbxV; Fri,  4 Dec 2009 16:12:48 -0800 (PST)\r
  11 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 436BF431FAE;\r
  13         Fri,  4 Dec 2009 16:12:48 -0800 (PST)\r
  14 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  15         id EE2122542FB; Fri,  4 Dec 2009 16:12:47 -0800 (PST)\r
  16 From: Carl Worth <cworth@cworth.org>\r
  17 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
  18  notmuch@notmuchmail.org\r
  19 In-Reply-To: <1259452657-27608-1-git-send-email-jrollins@finestructure.net>\r
  20 References: <1259452657-27608-1-git-send-email-jrollins@finestructure.net>\r
  21 Date: Fri, 04 Dec 2009 16:12:47 -0800\r
  22 Message-ID: <87ocmee0sg.fsf@yoom.home.cworth.org>\r
  23 MIME-Version: 1.0\r
  24 Content-Type: multipart/signed; boundary="=-=-=";\r
  25         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  26 Subject: Re: [notmuch] [PATCH 1/3] fix configure script to handle --prefix=\r
  27  and properly create Makefile.config\r
  28 X-BeenThere: notmuch@notmuchmail.org\r
  29 X-Mailman-Version: 2.1.12\r
  30 Precedence: list\r
  31 List-Id: "Use and development of the notmuch mail system."\r
  32         <notmuch.notmuchmail.org>\r
  33 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  34         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  35 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  36 List-Post: <mailto:notmuch@notmuchmail.org>\r
  37 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  38 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  39         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  40 X-List-Received-Date: Sat, 05 Dec 2009 00:12:49 -0000\r
  41 \r
  42 --=-=-=\r
  43 \r
  44 On Sat, 28 Nov 2009 18:57:35 -0500, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  45 > This also removes the Makefile.config from the repository, since it\r
  46 > shouldn't be kept in the repository and should be created by the\r
  47 > configure script.\r
  48 \r
  49 Hi Jamie,\r
  50 \r
  51 Handling --prefix will be a nice addition to our configure script. So,\r
  52 thanks!\r
  53 \r
  54 Your commit message has that flag word of "also" in it, and as it turns\r
  55 out, the removal of Makefile.config from the repository has actually\r
  56 happened already. But that was easy enough to fix.\r
  57 \r
  58 > +# option parsing\r
  59 > +for option; do\r
  60 > +    if [ "${option%=*}" = '--prefix' ] ; then\r
  61 > +     PREFIX="${option#*=}"\r
  62 > +    fi\r
  63 > +done\r
  64 \r
  65 I've gone ahead and committed that now. Then I noticed that we should\r
  66 really use ${option%%=*} to support the case of an option value\r
  67 containing an '=' character. So I fixed that.\r
  68 \r
  69 Then, since I was in the area, I added support to configure for\r
  70 capturing CFLAGS from the environment, I fixed this (and also "make\r
  71 CFLAGS=") to also influence C++ compilation (still can be separately\r
  72 overridden with CXXFLAGS), and I fixed our quiet-compilation mode to\r
  73 actually print the (user-specified) CFLAGS.\r
  74 \r
  75 Finally, I documented things by adding a "configure --help" to document\r
  76 CC, CFLAGS, and --prefix; and by making "make" tell the user about\r
  77 "./configure" and "./configure --help" when make runs configure\r
  78 implicitly.\r
  79 \r
  80 Our configuration system certainly isn't as full-featured yet as a\r
  81 standard autoconf-based configure script, but I'm quite happy with how\r
  82 clean it is for both users and developers.\r
  83 \r
  84 -Carl\r
  85 \r
  86 --=-=-=\r
  87 Content-Type: application/pgp-signature\r
  88 \r
  89 -----BEGIN PGP SIGNATURE-----\r
  90 Version: GnuPG v1.4.10 (GNU/Linux)\r
  91 \r
  92 iD8DBQFLGaV/6JDdNq8qSWgRAoTsAKCa8LqALafG9tGjAGTP4HQat8YHEwCffMjd\r
  93 qwhQlBNHuaC+vk1iNwhe/jM=\r
  94 =gFfw\r
  95 -----END PGP SIGNATURE-----\r
  96 --=-=-=--\r