13/d0c2c2e705a91f41602ab0bf0e9a1e70320f9d

   1 Return-Path: <jani@nikula.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 1BD6B431FC0\r
   6         for <notmuch@notmuchmail.org>; Sat,  4 Jan 2014 04:36:06 -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: -0.699\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.699 tagged_above=-999 required=5\r
  12         tests=[HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7] 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 eRXn7U0Ih5hV for <notmuch@notmuchmail.org>;\r
  16         Sat,  4 Jan 2014 04:35:56 -0800 (PST)\r
  17 Received: from mail-ve0-f174.google.com (mail-ve0-f174.google.com\r
  18         [209.85.128.174]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 09287431FBD\r
  21         for <notmuch@notmuchmail.org>; Sat,  4 Jan 2014 04:35:55 -0800 (PST)\r
  22 Received: by mail-ve0-f174.google.com with SMTP id pa12so8291846veb.19\r
  23         for <notmuch@notmuchmail.org>; Sat, 04 Jan 2014 04:35:55 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:mime-version:in-reply-to:references:date\r
  27         :message-id:subject:from:to:cc:content-type;\r
  28         bh=fDTtF+bLpWXoFX4MzHTuJHvwwgb9fyaPFwCC/fVlLlw=;\r
  29         b=lxPle/eMelZuCSP9/MuueKoRzk0Dhem2BAPNrYGAyKTKVIRYkyD+cmkwD1jUPMLo3p\r
  30         3Snhl6nrOslFpFTyLYHEXXtiYSgcf1hhuVn/F+Ofnl+gg2YpWS0XyKB3Os0romd/KaWP\r
  31         v9CQVN1bTohIQejw8VoruOsRT1KjuQ3w17+TXHcgVTtoucaXNJZ4Cbk/I/rOk075IL5D\r
  32         OFPSi25Ipq3D7qx+SgTkXsDHhnokP5wSgTT68XqKPyCt8WvJr52WnBtJ7ng2KzGHE4AL\r
  33         SaEY5x701z120CtuZo9+7rF3armFSuHlbt1Gm/xlMW3K1oXcN2jcQXKtacxPyHD4ncpk\r
  34         7M4A==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQmueCBDRYJBgizXiUz7LIG8BLNMQwsJ7j5w09dqN2Wf1werAbX6W6OhGQzCgfhYzx8XOrkP\r
  37 MIME-Version: 1.0\r
  38 X-Received: by 10.59.0.193 with SMTP id ba1mr13805413ved.12.1388838954326;\r
  39         Sat, 04 Jan 2014 04:35:54 -0800 (PST)\r
  40 Received: by 10.58.143.6 with HTTP; Sat, 4 Jan 2014 04:35:54 -0800 (PST)\r
  41 Received: by 10.58.143.6 with HTTP; Sat, 4 Jan 2014 04:35:54 -0800 (PST)\r
  42 In-Reply-To: <20140103214735.GG27614@danbala.tuwien.ac.at>\r
  43 References: <20140103214735.GG27614@danbala.tuwien.ac.at>\r
  44 Date: Sat, 4 Jan 2014 14:35:54 +0200\r
  45 Message-ID:\r
  46  <CAB+hUn-8eyY07y3j1N3JJLQ3qWVEVPoGWHcUVkovg34xH0ZOwQ@mail.gmail.com>\r
  47 Subject: Re: notmuch-0.16: realpath() compatibility issue;\r
  48         clang visibility problem\r
  49 From: Jani Nikula <jani@nikula.org>\r
  50 To: Thomas Klausner <tk@giga.or.at>\r
  51 Content-Type: multipart/alternative; boundary=047d7bdc855686369c04ef24443b\r
  52 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  53 X-BeenThere: notmuch@notmuchmail.org\r
  54 X-Mailman-Version: 2.1.13\r
  55 Precedence: list\r
  56 List-Id: "Use and development of the notmuch mail system."\r
  57         <notmuch.notmuchmail.org>\r
  58 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  59         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  60 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  61 List-Post: <mailto:notmuch@notmuchmail.org>\r
  62 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  63 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  65 X-List-Received-Date: Sat, 04 Jan 2014 12:36:06 -0000\r
  66 \r
  67 --047d7bdc855686369c04ef24443b\r
  68 Content-Type: text/plain; charset=UTF-8\r
  69 \r
  70 For the visibility issue please upgrade Notmuch.\r
  71 \r
  72 BR,\r
  73 Jani.\r
  74 \r
  75 On Jan 4, 2014 2:26 PM, "Thomas Klausner" <tk@giga.or.at> wrote:\r
  76 >\r
  77 > Hi!\r
  78 >\r
  79 > I'm currently starting to try out notmuch-0.16 on NetBSD. It went off\r
  80 > to a rocky start, since it segfaulted in the initial config setup.\r
  81 >\r
  82 > Debugging it I found that notmuch uses a glibc extension to realpath,\r
  83 > allowing NULL as second argument.\r
  84 >\r
  85 > I've converted it to use a prepared buffer instead; attached is a\r
  86 > possible patch that makes notmuch complete its setup phase for me, and\r
  87 > adds inclusion of the header files suggested by the realpath man page\r
  88 > on NetBSD. Please address this issue in some way in the next release.\r
  89 >\r
  90 > Additionally, when compiling with clang, there are issues with the\r
  91 > visibility. The symptoms are:\r
  92 >\r
  93 > In file included from lib/database.cc:21:\r
  94 > In file included from ./lib/database-private.h:33:\r
  95 > ./lib/notmuch-private.h:479:8: error: visibility does not match previous\r
  96 declaration\r
  97 > array subscriptstruct visible _notmuch_string_list {\r
  98 >        ^\r
  99 > ./lib/notmuch-private.h:67:33: note: expanded from macro 'visible'\r
 100 >  # define visible __attribute__((visibility("default")))\r
 101 >                                 ^\r
 102 > ./lib/notmuch-private.h:52:13: note: previous attribute is here\r
 103 > #pragma GCC visibility push(hidden)\r
 104 >             ^\r
 105 >\r
 106 > In file included from lib/parse-time-vrp.cc:23:\r
 107 > In file included from ./lib/database-private.h:33:\r
 108 > ./lib/notmuch-private.h:479:8: error: visibility does not match previous\r
 109 declaration\r
 110 > struct visible _notmuch_string_list {\r
 111 >        ^\r
 112 > ./lib/notmuch-private.h:67:33: note: expanded from macro 'visible'\r
 113 > # define visible __attribute__((visibility("default")))\r
 114 >                                 ^\r
 115 > ./lib/notmuch-private.h:52:13: note: previous attribute is here\r
 116 > #pragma GCC visibility push(hidden)\r
 117 >             ^\r
 118 > 1 warning generated.\r
 119 > In file included from lib/directory.cc:21:\r
 120 > ./lib/notmuch-private.h:479:8: error: visibility does not match previous\r
 121 declaration\r
 122 > struct visible _notmuch_string_list {\r
 123 >        ^\r
 124 > ./lib/notmuch-private.h:67:33: note: expanded from macro 'visible'\r
 125 > # define visible __attribute__((visibility("default")))\r
 126 >                                 ^\r
 127 > ./lib/notmuch-private.h:52:13: note: previous attribute is here\r
 128 > #pragma GCC visibility push(hidden)\r
 129 >             ^\r
 130 >\r
 131 > and so on. I guess it is because the visibility differs between c and\r
 132 > c++. I've disabled visibility locally, see second attached patch, but\r
 133 > of course that's not a solution, just a workaround. Suggestions\r
 134 > welcome.\r
 135 >\r
 136 > Thanks,\r
 137 >  Thomas\r
 138 >\r
 139 > _______________________________________________\r
 140 > notmuch mailing list\r
 141 > notmuch@notmuchmail.org\r
 142 > http://notmuchmail.org/mailman/listinfo/notmuch\r
 143 >\r
 144 \r
 145 --047d7bdc855686369c04ef24443b\r
 146 Content-Type: text/html; charset=UTF-8\r
 147 Content-Transfer-Encoding: quoted-printable\r
 148 \r
 149 <p dir=3D"ltr">For the visibility issue please upgrade Notmuch.</p>\r
 150 <p dir=3D"ltr">BR,<br>\r
 151 Jani.</p>\r
 152 <p dir=3D"ltr">On Jan 4, 2014 2:26 PM, &quot;Thomas Klausner&quot; &lt;<a h=\r
 153 ref=3D"mailto:tk@giga.or.at">tk@giga.or.at</a>&gt; wrote:<br>\r
 154 &gt;<br>\r
 155 &gt; Hi!<br>\r
 156 &gt;<br>\r
 157 &gt; I&#39;m currently starting to try out notmuch-0.16 on NetBSD. It went =\r
 158 off<br>\r
 159 &gt; to a rocky start, since it segfaulted in the initial config setup.<br>\r
 160 &gt;<br>\r
 161 &gt; Debugging it I found that notmuch uses a glibc extension to realpath,<=\r
 162 br>\r
 163 &gt; allowing NULL as second argument.<br>\r
 164 &gt;<br>\r
 165 &gt; I&#39;ve converted it to use a prepared buffer instead; attached is a<=\r
 166 br>\r
 167 &gt; possible patch that makes notmuch complete its setup phase for me, and=\r
 168 <br>\r
 169 &gt; adds inclusion of the header files suggested by the realpath man page<=\r
 170 br>\r
 171 &gt; on NetBSD. Please address this issue in some way in the next release.<=\r
 172 br>\r
 173 &gt;<br>\r
 174 &gt; Additionally, when compiling with clang, there are issues with the<br>\r
 175 &gt; visibility. The symptoms are:<br>\r
 176 &gt;<br>\r
 177 &gt; In file included from lib/database.cc:21:<br>\r
 178 &gt; In file included from ./lib/database-private.h:33:<br>\r
 179 &gt; ./lib/notmuch-private.h:479:8: error: visibility does not match previo=\r
 180 us declaration<br>\r
 181 &gt; array subscriptstruct visible _notmuch_string_list {<br>\r
 182 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0^<br>\r
 183 &gt; ./lib/notmuch-private.h:67:33: note: expanded from macro &#39;visible&=\r
 184 #39;<br>\r
 185 &gt; =C2=A0# define visible __attribute__((visibility(&quot;default&quot;))=\r
 186 )<br>\r
 187 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =\r
 188 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 189 &gt; ./lib/notmuch-private.h:52:13: note: previous attribute is here<br>\r
 190 &gt; #pragma GCC visibility push(hidden)<br>\r
 191 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 192 &gt;<br>\r
 193 &gt; In file included from lib/parse-time-vrp.cc:23:<br>\r
 194 &gt; In file included from ./lib/database-private.h:33:<br>\r
 195 &gt; ./lib/notmuch-private.h:479:8: error: visibility does not match previo=\r
 196 us declaration<br>\r
 197 &gt; struct visible _notmuch_string_list {<br>\r
 198 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0^<br>\r
 199 &gt; ./lib/notmuch-private.h:67:33: note: expanded from macro &#39;visible&=\r
 200 #39;<br>\r
 201 &gt; # define visible __attribute__((visibility(&quot;default&quot;)))<br>\r
 202 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =\r
 203 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 204 &gt; ./lib/notmuch-private.h:52:13: note: previous attribute is here<br>\r
 205 &gt; #pragma GCC visibility push(hidden)<br>\r
 206 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 207 &gt; 1 warning generated.<br>\r
 208 &gt; In file included from lib/directory.cc:21:<br>\r
 209 &gt; ./lib/notmuch-private.h:479:8: error: visibility does not match previo=\r
 210 us declaration<br>\r
 211 &gt; struct visible _notmuch_string_list {<br>\r
 212 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0^<br>\r
 213 &gt; ./lib/notmuch-private.h:67:33: note: expanded from macro &#39;visible&=\r
 214 #39;<br>\r
 215 &gt; # define visible __attribute__((visibility(&quot;default&quot;)))<br>\r
 216 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =\r
 217 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 218 &gt; ./lib/notmuch-private.h:52:13: note: previous attribute is here<br>\r
 219 &gt; #pragma GCC visibility push(hidden)<br>\r
 220 &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 221 &gt;<br>\r
 222 &gt; and so on. I guess it is because the visibility differs between c and<=\r
 223 br>\r
 224 &gt; c++. I&#39;ve disabled visibility locally, see second attached patch, =\r
 225 but<br>\r
 226 &gt; of course that&#39;s not a solution, just a workaround. Suggestions<br=\r
 227 >\r
 228 &gt; welcome.<br>\r
 229 &gt;<br>\r
 230 &gt; Thanks,<br>\r
 231 &gt; =C2=A0Thomas<br>\r
 232 &gt;<br>\r
 233 &gt; _______________________________________________<br>\r
 234 &gt; notmuch mailing list<br>\r
 235 &gt; <a href=3D"mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a>=\r
 236 <br>\r
 237 &gt; <a href=3D"http://notmuchmail.org/mailman/listinfo/notmuch">http://not=\r
 238 muchmail.org/mailman/listinfo/notmuch</a><br>\r
 239 &gt;<br>\r
 240 </p>\r
 241 \r
 242 --047d7bdc855686369c04ef24443b--\r