6b/523e1660f184d651e9a08e4757967c20cb8bc0

   1 Return-Path: <cworth@cworth.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 A23C9431FC0;\r
   6         Fri, 27 Nov 2009 22:22:16 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id H7hjlaqb09Jg; Fri, 27 Nov 2009 22:22:16 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id EB12D431FAE;\r
  13         Fri, 27 Nov 2009 22:22:15 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: "Jeffrey C. Ollie" <jeff@ocjtech.us>, Not Much Mail\r
  16         <notmuch@notmuchmail.org>\r
  17 In-Reply-To: <1259378883-9181-1-git-send-email-jeff@ocjtech.us>\r
  18 References: <1259378883-9181-1-git-send-email-jeff@ocjtech.us>\r
  19 Date: Fri, 27 Nov 2009 22:22:01 -0800\r
  20 Message-ID: <87zl67rwxi.fsf@yoom.home.cworth.org>\r
  21 MIME-Version: 1.0\r
  22 Content-Type: multipart/signed; boundary="=-=-=";\r
  23         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  24 Subject: Re: [notmuch] [PATCH] Use libgcrypt for hashing.\r
  25 X-BeenThere: notmuch@notmuchmail.org\r
  26 X-Mailman-Version: 2.1.12\r
  27 Precedence: list\r
  28 List-Id: "Use and development of the notmuch mail system."\r
  29         <notmuch.notmuchmail.org>\r
  30 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  31         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  32 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  33 List-Post: <mailto:notmuch@notmuchmail.org>\r
  34 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  35 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  36         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  37 X-List-Received-Date: Sat, 28 Nov 2009 06:22:16 -0000\r
  38 \r
  39 --=-=-=\r
  40 \r
  41 On Fri, 27 Nov 2009 21:28:03 -0600, "Jeffrey C. Ollie" <jeff@ocjtech.us> wrote:\r
  42 > Instead of including a private implementation of the SHA1 hash, use\r
  43 > libgcrypt.  This means less code of our own to maintain and it will be\r
  44 > easier to switch to a different hash function like SHA256.\r
  45 \r
  46 I don't believe we have a significant code-maintenance burden with\r
  47 libsha1.c. And as for different hash functions, the only use of sha-1 in\r
  48 notmuch is as a fallback in the case of a message not including a\r
  49 Message-ID header.\r
  50 \r
  51 So I don't see it as important at all to try to remove this code.\r
  52 \r
  53 > libgcrypt was chosen because it has a fairly simple API, it's well\r
  54 > tested (it's used in gnutls and gnupg2), and it's licensed under the\r
  55 > LGPL.\r
  56 \r
  57 What might make more sense is an option to compile against an existing\r
  58 library (if present) but not to introduce an error in the build if the\r
  59 library is not present, (in which case just build the builtin libsha1.c\r
  60 code).\r
  61 \r
  62 But if that wouldn't solve the problem you were trying to solve, (to\r
  63 actually remove libsha1.c), then maybe we don't need to do anything for\r
  64 now?\r
  65 \r
  66 -Carl\r
  67 \r
  68 --=-=-=\r
  69 Content-Type: application/pgp-signature\r
  70 \r
  71 -----BEGIN PGP SIGNATURE-----\r
  72 Version: GnuPG v1.4.10 (GNU/Linux)\r
  73 \r
  74 iD8DBQFLEMGJ6JDdNq8qSWgRAvfWAJ4oS+MggzMOlvdSsXg7pkkdII/KYgCfaC2d\r
  75 klL7VI/fykiVkVECXRP9OJI=\r
  76 =3s26\r
  77 -----END PGP SIGNATURE-----\r
  78 --=-=-=--\r