3e/63f0ee9ba257a6ad737323e96a5509ee62b8c9

   1 Return-Path: <jani@nikula.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 8238C431FAF\r
   6         for <notmuch@notmuchmail.org>; Sat, 14 Apr 2012 14:08:00 -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 OzCN4Oi4oKe0 for <notmuch@notmuchmail.org>;\r
  16         Sat, 14 Apr 2012 14:07:59 -0700 (PDT)\r
  17 Received: from mail-lb0-f181.google.com (mail-lb0-f181.google.com\r
  18         [209.85.217.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 8D489431FAE\r
  21         for <notmuch@notmuchmail.org>; Sat, 14 Apr 2012 14:07:59 -0700 (PDT)\r
  22 Received: by lbdb5 with SMTP id b5so209047lbd.26\r
  23         for <notmuch@notmuchmail.org>; Sat, 14 Apr 2012 14:07:57 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=from:to:subject:in-reply-to:references:date:message-id:mime-version\r
  27         :content-type:x-gm-message-state;\r
  28         bh=gjmilRSK9VpeQw12soVGWPOsxZqIZqILR+xwf7F76qg=;\r
  29         b=SKA+35VkydQUQKPLt/wbEKsdFiSxXtCbVSIG0jaqWSeBe2917KIqyrpwbe9LlqlJAk\r
  30         SQ7yoXyXTSg/icnrLfNaQwDZ4PLGkwR7YzjBy2NzsSYY7+D+eQccjISHj6CApxd35ECp\r
  31         RSjPKU3xXXsYR+nAtb+E032rPonRW9iaUxZyUHnKqMPhiKNb+BROwhkVh/I9dqrw3ga1\r
  32         OhECn2HLFQdyDT0XuzfyN/F7eu3vPjO/oz45Y0XRBheR56Rv/jiGWa3oD4NHyI9rqHxS\r
  33         oDomSQNMMtizVqqyyJ3aL7J0puhl2j3Yh1V0kGIHQq/K4ELdveu32D1GvbSxDutBPIW3\r
  34         JiOw==\r
  35 Received: by 10.152.129.137 with SMTP id nw9mr5601976lab.48.1334437677714;\r
  36         Sat, 14 Apr 2012 14:07:57 -0700 (PDT)\r
  37 Received: from localhost (dsl-hkibrasgw4-fe50dc00-68.dhcp.inet.fi.\r
  38         [80.220.80.68])\r
  39         by mx.google.com with ESMTPS id ox7sm17565061lbb.17.2012.04.14.14.07.55\r
  40         (version=SSLv3 cipher=OTHER); Sat, 14 Apr 2012 14:07:56 -0700 (PDT)\r
  41 From: Jani Nikula <jani@nikula.org>\r
  42 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
  43  notmuch@notmuchmail.org\r
  44 Subject: Re: [PATCH v2 0/6] batch tagging support: "notmuch tag --stdin"\r
  45 In-Reply-To: <87d37ays85.fsf@servo.finestructure.net>\r
  46 References: <cover.1334404979.git.jani@nikula.org>\r
  47         <87d37ays85.fsf@servo.finestructure.net>User-Agent:\r
  48         Notmuch/0.12+81~g839a805 (http://notmuchmail.org) Emacs/23.3.1\r
  49         (i686-pc-linux-gnu)\r
  50 Date: Sun, 15 Apr 2012 00:07:53 +0300\r
  51 Message-ID: <87ty0mdnti.fsf@nikula.org>\r
  52 MIME-Version: 1.0\r
  53 Content-Type: text/plain; charset=us-ascii\r
  54 X-Gm-Message-State:\r
  55  ALoCoQkDsgaK8XSM7ekX7NfU3IjVRQXrXwVmAtVqJKd4Rq+g5dbKvRqz1mmeoKj7/Rozw5jpRoVg\r
  56 X-BeenThere: notmuch@notmuchmail.org\r
  57 X-Mailman-Version: 2.1.13\r
  58 Precedence: list\r
  59 List-Id: "Use and development of the notmuch mail system."\r
  60         <notmuch.notmuchmail.org>\r
  61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  64 List-Post: <mailto:notmuch@notmuchmail.org>\r
  65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  68 X-List-Received-Date: Sat, 14 Apr 2012 21:08:00 -0000\r
  69 \r
  70 Jameson Graef Rollins <jrollins@finestructure.net> writes:\r
  71 \r
  72 > On Sat, Apr 14 2012, Jani Nikula <jani@nikula.org> wrote:\r
  73 >> This series adds support for batch tagging through stdin to "notmuch\r
  74 >> tag". This should be useful and efficient for e.g. initial tagging\r
  75 >> scripts. Also, this adds locking around a batch of tag changes, which is\r
  76 >> also useful for initial tagging scripts. See the test patch for an\r
  77 >> example using a "here document".\r
  78 >\r
  79 > My issues from v1 still stand.  I would rather see some unification with\r
  80 > the existing tag file format, rather than introduce a second format,\r
  81 > with it's increased maintenance burden and confusion to users.  Either\r
  82 > that or a more general batch command interface.\r
  83 \r
  84 The existing dump/restore file format is a dead end. It fails\r
  85 magnificently with tags and message-ids that have spaces or some other\r
  86 special characters in them. It doesn't support removal of tags. There's\r
  87 zero room for extensibility. I can't imagine how that could be used for\r
  88 notmuch tag.\r
  89 \r
  90 David originally wrote the dump/restore patches\r
  91 id:"1324214111-32079-1-git-send-email-david@tethera.net" to add a\r
  92 "notmuch" format (yes, a second format) in addition to the "sup" format\r
  93 we currently have, to fix the issues with special characters. That too\r
  94 lacked features to support batch tagging in notmuch new. The format\r
  95 proposed here would fix the issues and work with notmuch tag, and it\r
  96 would be obvious to use for anyone who has used notmuch tag.\r
  97 \r
  98 I see a general batch command interface overkill. Too many problems to\r
  99 solve for little gain (see the previous thread for details). What other\r
 100 notmuch commands than tag would really benefit from it? If you want to\r
 101 do search or show or whatever, the general batch command interface to\r
 102 use is /bin/sh.\r
 103 \r
 104 For notmuch tag, particularly in connection with notmuch new, there's a\r
 105 clear benefit in having a batch mode: if you have "new" in new.tags, and\r
 106 do initial tagging on the "new" tagged messages (e.g. in post-new hook),\r
 107 you don't want anyone messing with the database while you're processing\r
 108 the messages tagged "new". The batch tagging of this series keeps the db\r
 109 locked for the duration of such initial tagging.\r
 110 \r
 111 I'm totally fine with modifying the proposed format (e.g. change "T" to\r
 112 "tag", make things compatible with a future general batch mode), but to\r
 113 be absolutely clear: I will not implement a general batch command\r
 114 mode. If that is deemed to be what is wanted, that's okay too. I can\r
 115 share my WIP patches for dump/restore, and focus on something else.\r
 116 \r
 117 \r
 118 BR,\r
 119 Jani.\r