c0/9acaf712d6ac170b16aef98e5c20b7f3140a69

   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 arlo.cworth.org (Postfix) with ESMTP id 75DC16DE0926\r
   6  for <notmuch@notmuchmail.org>; Thu, 11 Aug 2016 23:17:40 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.008\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.008 tagged_above=-999 required=5 tests=[AWL=0.003,\r
  12   SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id AiIIQJSqcdSU for <notmuch@notmuchmail.org>;\r
  16  Thu, 11 Aug 2016 23:17:32 -0700 (PDT)\r
  17 Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197])\r
  18  by arlo.cworth.org (Postfix) with ESMTPS id 0A60D6DE091F\r
  19  for <notmuch@notmuchmail.org>; Thu, 11 Aug 2016 23:17:31 -0700 (PDT)\r
  20 Received: from remotemail by fethera.tethera.net with local (Exim 4.84_2)\r
  21  (envelope-from <david@tethera.net>)\r
  22  id 1bY5mq-0006tH-P3; Fri, 12 Aug 2016 02:17:44 -0400\r
  23 Received: (nullmailer pid 21013 invoked by uid 1000);\r
  24  Fri, 12 Aug 2016 06:17:28 -0000\r
  25 From: David Bremner <david@tethera.net>\r
  26 To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>,\r
  27  Notmuch Mail <notmuch@notmuchmail.org>\r
  28 Subject: Re: [PATCH v4 02/16] Move crypto.c into libutil\r
  29 In-Reply-To: <1467970047-8013-3-git-send-email-dkg@fifthhorseman.net>\r
  30 References: <1467970047-8013-1-git-send-email-dkg@fifthhorseman.net>\r
  31  <1467970047-8013-3-git-send-email-dkg@fifthhorseman.net>\r
  32 User-Agent: Notmuch/0.22.1+61~g2ce0f13 (https://notmuchmail.org) Emacs/24.5.1\r
  33  (x86_64-pc-linux-gnu)\r
  34 Date: Fri, 12 Aug 2016 15:17:28 +0900\r
  35 Message-ID: <874m6qqznr.fsf@maritornes.cs.unb.ca>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.20\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42  <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  44  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Fri, 12 Aug 2016 06:17:40 -0000\r
  51 \r
  52 Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:\r
  53 > +++ b/util/crypto.c\r
  54 > @@ -0,0 +1,138 @@\r
  55 \r
  56 > +#include "notmuch.h"\r
  57 \r
  58 It feels wrong to me for a file in util/ to include notmuch.h.  It seems\r
  59 the same situation holds with search-path.h.  There it seems we use\r
  60 notmuch_bool_t (although I'm not convinced that's the right return\r
  61 type). If that's the only reason maybe we should either factor out the\r
  62 definition or just return ints.\r
  63 \r
  64 > +/* Create a PKCS7 context (GMime 2.6) */\r
  65 > +static notmuch_crypto_context_t *\r
  66 > +create_pkcs7_context (notmuch_crypto_t *crypto)\r
  67 > +{\r
  68 > +    notmuch_crypto_context_t *pkcs7ctx;\r
  69 \r
  70 I guess this is leftover, and causes the build failure.\r
  71 \r
  72 > +\r
  73 > +#include "notmuch.h"\r
  74 \r
  75 Same questions notmuch.h here.\r
  76 \r
  77 d\r