[PATCH 6/8] CLI: refactor dumping of tags.
[notmuch-archives.git] / 12 / 3f39a135ae6be945f12ae25854e5c2cf90b3f1
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 84A3C431FB6\r
6         for <notmuch@notmuchmail.org>; Fri,  1 Jun 2012 11:17:11 -0700 (PDT)\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
8 X-Spam-Flag: NO\r
9 X-Spam-Score: -2.29\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 kTEcpI3dKWfh for <notmuch@notmuchmail.org>;\r
16         Fri,  1 Jun 2012 11:17:11 -0700 (PDT)\r
17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
18         [131.215.239.19])\r
19         by olra.theworths.org (Postfix) with ESMTP id EE2E2431FAF\r
20         for <notmuch@notmuchmail.org>; Fri,  1 Jun 2012 11:17:10 -0700 (PDT)\r
21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
22         by earth-doxen-postvirus (Postfix) with ESMTP id A92D266E011D\r
23         for <notmuch@notmuchmail.org>; Fri,  1 Jun 2012 11:17:10 -0700 (PDT)\r
24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
25 Received: from finestructure.net (DHCP-123-229.caltech.edu [131.215.123.229])\r
26         (Authenticated sender: jrollins)\r
27         by earth-doxen-submit (Postfix) with ESMTP id DA56766E0174\r
28         for <notmuch@notmuchmail.org>; Fri,  1 Jun 2012 11:17:08 -0700 (PDT)\r
29 Received: by finestructure.net (Postfix, from userid 1000)\r
30         id C4D7014F; Fri,  1 Jun 2012 11:17:07 -0700 (PDT)\r
31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
32 To: Notmuch Mail <notmuch@notmuchmail.org>\r
33 Subject: bug in configure script\r
34 User-Agent: Notmuch/0.13+24~g16915cd (http://notmuchmail.org) Emacs/23.4.1\r
35         (x86_64-pc-linux-gnu)\r
36 Date: Fri, 01 Jun 2012 11:17:05 -0700\r
37 Message-ID: <87pq9ivqvy.fsf@servo.finestructure.net>\r
38 MIME-Version: 1.0\r
39 Content-Type: multipart/signed; boundary="=-=-=";\r
40         micalg=pgp-sha256; protocol="application/pgp-signature"\r
41 X-BeenThere: notmuch@notmuchmail.org\r
42 X-Mailman-Version: 2.1.13\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: Fri, 01 Jun 2012 18:17:11 -0000\r
54 \r
55 --=-=-=\r
56 \r
57 There's a bug in the configure script that is causing auto-reruns of\r
58 ./configure to not inherit original command line options if there was\r
59 more than one.  For instance, if I run:\r
60 \r
61 ./configure --with-gmime-version=2.4 --prefix=/home/jrollins/opt/notmuch\r
62 \r
63 Then in Makefile.config I get:\r
64 \r
65 configure_options = --with-gmime-version=2.4--prefix=/home/jrollins/opt/notmuch\r
66 \r
67 This means that auto-reruns of configure will not get the proper\r
68 options.\r
69 \r
70 I tracked this down to an issue with IFS and /bin/sh.  The first line of\r
71 ./configure is:\r
72 \r
73 readonly DEFAULT_IFS=$IFS\r
74 \r
75 DEFAULT_IFS is then used to reset IFS after it is modified within the\r
76 script.  The problem is that /bin/sh is setting DEFAULT_IFS to be NULL\r
77 (i.e. ''), which leads to no separation between variables when "$@" is\r
78 expanded (which is itself problematic since I don't think "@" should use\r
79 the IFS when expanded).  So this might be a bug in dash.  I don't know.\r
80 In any event we need to fix this somehow.  I see two obvious solutions:\r
81 \r
82 * use /bin/bash.  This a one line diff that fixes the problem\r
83   immediately.\r
84 \r
85 * replace:\r
86 \r
87     IFS=$DEFAULT_IFS\r
88 \r
89   with:\r
90 \r
91     unset IFS\r
92 \r
93   Unsetting IFS also resets IFS to the default, without going through\r
94   this intermediate step.\r
95 \r
96 I'm ok with either solution, but I'll wait for some feedback since I\r
97 imagine someone will have an opinion.\r
98 \r
99 jamie.\r
100 \r
101 --=-=-=\r
102 Content-Type: application/pgp-signature\r
103 \r
104 -----BEGIN PGP SIGNATURE-----\r
105 Version: GnuPG v1.4.12 (GNU/Linux)\r
106 \r
107 iQIcBAEBCAAGBQJPyQchAAoJEO00zqvie6q8VlEP/AkYhn/YzNpESily/3dO1lfO\r
108 +XLDopIpGAI8Uf49T5MfPlWDRo/HGT1V85O+uZJrCT4Y4V6jxxbGxiejuZUUVQsA\r
109 gwcW0V34M0BN9kyDbJhjA/VUQcIt0gbG4NYo71hwS6fFMItjDIOZXc46WirLiueS\r
110 6SWcThQExzc6TXjtHkdhbaMhF0gTBslh+rrBO94SXJSyu7xlRTBUUG2ZBMlODEhw\r
111 xdzWiGiQAeOpLmYmMGWKZrVgBJxMorF1TOgvoBGiXoK/fJVoyTGjAPn0B9E5a3V0\r
112 sVdJAOgJHLnvsKL9vgJA1KdWXw80JWYfqI8FLXCrdq2TQArT/lBtpJ+B9d7DfTES\r
113 Uk+uo5pu+X/DpBShBwYPFtLuILoVyuPzvdcDXoDT4PRuaIIPn5MyjwKA7bDIW4lv\r
114 rjJ5ZZWrL3OWh5uXXN0i3wxc9XNgOFtPhRXhTiCTAmn4dU+FQC4I9/9oc0m+pRQm\r
115 1K9OH9U0sIMinXksEBxtrF9ZHFUE6SEtejnb4j1jL/VrxRaKIvkbDtvBrue053ve\r
116 R9XdcN5tpkqzDrKB6sM6ZMt1tQqPawigOTr5Ed1lxMgRl268ErncdX6Xe5fcnxB3\r
117 03pBYYwSPgMt+7JpN3U79Lqi/4mSjBF8zKxkVZzVwKauooZDMdWl48LOTdSe0q56\r
118 PgCyxPuqf25rK05l2PWN\r
119 =1jlW\r
120 -----END PGP SIGNATURE-----\r
121 --=-=-=--\r