da/45691030f753072eb592168aa941ecab7a3246

   1 Return-Path: <tomi.ollila@iki.fi>\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 0D4B9431FBF\r
   6         for <notmuch@notmuchmail.org>; Sun, 31 Mar 2013 01:50:22 -0700 (PDT)\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 54SC6uTMeSx1 for <notmuch@notmuchmail.org>;\r
  16         Sun, 31 Mar 2013 01:50:20 -0700 (PDT)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 91740431FAE\r
  19         for <notmuch@notmuchmail.org>; Sun, 31 Mar 2013 01:50:20 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21         by guru.guru-group.fi (Postfix) with ESMTP id 7542F100148;\r
  22         Sun, 31 Mar 2013 11:50:19 +0300 (EEST)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH v2 0/3] cli: clean up gmime 2.4 vs. 2.6 conditional build\r
  26 In-Reply-To: <cover.1364651359.git.jani@nikula.org>\r
  27 References: <cover.1364651359.git.jani@nikula.org>\r
  28 User-Agent: Notmuch/0.15.2+50~g43fdda5 (http://notmuchmail.org) Emacs/24.2.1\r
  29         (x86_64-unknown-linux-gnu)\r
  30 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  31         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  32         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  33 Date: Sun, 31 Mar 2013 11:50:19 +0300\r
  34 Message-ID: <m2ppyggdh0.fsf@guru.guru-group.fi>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.13\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41         <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Sun, 31 Mar 2013 08:50:22 -0000\r
  50 \r
  51 On Sat, Mar 30 2013, Jani Nikula <jani@nikula.org> wrote:\r
  52 \r
  53 > This is v2 of [1]. Added comments per David's request, and while at it,\r
  54 > added a third patch to conform the existing conditional build in notmuch\r
  55 > show to the same style. The whole series should have no functional\r
  56 > changes, and thus v2 should have no functional changes since v1. ;)\r
  57 >\r
  58 > I have not tested this on gmime 2.4.\r
  59 \r
  60 I have gmime 2.4(.24) on this system. The code compiles and\r
  61 exactly the same set of tests pass/fail as without this patch\r
  62 (unfortunately this system cannot cope with just some of the\r
  63 crypto patches, so I cannot give absolute guarantee:\r
  64 \r
  65 crypto: Testing PGP/MIME signature verification and decryption\r
  66  PASS   emacs delivery of signed message\r
  67  PASS   signature verification\r
  68  PASS   signature verification with full owner trust\r
  69  PASS   signature verification with signer key unavailable\r
  70  PASS   emacs delivery of encrypted message with attachment\r
  71  FAIL   decryption, --format=text\r
  72  FAIL   decryption, --format=json\r
  73  FAIL   decryption, --format=json, --part=4\r
  74  FAIL   decrypt attachment (--part=5 --format=raw)\r
  75  FAIL   decryption failure with missing key\r
  76  PASS   emacs delivery of encrypted + signed message\r
  77  FAIL   decryption + signature verification\r
  78  FAIL   reply to encrypted message\r
  79  FAIL   signature verification with revoked key\r
  80 \r
  81 anyway, as said, exactly the same set fails without these pathes\r
  82 \r
  83 The reason for failure is most probably in my system and not something\r
  84 w/ notmuch crypto support -- If I run `emacs foo.org.gpg` this just\r
  85 hangs waiting me to give passphrase in (invisible?) curses request)\r
  86 \r
  87 So, bottom line: LGTM.\r
  88 \r
  89 > BR,\r
  90 > Jani.\r
  91 \r
  92 Tomi\r
  93 \r
  94 \r
  95 > [1] id:a9f12ba474fc51df71e2fd2b7a20a8d101729c6e.1362319765.git.jani@nikula.org\r
  96 >\r
  97 > Jani Nikula (3):\r
  98 >   cli: crypto: abstract gpg context creation for clarity\r
  99 >   cli: mime node: abstract decryption and signature verification\r
 100 >   cli: conform to same conditional build style as elsewhere in\r
 101 >     notmuch-show\r
 102 >\r
 103 >  crypto.c       |   66 ++++++++++++-----\r
 104 >  mime-node.c    |  218 ++++++++++++++++++++++++++++++++++++--------------------\r
 105 >  notmuch-show.c |   46 ++++++------\r
 106 >  3 files changed, 215 insertions(+), 115 deletions(-)\r
 107 >\r
 108 > -- \r
 109 > 1.7.10.4\r
 110 >\r
 111 > _______________________________________________\r
 112 > notmuch mailing list\r
 113 > notmuch@notmuchmail.org\r
 114 > http://notmuchmail.org/mailman/listinfo/notmuch\r