35/3d53a47d66c99175a8496b3ec93669385c982a

   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 494E1431FC0\r
   6         for <notmuch@notmuchmail.org>; Sat,  4 Jan 2014 04:46:47 -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 RNHtIL1Bi828 for <notmuch@notmuchmail.org>;\r
  16         Sat,  4 Jan 2014 04:46:39 -0800 (PST)\r
  17 Received: from mail-pa0-f51.google.com (mail-pa0-f51.google.com\r
  18         [209.85.220.51]) (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 672FE431FBD\r
  21         for <notmuch@notmuchmail.org>; Sat,  4 Jan 2014 04:46:39 -0800 (PST)\r
  22 Received: by mail-pa0-f51.google.com with SMTP id fa1so16878769pad.10\r
  23         for <notmuch@notmuchmail.org>; Sat, 04 Jan 2014 04:46:37 -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=JtSa5qOKFXCZ/rkmDJn8clo+cswnzCqKUs59YmhGYp4=;\r
  29         b=KthG+TaUgaROCkXaGKNjO6EXnyeherW9uqoSmXF0nQfkgMPvAHHttZyjcIp4bN+qFe\r
  30         IovriRb/BD5G9dc7VYHF8iO7dPafYTvmNpvahqv/yCC3R794fxFZBgmnlSAQM5ofUtBY\r
  31         aqnbZAVOzOyepnuB2vbdgGLcVsY64ZjOeG+ZJDBDWzZi5S07DtAWTy6bUBxuOqrqvmo+\r
  32         WXSwuKF6iJeWKrZGQYD2W9PdlaArCpXxmACHH1DjjZpMlaC7qRMfw0yj9VyJrd/+wvSw\r
  33         Gq/N46KXnqxXOKdnz3f8uRK3sHQwnHRqv4bI+obTrcUyJbFV5GC6tG2GzZ2yxFM6l1cO\r
  34         DdAg==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQlWHuEZZM7CymHHw0MUVGpglcot2ZWhLJ03Qfyn7MfCBrp20Z7+fFiTeLpRdD0umBoMozYM\r
  37 MIME-Version: 1.0\r
  38 X-Received: by 10.68.99.226 with SMTP id et2mr56822556pbb.8.1388839597368;\r
  39         Sat, 04 Jan 2014 04:46:37 -0800 (PST)\r
  40 Received: by 10.70.42.228 with HTTP; Sat, 4 Jan 2014 04:46:37 -0800 (PST)\r
  41 Received: by 10.70.42.228 with HTTP; Sat, 4 Jan 2014 04:46:37 -0800 (PST)\r
  42 In-Reply-To:\r
  43  <CAB+hUn-8eyY07y3j1N3JJLQ3qWVEVPoGWHcUVkovg34xH0ZOwQ@mail.gmail.com>\r
  44 References: <20140103214735.GG27614@danbala.tuwien.ac.at>\r
  45         <CAB+hUn-8eyY07y3j1N3JJLQ3qWVEVPoGWHcUVkovg34xH0ZOwQ@mail.gmail.com>\r
  46 Date: Sat, 4 Jan 2014 14:46:37 +0200\r
  47 Message-ID:\r
  48  <CAB+hUn-ov+2roT-tVTcjuPbckiTkRPtUxazfmY1KFeeVoMQJCA@mail.gmail.com>\r
  49 Subject: Re: notmuch-0.16: realpath() compatibility issue;\r
  50         clang visibility problem\r
  51 From: Jani Nikula <jani@nikula.org>\r
  52 To: Thomas Klausner <tk@giga.or.at>\r
  53 Content-Type: multipart/alternative; boundary=047d7b6dd09cda0cb204ef246ae9\r
  54 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Sat, 04 Jan 2014 12:46:47 -0000\r
  68 \r
  69 --047d7b6dd09cda0cb204ef246ae9\r
  70 Content-Type: text/plain; charset=UTF-8\r
  71 \r
  72 I guess we should look at realpath() compatibility, but in fairness passing\r
  73 NULL for the second parameter is according to POSIX.1-2008, not glibc\r
  74 extension.\r
  75 \r
  76 On Jan 4, 2014 2:35 PM, "Jani Nikula" <jani@nikula.org> wrote:\r
  77 >\r
  78 > For the visibility issue please upgrade Notmuch.\r
  79 >\r
  80 > BR,\r
  81 > Jani.\r
  82 >\r
  83 > On Jan 4, 2014 2:26 PM, "Thomas Klausner" <tk@giga.or.at> wrote:\r
  84 > >\r
  85 > > Hi!\r
  86 > >\r
  87 > > I'm currently starting to try out notmuch-0.16 on NetBSD. It went off\r
  88 > > to a rocky start, since it segfaulted in the initial config setup.\r
  89 > >\r
  90 > > Debugging it I found that notmuch uses a glibc extension to realpath,\r
  91 > > allowing NULL as second argument.\r
  92 > >\r
  93 > > I've converted it to use a prepared buffer instead; attached is a\r
  94 > > possible patch that makes notmuch complete its setup phase for me, and\r
  95 > > adds inclusion of the header files suggested by the realpath man page\r
  96 > > on NetBSD. Please address this issue in some way in the next release.\r
  97 > >\r
  98 > > Additionally, when compiling with clang, there are issues with the\r
  99 > > visibility. The symptoms are:\r
 100 > >\r
 101 > > In file included from lib/database.cc:21:\r
 102 > > In file included from ./lib/database-private.h:33:\r
 103 > > ./lib/notmuch-private.h:479:8: error: visibility does not match\r
 104 previous declaration\r
 105 > > array subscriptstruct visible _notmuch_string_list {\r
 106 > >        ^\r
 107 > > ./lib/notmuch-private.h:67:33: note: expanded from macro 'visible'\r
 108 > >  # define visible __attribute__((visibility("default")))\r
 109 > >                                 ^\r
 110 > > ./lib/notmuch-private.h:52:13: note: previous attribute is here\r
 111 > > #pragma GCC visibility push(hidden)\r
 112 > >             ^\r
 113 > >\r
 114 > > In file included from lib/parse-time-vrp.cc:23:\r
 115 > > In file included from ./lib/database-private.h:33:\r
 116 > > ./lib/notmuch-private.h:479:8: error: visibility does not match\r
 117 previous declaration\r
 118 > > struct visible _notmuch_string_list {\r
 119 > >        ^\r
 120 > > ./lib/notmuch-private.h:67:33: note: expanded from macro 'visible'\r
 121 > > # define visible __attribute__((visibility("default")))\r
 122 > >                                 ^\r
 123 > > ./lib/notmuch-private.h:52:13: note: previous attribute is here\r
 124 > > #pragma GCC visibility push(hidden)\r
 125 > >             ^\r
 126 > > 1 warning generated.\r
 127 > > In file included from lib/directory.cc:21:\r
 128 > > ./lib/notmuch-private.h:479:8: error: visibility does not match\r
 129 previous declaration\r
 130 > > struct visible _notmuch_string_list {\r
 131 > >        ^\r
 132 > > ./lib/notmuch-private.h:67:33: note: expanded from macro 'visible'\r
 133 > > # define visible __attribute__((visibility("default")))\r
 134 > >                                 ^\r
 135 > > ./lib/notmuch-private.h:52:13: note: previous attribute is here\r
 136 > > #pragma GCC visibility push(hidden)\r
 137 > >             ^\r
 138 > >\r
 139 > > and so on. I guess it is because the visibility differs between c and\r
 140 > > c++. I've disabled visibility locally, see second attached patch, but\r
 141 > > of course that's not a solution, just a workaround. Suggestions\r
 142 > > welcome.\r
 143 > >\r
 144 > > Thanks,\r
 145 > >  Thomas\r
 146 > >\r
 147 > > _______________________________________________\r
 148 > > notmuch mailing list\r
 149 > > notmuch@notmuchmail.org\r
 150 > > http://notmuchmail.org/mailman/listinfo/notmuch\r
 151 > >\r
 152 \r
 153 --047d7b6dd09cda0cb204ef246ae9\r
 154 Content-Type: text/html; charset=UTF-8\r
 155 Content-Transfer-Encoding: quoted-printable\r
 156 \r
 157 <p dir=3D"ltr">I guess we should look at realpath() compatibility, but in f=\r
 158 airness passing NULL for the second parameter is according to POSIX.1-2008,=\r
 159  not glibc extension.<br></p>\r
 160 <p dir=3D"ltr">On Jan 4, 2014 2:35 PM, &quot;Jani Nikula&quot; &lt;<a href=\r
 161 =3D"mailto:jani@nikula.org">jani@nikula.org</a>&gt; wrote:<br>\r
 162 &gt;<br>\r
 163 &gt; For the visibility issue please upgrade Notmuch.<br>\r
 164 &gt;<br>\r
 165 &gt; BR,<br>\r
 166 &gt; Jani.<br>\r
 167 &gt;<br>\r
 168 &gt; On Jan 4, 2014 2:26 PM, &quot;Thomas Klausner&quot; &lt;<a href=3D"mai=\r
 169 lto:tk@giga.or.at">tk@giga.or.at</a>&gt; wrote:<br>\r
 170 &gt; &gt;<br>\r
 171 &gt; &gt; Hi!<br>\r
 172 &gt; &gt;<br>\r
 173 &gt; &gt; I&#39;m currently starting to try out notmuch-0.16 on NetBSD. It =\r
 174 went off<br>\r
 175 &gt; &gt; to a rocky start, since it segfaulted in the initial config setup=\r
 176 .<br>\r
 177 &gt; &gt;<br>\r
 178 &gt; &gt; Debugging it I found that notmuch uses a glibc extension to realp=\r
 179 ath,<br>\r
 180 &gt; &gt; allowing NULL as second argument.<br>\r
 181 &gt; &gt;<br>\r
 182 &gt; &gt; I&#39;ve converted it to use a prepared buffer instead; attached =\r
 183 is a<br>\r
 184 &gt; &gt; possible patch that makes notmuch complete its setup phase for me=\r
 185 , and<br>\r
 186 &gt; &gt; adds inclusion of the header files suggested by the realpath man =\r
 187 page<br>\r
 188 &gt; &gt; on NetBSD. Please address this issue in some way in the next rele=\r
 189 ase.<br>\r
 190 &gt; &gt;<br>\r
 191 &gt; &gt; Additionally, when compiling with clang, there are issues with th=\r
 192 e<br>\r
 193 &gt; &gt; visibility. The symptoms are:<br>\r
 194 &gt; &gt;<br>\r
 195 &gt; &gt; In file included from lib/database.cc:21:<br>\r
 196 &gt; &gt; In file included from ./lib/database-private.h:33:<br>\r
 197 &gt; &gt; ./lib/notmuch-private.h:479:8: error: visibility does not match p=\r
 198 revious declaration<br>\r
 199 &gt; &gt; array subscriptstruct visible _notmuch_string_list {<br>\r
 200 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0^<br>\r
 201 &gt; &gt; ./lib/notmuch-private.h:67:33: note: expanded from macro &#39;vis=\r
 202 ible&#39;<br>\r
 203 &gt; &gt; =C2=A0# define visible __attribute__((visibility(&quot;default&qu=\r
 204 ot;)))<br>\r
 205 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =\r
 206 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 207 &gt; &gt; ./lib/notmuch-private.h:52:13: note: previous attribute is here<b=\r
 208 r>\r
 209 &gt; &gt; #pragma GCC visibility push(hidden)<br>\r
 210 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 211 &gt; &gt;<br>\r
 212 &gt; &gt; In file included from lib/parse-time-vrp.cc:23:<br>\r
 213 &gt; &gt; In file included from ./lib/database-private.h:33:<br>\r
 214 &gt; &gt; ./lib/notmuch-private.h:479:8: error: visibility does not match p=\r
 215 revious declaration<br>\r
 216 &gt; &gt; struct visible _notmuch_string_list {<br>\r
 217 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0^<br>\r
 218 &gt; &gt; ./lib/notmuch-private.h:67:33: note: expanded from macro &#39;vis=\r
 219 ible&#39;<br>\r
 220 &gt; &gt; # define visible __attribute__((visibility(&quot;default&quot;)))=\r
 221 <br>\r
 222 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =\r
 223 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 224 &gt; &gt; ./lib/notmuch-private.h:52:13: note: previous attribute is here<b=\r
 225 r>\r
 226 &gt; &gt; #pragma GCC visibility push(hidden)<br>\r
 227 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 228 &gt; &gt; 1 warning generated.<br>\r
 229 &gt; &gt; In file included from lib/directory.cc:21:<br>\r
 230 &gt; &gt; ./lib/notmuch-private.h:479:8: error: visibility does not match p=\r
 231 revious declaration<br>\r
 232 &gt; &gt; struct visible _notmuch_string_list {<br>\r
 233 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0^<br>\r
 234 &gt; &gt; ./lib/notmuch-private.h:67:33: note: expanded from macro &#39;vis=\r
 235 ible&#39;<br>\r
 236 &gt; &gt; # define visible __attribute__((visibility(&quot;default&quot;)))=\r
 237 <br>\r
 238 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =\r
 239 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 240 &gt; &gt; ./lib/notmuch-private.h:52:13: note: previous attribute is here<b=\r
 241 r>\r
 242 &gt; &gt; #pragma GCC visibility push(hidden)<br>\r
 243 &gt; &gt; =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ^<br>\r
 244 &gt; &gt;<br>\r
 245 &gt; &gt; and so on. I guess it is because the visibility differs between c=\r
 246  and<br>\r
 247 &gt; &gt; c++. I&#39;ve disabled visibility locally, see second attached pa=\r
 248 tch, but<br>\r
 249 &gt; &gt; of course that&#39;s not a solution, just a workaround. Suggestio=\r
 250 ns<br>\r
 251 &gt; &gt; welcome.<br>\r
 252 &gt; &gt;<br>\r
 253 &gt; &gt; Thanks,<br>\r
 254 &gt; &gt; =C2=A0Thomas<br>\r
 255 &gt; &gt;<br>\r
 256 &gt; &gt; _______________________________________________<br>\r
 257 &gt; &gt; notmuch mailing list<br>\r
 258 &gt; &gt; <a href=3D"mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.or=\r
 259 g</a><br>\r
 260 &gt; &gt; <a href=3D"http://notmuchmail.org/mailman/listinfo/notmuch">http:=\r
 261 //notmuchmail.org/mailman/listinfo/notmuch</a><br>\r
 262 &gt; &gt;<br>\r
 263 </p>\r
 264 \r
 265 --047d7b6dd09cda0cb204ef246ae9--\r