eb/ac88ba1e446aea54baae86e70796be71535ae1

   1 Return-Path: <jrollins@finestructure.net>\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 89DCC431FAE\r
   6         for <notmuch@notmuchmail.org>; Fri,  4 Dec 2009 16:20:31 -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 F1fw0NbsB-RS for <notmuch@notmuchmail.org>;\r
  11         Fri,  4 Dec 2009 16:20:29 -0800 (PST)\r
  12 Received: from tarap.cc.columbia.edu (tarap.cc.columbia.edu [128.59.29.7])\r
  13         by olra.theworths.org (Postfix) with ESMTP id 5527A431FBC\r
  14         for <notmuch@notmuchmail.org>; Fri,  4 Dec 2009 16:20:29 -0800 (PST)\r
  15 Received: from servo.finestructure.net (cpe-72-227-128-66.nyc.res.rr.com\r
  16         [72.227.128.66])\r
  17         (user=jgr2110 author=jrollins@finestructure.net mech=PLAIN bits=0)\r
  18         by tarap.cc.columbia.edu (8.14.3/8.14.3) with ESMTP id nB50KR54010715\r
  19         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  20         Fri, 4 Dec 2009 19:20:28 -0500 (EST)\r
  21 Received: from jrollins by servo.finestructure.net with local (Exim 4.69)\r
  22         (envelope-from <jrollins@finestructure.net>)\r
  23         id 1NGiOI-0008BL-Ul; Fri, 04 Dec 2009 19:20:50 -0500\r
  24 Date: Fri, 4 Dec 2009 19:20:50 -0500\r
  25 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  26 To: Carl Worth <cworth@cworth.org>\r
  27 Message-ID: <20091205002050.GA26479@finestructure.net>\r
  28 References: <1259452657-27608-1-git-send-email-jrollins@finestructure.net>\r
  29         <87ocmee0sg.fsf@yoom.home.cworth.org>\r
  30 MIME-Version: 1.0\r
  31 Content-Type: multipart/signed; micalg=pgp-sha256;\r
  32         protocol="application/pgp-signature"; boundary="qDbXVdCdHGoSgWSk"\r
  33 Content-Disposition: inline\r
  34 In-Reply-To: <87ocmee0sg.fsf@yoom.home.cworth.org>\r
  35 User-Agent: Mutt/1.5.20 (2009-06-14)\r
  36 X-No-Spam-Score: Local\r
  37 X-Scanned-By: MIMEDefang 2.68 on 128.59.29.7\r
  38 Cc: notmuch@notmuchmail.org\r
  39 Subject: Re: [notmuch] [PATCH 1/3] fix configure script to handle --prefix=\r
  40  and properly create Makefile.config\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.12\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Sat, 05 Dec 2009 00:20:31 -0000\r
  54 \r
  55 \r
  56 --qDbXVdCdHGoSgWSk\r
  57 Content-Type: text/plain; charset=us-ascii\r
  58 Content-Disposition: inline\r
  59 Content-Transfer-Encoding: quoted-printable\r
  60 \r
  61 On Fri, Dec 04, 2009 at 04:12:47PM -0800, Carl Worth wrote:\r
  62 > Handling --prefix will be a nice addition to our configure script. So,\r
  63 > thanks!\r
  64 \r
  65 Yeah, it's definitely needed for the Debian packaging as well.\r
  66 \r
  67 > Your commit message has that flag word of "also" in it, and as it turns\r
  68 > out, the removal of Makefile.config from the repository has actually\r
  69 > happened already. But that was easy enough to fix.\r
  70 \r
  71 I was thinking that the removal of the Makefile.config from the repo\r
  72 went together with the new auto-generation of that file from configure\r
  73 script.  Do you think they still should have been separate patches?\r
  74 \r
  75 > > +# option parsing\r
  76 > > +for option; do\r
  77 > > +    if [ "${option%=3D*}" =3D '--prefix' ] ; then\r
  78 > > +   PREFIX=3D"${option#*=3D}"\r
  79 > > +    fi\r
  80 > > +done\r
  81 >=20\r
  82 > I've gone ahead and committed that now. Then I noticed that we should\r
  83 > really use ${option%%=3D*} to support the case of an option value\r
  84 > containing an '=3D' character. So I fixed that.\r
  85 \r
  86 Ah, good catch.  Sorry about that. =20\r
  87 \r
  88 > Our configuration system certainly isn't as full-featured yet as a\r
  89 > standard autoconf-based configure script, but I'm quite happy with how\r
  90 > clean it is for both users and developers.\r
  91 \r
  92 Autoconf terrifies me, so I agree I'm quite happy with the simple\r
  93 configure script we have right now.  If it gets the job done without\r
  94 having to deal with autoconf then that's great in my book.\r
  95 \r
  96 jamie.\r
  97 \r
  98 --qDbXVdCdHGoSgWSk\r
  99 Content-Type: application/pgp-signature; name="signature.asc"\r
 100 Content-Description: Digital signature\r
 101 Content-Disposition: attachment\r
 102 \r
 103 -----BEGIN PGP SIGNATURE-----\r
 104 Version: GnuPG v1.4.10 (GNU/Linux)\r
 105 \r
 106 iQIcBAEBCAAGBQJLGadeAAoJEO00zqvie6q87KwP/RNP7FehU5TI7K3tQdFlF35B\r
 107 7DS4sNp7N+O0fnkE2V4M+b9WCXwZkQ9i9d6BmefLXM/GUkgdKP+X90GygzW7POvv\r
 108 /iHwk0XJuU1drq5uGk4Dt/rnlGLQI988lLh4JA+Fv6VOgxDlXGeYA/2ATGDLJgLx\r
 109 wq0UuWqApYongpmnJ8RwDeD9QlJK8TBxsfBjGZfzj2boo8r4kJ1iU3NHID/MjetV\r
 110 /WF4LM7zAfU7wa5wQOu54rOGOg6WULB36wjpcy2CvXv2kWemN1zQ2ImLx935q/BD\r
 111 KFLEauadZTcKuy2ep0AY8tKqUeytAvilxhunbVyaY0pg+561gnygzqy/Wh31OL74\r
 112 SYkKN3nd+ZUp6eocOaEVooOJp52X+xH1wfMo1jMRTMB26zhAUt35uULbRt+fzdsA\r
 113 URprLhRgQh4spRP9tqfmY3Jbgef48OjQV5r228kkeMpNu3tRH81ngyunU7YnL32u\r
 114 RoE4ZK1EQQ/q7mqXTvZNl7TZnETbNOi+VEe1JvglQHFM6guitXVfgCugw+HTtA5n\r
 115 0mcxWRKhbaeGkd3dwJQNb8w++UzFW8fA4hao6M14jZnqzKklLfRtTTVRzHFZXcWA\r
 116 Zx7uoUoQido2dw7+locnXW/8X43ORHlFcD8+Kze4KuE3mA1X3xSExz4ngx2lLZka\r
 117 cOk5+oJJ8VgWhZkd18hi\r
 118 =WZqM\r
 119 -----END PGP SIGNATURE-----\r
 120 \r
 121 --qDbXVdCdHGoSgWSk--\r