bc/563720e41e87d6275c1d1a9d7fd19c2768a5bd

   1 Return-Path: <scott@quadhome.com>\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 6BD54431FBF\r
   6         for <notmuch@notmuchmail.org>; Fri, 18 Dec 2009 10:45:21 -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 B4-F0UdsLxyy for <notmuch@notmuchmail.org>;\r
  11         Fri, 18 Dec 2009 10:45:20 -0800 (PST)\r
  12 Received: from megan.quadhome.com (megan.quadhome.com [209.9.237.126])\r
  13         by olra.theworths.org (Postfix) with ESMTP id BC4AC431FAE\r
  14         for <notmuch@notmuchmail.org>; Fri, 18 Dec 2009 10:45:20 -0800 (PST)\r
  15 Received: from elise.quadhome.com (c-24-22-29-24.hsd1.or.comcast.net\r
  16         [24.22.29.24])\r
  17         by megan.quadhome.com (Postfix) with ESMTPSA id 949091083BD\r
  18         for <notmuch@notmuchmail.org>; Fri, 18 Dec 2009 10:45:19 -0800 (PST)\r
  19 Content-Type: text/plain; charset=UTF-8\r
  20 From: Scott Robinson <scott@quadhome.com>\r
  21 To: notmuch <notmuch@notmuchmail.org>\r
  22 In-reply-to: <87iqc4yyo8.fsf@yoom.home.cworth.org>\r
  23 References: <1261114167-sup-8228@lisa>\r
  24         <1261141195-5469-1-git-send-email-david@tethera.net>\r
  25         <87iqc4yyo8.fsf@yoom.home.cworth.org>\r
  26 Date: Fri, 18 Dec 2009 10:45:17 -0800\r
  27 Message-Id: <1261161342-sup-2335@lisa>\r
  28 User-Agent: Sup/git\r
  29 Content-Transfer-Encoding: 8bit\r
  30 Subject: Re: [notmuch] [PATCH] Add an "--output=(json|text|)" command-line\r
  31         option to both notmuch-search and notmuch-show.\r
  32 X-BeenThere: notmuch@notmuchmail.org\r
  33 X-Mailman-Version: 2.1.12\r
  34 Precedence: list\r
  35 List-Id: "Use and development of the notmuch mail system."\r
  36         <notmuch.notmuchmail.org>\r
  37 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  38         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  39 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  40 List-Post: <mailto:notmuch@notmuchmail.org>\r
  41 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  42 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  44 X-List-Received-Date: Fri, 18 Dec 2009 18:45:21 -0000\r
  45 \r
  46 Excerpts from Carl Worth's message of Fri Dec 18 09:33:43 -0800 2009:\r
  47 > On Fri, 18 Dec 2009 08:59:55 -0400, david@tethera.net wrote:\r
  48 > > It took me a little work to apply Scott's patch, so rather than asking\r
  49 > > him to resend it from git-send-email, I am just sending. I hope no-one\r
  50 > > is offended (much).\r
  51 > \r
  52 > I think that's great! Collaboration is what this is all about.\r
  53 \r
  54 Me too!\r
  55 \r
  56 I've never used git-send-email. I'll give it a whirl on my next patch.\r
  57 \r
  58 > > I'm thinking that the patch I sent out last night to only dump message\r
  59 > > ids could be reworked to use the framework of this patch.  I also\r
  60 > > think it would be reasonably simple to add an --output=mbox option,\r
  61 > > for archiving and so on.\r
  62 > \r
  63 > I think that selecting *what* to emit is orthogonal from selecting *how*\r
  64 > to format that output. See some ideas in the TODO file, (where I\r
  65 > proposed --for and --format options for these). Having a way to do mbox\r
  66 > output for export would indeed be very nice.\r
  67 \r
  68 Haha! I originally used "--format" and changed for some reason that escapes me\r
  69 now.\r
  70 \r
  71 Implementing an "mbox" formatted output in the current logic wouldn't be\r
  72 archive perfect. The message body is emitted on a per-part basis.\r
  73 \r
  74 What I would do is change the semantics of format->body to be called from\r
  75 show_message. Then the text and json parts would point at the original\r
  76 implementation passing off their per-part function pointers. And, a new mbox\r
  77 implementation would just dump the full message body.\r
  78 -- \r
  79 Scott Robinson | http://quadhome.com/\r
  80 \r
  81 Q: Why are my replies five sentences or less?\r
  82 A: http://five.sentenc.es/\r