02/ac8b6ba8ae71f9566490b971e6a78e0b6695cc

   1 Return-Path: <david@tethera.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 12267431FC0\r
   6         for <notmuch@notmuchmail.org>; Sat,  4 Jan 2014 05:18:35 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 bR2E+QpzxqlK for <notmuch@notmuchmail.org>;\r
  16         Sat,  4 Jan 2014 05:18:27 -0800 (PST)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 08E13431FBD\r
  21         for <notmuch@notmuchmail.org>; Sat,  4 Jan 2014 05:18:26 -0800 (PST)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1VzR7L-0005H9-RW; Sat, 04 Jan 2014 09:18:19 -0400\r
  25 Received: (nullmailer pid 21265 invoked by uid 1000); Sat, 04 Jan 2014\r
  26         13:18:15 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Thomas Klausner <tk@giga.or.at>, notmuch@notmuchmail.org\r
  29 Subject: Re: notmuch-0.16: realpath() compatibility issue; clang visibility\r
  30         problem\r
  31 In-Reply-To: <20140103214735.GG27614@danbala.tuwien.ac.at>\r
  32 References: <20140103214735.GG27614@danbala.tuwien.ac.at>\r
  33 User-Agent: Notmuch/0.17 (http://notmuchmail.org) Emacs/24.3.1\r
  34         (x86_64-pc-linux-gnu)\r
  35 Date: Sat, 04 Jan 2014 09:18:15 -0400\r
  36 Message-ID: <871u0nnaw8.fsf@zancas.localnet>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Sat, 04 Jan 2014 13:18:35 -0000\r
  52 \r
  53 Thomas Klausner <tk@giga.or.at> writes:\r
  54 \r
  55 >                                 ^\r
  56 > ./lib/notmuch-private.h:52:13: note: previous attribute is here\r
  57 > #pragma GCC visibility push(hidden)\r
  58 >             ^\r
  59 \r
  60 The clang related issues might be fixed in 0.17; can you try that (or\r
  61 git master)?\r
  62 \r
  63 >      size_t length;\r
  64 > -    char *data, *filename;\r
  65 > +    char *data, filename[MAXPATHLEN];\r
  66 >      GError *error = NULL;\r
  67 \r
  68 I'm not sure what the right answer is here. MATHPATHLEN (and PATH_MAX)\r
  69 are not necessarily defined; in particular this would break\r
  70 compilation on GNU Hurd. Perhaps we should ship a compatibility\r
  71 implementation of a POSIX.1-2008 compatible [1] realpath. Or maybe\r
  72 realpath can be avoided completely here.\r
  73 \r
  74 > +         strcpy(filename, config->filename);\r
  75 \r
  76 Any reason not to use strncpy here?\r
  77 \r
  78 Of course bug reports and fixes in any form are always welcome, but even\r
  79 more appreciated if they roughly follow [2]; mainly patches from git\r
  80 with sensible commit messages, and some minor coding style issues.\r
  81 \r
  82 cheers,\r
  83 \r
  84 d\r
  85 \r
  86 \r
  87 [1]: http://pubs.opengroup.org/onlinepubs/9699919799/ \r
  88 [2]: http://notmuchmail.org/contributing/\r