6b/9fa94a8f045ec19f63d721955b9a9035800c9e

   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 8FF86431FC0;\r
   6         Fri, 27 Nov 2009 19:27:12 -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 O0BivdOFRl26; Fri, 27 Nov 2009 19:27:11 -0800 (PST)\r
  11 Received: from cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 68A73431FAE;\r
  13         Fri, 27 Nov 2009 19:27:11 -0800 (PST)\r
  14 From: Carl Worth <cworth@cworth.org>\r
  15 To: Michiel Buddingh' <michiel@michielbuddingh.net>, notmuch@notmuchmail.org\r
  16 In-Reply-To: <4b0eef22.JwxdgTGElffx149F%michiel@michielbuddingh.net>\r
  17 References: <87fx8bygi7.fsf@linux.vnet.ibm.com>\r
  18         <87bpiv4t9h.fsf@yoom.home.cworth.org>\r
  19         <87y6lz39nd.fsf@yoom.home.cworth.org>\r
  20         <20091121221207.GB17268@jukie.net>\r
  21         <9cce5525b093b87fe74d427954ffad89@localhost>\r
  22         <87d43b2oif.fsf@yoom.home.cworth.org>\r
  23         <9bfdedddeab9c58cd45d8d448323d0fc@localhost>\r
  24         <87skc23327.fsf@yoom.home.cworth.org>\r
  25         <4b0eef22.JwxdgTGElffx149F%michiel@michielbuddingh.net>\r
  26 Date: Fri, 27 Nov 2009 19:26:57 -0800\r
  27 Message-ID: <87ws1bjpmm.fsf@yoom.home.cworth.org>\r
  28 MIME-Version: 1.0\r
  29 Content-Type: text/plain; charset=us-ascii\r
  30 Subject: Re: [notmuch] [PATCH] notmuch: Add Maildir directory name as tag\r
  31  name for messages\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: Sat, 28 Nov 2009 03:27:12 -0000\r
  45 \r
  46 On Thu, 26 Nov 2009 22:12:02 +0100, Michiel Buddingh' <michiel@michielbuddingh.net> wrote:\r
  47 > Carl Worth <cworth@cworth.org> wrote:\r
  48 >   I considered that approach; ideally, we could test for the presence of\r
  49 > all three of cur, tmp and new--but this is rather messy to do in the\r
  50 > current treewalk structure.  Taking any one of them as proof positive of\r
  51 > a Maildir might lead to unpleasant surprises--it's not all that incon-\r
  52 > ceivable for someone to name a mail folder 'tmp'.\r
  53 \r
  54 The auto-detection is just three additional stats (at most) for each\r
  55 directory, right? That seems cheap enough to me.\r
  56 \r
  57 >   There's another matter; Some mail stores will place (large) index files\r
  58 > in folder roots, i.e. one level above cur/, tmp/ and new/.  Looking\r
  59 > at the ones dovecot (an IMAP server) uses, I can make out a from header,\r
  60 > a subject header, and a message-id, as plaintext in the first 100k or\r
  61 > so.  It's not all that inconceivable that notmuch might register it as\r
  62 > a 'real' email, with unpleasant consequences for the index.\r
  63 \r
  64 That seems orthogonal to me. Would the dovecot index files be easy to\r
  65 skip with a pattern-based blacklist?\r
  66 \r
  67 >   I've seen some patches fly by that add support for multiple mail\r
  68 > stores.  Turning on Maildir support on a per-directory basis might\r
  69 > resolve that problem while still supporting heterogenous mail archives\r
  70 > to some degree.  I am not convinced we can do the right thing\r
  71 > automatically without causing some grief to a subset of users.\r
  72 \r
  73 With sup, I had the opposite extreme compared to current notmuch. Every\r
  74 maildir in the hierarchy had to be configured independently. That was a\r
  75 lot of pain, and is precisely why notmuch started out by simply taking a\r
  76 single top-level directory.\r
  77 \r
  78 > Haven't tested it, but it seems you can put\r
  79 > \r
  80 > [core]\r
  81 >     whitespace = trailing-space,space-before-tab\r
  82 \r
  83 Yes. According to the documentation of git-config, those two values are\r
  84 the default. But the documentation also only says that these will make\r
  85 "git diff" display the undesired whitespace in red, and "git apply\r
  86 --whitespace=error" refuse to apply.\r
  87 \r
  88 I can't find a builtin way to make "git commit" complain, or I would\r
  89 recommend that.\r
  90 \r
  91 > I'll be happy to implement them, although I'd like for others to chime\r
  92 > in on the configure-as-Maildir vs. autodetect-Maildir issue.  And thanks\r
  93 > for your patience in working through my patch.\r
  94 \r
  95 No problem at all. I'll look forward to the next version of the\r
  96 patch. Consider mine as a vote for autodetection of maildir rather than\r
  97 configuration.\r
  98 \r
  99 -Carl\r