38/ac9a95c5c6793b107a5a17cb43ac67644bde09

   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 2753E40D155\r
   6         for <notmuch@notmuchmail.org>; Thu,  4 Nov 2010 12:26:07 -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: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id d9tCtO1sPwF1; Thu,  4 Nov 2010 12:25:56 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id B50714196F0;\r
  19         Thu,  4 Nov 2010 12:25:56 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id 4566F254102; Thu,  4 Nov 2010 12:25:56 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: Michal Sojka <sojkam1@fel.cvut.cz>, notmuch@notmuchmail.org\r
  24 Subject: Re: [PATCH 0/2] Notmuch cat v2\r
  25 In-Reply-To: <1287739684-26188-1-git-send-email-sojkam1@fel.cvut.cz>\r
  26 References: <1287739684-26188-1-git-send-email-sojkam1@fel.cvut.cz>\r
  27 User-Agent: Notmuch/0.4 (http://notmuchmail.org) Emacs/23.2.1\r
  28         (i486-pc-linux-gnu)\r
  29 Date: Thu, 04 Nov 2010 12:25:56 -0700\r
  30 Message-ID: <874obwriuz.fsf@yoom.home.cworth.org>\r
  31 MIME-Version: 1.0\r
  32 Content-Type: multipart/signed; boundary="=-=-=";\r
  33         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.13\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38         <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Thu, 04 Nov 2010 19:26:07 -0000\r
  47 \r
  48 --=-=-=\r
  49 Content-Transfer-Encoding: quoted-printable\r
  50 \r
  51 On Fri, 22 Oct 2010 11:28:02 +0200, Michal Sojka <sojkam1@fel.cvut.cz> wrot=\r
  52 e:\r
  53 > this is my second attempt to implement notmuch cat subcommand (the\r
  54 > first version was posted in\r
  55 > id:1271747793-17507-1-git-send-email-sojkam1@fel.cvut.cz). This\r
  56 > subcommand outputs the given message to stdout.\r
  57 \r
  58 Hi Michal,\r
  59 \r
  60 I'm *really* excited about this feature. The ability to more easily use\r
  61 notmuch over ssh is really appealing, and as I mentioned previously, I\r
  62 see this patch as a prerequisite for the maildir-synchronization work.\r
  63 \r
  64 > In this version the arguments are classical notmuch search terms and\r
  65 > not a filename as in the previous version. Emacs interface then uses\r
  66 > message-id to retrieve the message.\r
  67 \r
  68 Excellent. This is much better than before.\r
  69 \r
  70 > Some people suggested that cat could be implemented as a special\r
  71 > format of show subcommand. That would be possible, but it seems that\r
  72 > show command always constructs threads form the messages which means\r
  73 > that is executes several database queries. I consider this as\r
  74 > unnecessary overhead and for that reason cat is a separate subcommand.\r
  75 \r
  76 So I'll argue for "notmuch show --format=3Draw" here.\r
  77 \r
  78 You're correct that the current implementation of "notmuch show" calls\r
  79 notmuch_database_search_threads and --format=3Draw doesn't need to or want\r
  80 to do that. But see, for example, recent notmuch-search.c and you'll see\r
  81 that depending on the --output option, the "notmuch search" command will\r
  82 sometimes call search_threads and will sometimes call\r
  83 search_messages. So I think we want the same thing here.\r
  84 \r
  85 Meanwhile, since this command is only acting on the first message\r
  86 matched, don't you think it should be documented as only working with a\r
  87 search term that matches a single message? Then it could simply print an\r
  88 error message if the terms matched multiple messages.[*]\r
  89 \r
  90 =2DCarl\r
  91 \r
  92 [*] And we'll want that same behavior for "notmuch part" as well. That's\r
  93 already implemented in notmuch-show.c and should also be implemented as\r
  94 a sub-option of "notmuch show" I think, (which is probably another\r
  95 outstanding patch for me to review). So there might be room for a little\r
  96 code-sharing there.\r
  97 \r
  98 =2D-=20\r
  99 carl.d.worth@intel.com\r
 100 \r
 101 --=-=-=\r
 102 Content-Type: application/pgp-signature\r
 103 \r
 104 -----BEGIN PGP SIGNATURE-----\r
 105 Version: GnuPG v1.4.10 (GNU/Linux)\r
 106 \r
 107 iD8DBQFM0wjE6JDdNq8qSWgRAjVBAJ4/esUuQcdwEhdWBaq3JqEOKcSlbQCfQX88\r
 108 ve53lOI7bljWgKUTha0qygg=\r
 109 =IGwF\r
 110 -----END PGP SIGNATURE-----\r
 111 --=-=-=--\r