58/a902fb490168df0e328c72707338be43bfccb0

   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 C1350431FD0\r
   6         for <notmuch@notmuchmail.org>; Tue, 23 Nov 2010 18:09:04 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -1\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1] 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 1xbUC3DHyRY0; Tue, 23 Nov 2010 18:09:03 -0800 (PST)\r
  16 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  17         by olra.theworths.org (Postfix) with ESMTP id D3495431FB5;\r
  18         Tue, 23 Nov 2010 18:09:03 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id 5090825412B; Tue, 23 Nov 2010 18:09:03 -0800 (PST)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: notmuch@notmuchmail.org\r
  23 Subject: Inconsistent output from "notmuch search --output=<foo>"\r
  24 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.2.1\r
  25         (i486-pc-linux-gnu)\r
  26 Date: Tue, 23 Nov 2010 18:09:03 -0800\r
  27 Message-ID: <871v6b79s0.fsf@yoom.home.cworth.org>\r
  28 MIME-Version: 1.0\r
  29 Content-Type: text/plain; charset=us-ascii\r
  30 X-BeenThere: notmuch@notmuchmail.org\r
  31 X-Mailman-Version: 2.1.13\r
  32 Precedence: list\r
  33 List-Id: "Use and development of the notmuch mail system."\r
  34         <notmuch.notmuchmail.org>\r
  35 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  36         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  37 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  38 List-Post: <mailto:notmuch@notmuchmail.org>\r
  39 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  40 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  41         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  42 X-List-Received-Date: Wed, 24 Nov 2010 02:09:04 -0000\r
  43 \r
  44 I just committed a bug fix for the missing final newline from "notmuch\r
  45 search --output=tags". I don't recall who reported the bug, but thanks!\r
  46 \r
  47 Michal, in trying to add a test for that bug, I found that the current\r
  48 test suite infrastructure can't catch it because the shell's $()\r
  49 construct doesn't distinguish whether that final newline is present or\r
  50 not. I don't see an easy way to fix this, (other than making all tests\r
  51 put results into files and making test_expect_equal accept those\r
  52 filenames). Do you see any easy fix?\r
  53 \r
  54 Meanwhile, while adding the --output= test, I noticed some inconsistency\r
  55 in the output:\r
  56 \r
  57         $ notmuch search --output=threads ... | head -1\r
  58         thread:0000000000000c3c\r
  59 \r
  60         $ notmuch search --output=messages ... | head -1\r
  61         id:1272355278.3878.111.camel@thor.local\r
  62 \r
  63         $ notmuch search --output=files ... | head -1\r
  64         /path/to/maildir/1272355352.M909256P19063V18F0_0,S=9415\r
  65 \r
  66         $ notmuch search --output=tags ... | head -1\r
  67         attachment\r
  68 \r
  69 The inconsistency is the presence of the "thread:" and "id:" prefixes in\r
  70 the first two cases, (note that there isn't any "tag:" prefix in the\r
  71 last case). I can't find any good justification for these.\r
  72 \r
  73 I think the right answer is to drop those prefixes in the output. Does\r
  74 anybody disagree? Does anyone have any scripts that are already\r
  75 consuming the output of "notmuch search --output=threads" or "notmuch\r
  76 search --output=messages" yet?\r
  77 \r
  78 Note that the --format=json output won't be affected by the change I'm\r
  79 proposing here.\r
  80 \r
  81 -Carl\r
  82 \r
  83 --\r
  84 carl.d.worth@intel.com\r