b6/77330ddac7182c87f268e1f7f7b6141d43052a

   1 Return-Path: <pieter@praet.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 F3F9B42119B\r
   6         for <notmuch@notmuchmail.org>; Thu, 30 Jun 2011 10:54:15 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 ycOxF72684mA for <notmuch@notmuchmail.org>;\r
  16         Thu, 30 Jun 2011 10:54:15 -0700 (PDT)\r
  17 Received: from mail-wy0-f181.google.com (mail-wy0-f181.google.com\r
  18         [74.125.82.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 4F15142118E\r
  21         for <notmuch@notmuchmail.org>; Thu, 30 Jun 2011 10:54:15 -0700 (PDT)\r
  22 Received: by wyh22 with SMTP id 22so1913738wyh.26\r
  23         for <notmuch@notmuchmail.org>; Thu, 30 Jun 2011 10:54:14 -0700 (PDT)\r
  24 Received: by 10.227.32.69 with SMTP id b5mr2071390wbd.99.1309456453917;\r
  25         Thu, 30 Jun 2011 10:54:13 -0700 (PDT)\r
  26 Received: from localhost ([109.131.21.173])\r
  27         by mx.google.com with ESMTPS id gb1sm1792496wbb.54.2011.06.30.10.54.12\r
  28         (version=TLSv1/SSLv3 cipher=OTHER);\r
  29         Thu, 30 Jun 2011 10:54:13 -0700 (PDT)\r
  30 From: Pieter Praet <pieter@praet.org>\r
  31 To: Carl Worth <cworth@cworth.org>, notmuch@notmuchmail.org\r
  32 Subject: Re: [PATCH] remove prefixes from `--output={threads,\r
  33         messages}' results\r
  34 In-Reply-To: <871uyb479v.fsf@yoom.home.cworth.org>\r
  35 References: <877hbiep5g.fsf@SSpaeth.de>\r
  36         <1309421989-22410-1-git-send-email-pieter@praet.org>\r
  37         <871uyb479v.fsf@yoom.home.cworth.org>\r
  38 User-Agent: Notmuch/0.5-315-g34bd5eb (http://notmuchmail.org) Emacs/23.1.50.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Thu, 30 Jun 2011 19:54:10 +0200\r
  41 Message-ID: <87fwmrdx31.fsf@praet.org>\r
  42 MIME-Version: 1.0\r
  43 Content-Type: text/plain; charset=us-ascii\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Thu, 30 Jun 2011 17:54:16 -0000\r
  57 \r
  58 On Thu, 30 Jun 2011 09:24:12 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  59 Non-text part: multipart/signed\r
  60 > On Thu, 30 Jun 2011 10:19:49 +0200, Pieter Praet <pieter@praet.org> wrote:\r
  61 > > Alter `do_search_threads()' and `do_search_messages()'\r
  62 > > to not prepend each result with `thread:' respectively `id:'.\r
  63 > \r
  64 > My one concern here is that I've sometimes had a message-id without the\r
  65 > id prefix and run a command like this:\r
  66 > \r
  67 >     notmuch search 1309421989-22410-1-git-send-email-pieter@praet.org\r
  68 > \r
  69 > And I've gotten confused when I've received no output, (didn't I receive\r
  70 > that mail? what happened?).\r
  71 \r
  72 And unfortunately, that's not the only concern.\r
  73 \r
  74 While this patch removes a mere 10 chars, it has (way too) far-reaching\r
  75 consequences, not only for the test suite, but for the entire codebase.\r
  76 \r
  77 Truth be told, this was never intended to be merged. I only sent this\r
  78 patch to give a ~fair chance to what appears to be the general\r
  79 consensus, but I much prefer the fix for do_search_tags() [1].\r
  80 \r
  81 In fact, all the followup patches (the seemingly haphazard reply nesting\r
  82 is intentional) are intended to be used with the do_search_tags() fix,\r
  83 which I tentatively consider a much cleaner path.\r
  84 \r
  85 > But I think this is a separate bug where the right fix is to make any\r
  86 > search terms with no prefix search through *all* prefixed terms\r
  87 > generated from email content. This would allow us to also avoid indexing\r
  88 > some content twice, (currently we store subject, from, and to both with\r
  89 > a prefix and without a prefix).\r
  90 \r
  91 Indeed a most desirable (albeit perhaps long-term) goal.\r
  92 \r
  93 Much as I'd like, I won't be able to make myself useful in that area for\r
  94 the time being, as the only C I consider myself sufficiently capable of\r
  95 pushing around is the one ending with "offee" :<\r
  96 \r
  97 > -Carl\r
  98 > \r
  99 > -- \r
 100 > carl.d.worth@intel.com\r
 101 Non-text part: application/pgp-signature\r
 102 \r
 103 Peace\r
 104 \r
 105 -- \r
 106 Pieter\r
 107 \r
 108 [1] id:"1309422029-22924-1-git-send-email-pieter@praet.org"\r