65/efdd1058b84a5797af45c4b7771086574564ad

   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 0A2EB431FAF\r
   6         for <notmuch@notmuchmail.org>; Tue, 22 Jan 2013 12:05:20 -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: -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 wM3yW1+qhuh1 for <notmuch@notmuchmail.org>;\r
  16         Tue, 22 Jan 2013 12:05:19 -0800 (PST)\r
  17 Received: from mail-lb0-f178.google.com (mail-lb0-f178.google.com\r
  18         [209.85.217.178]) (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 2680B431FAE\r
  21         for <notmuch@notmuchmail.org>; Tue, 22 Jan 2013 12:05:19 -0800 (PST)\r
  22 Received: by mail-lb0-f178.google.com with SMTP id n1so3104434lba.23\r
  23         for <notmuch@notmuchmail.org>; Tue, 22 Jan 2013 12:05:16 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=x-received:from:to:subject:in-reply-to:references:user-agent:date\r
  27         :message-id:mime-version:content-type:x-gm-message-state;\r
  28         bh=Ly/rtQouJk5D/J3PT7W/8PdqNdFYTdbAu6XHhb7Jseo=;\r
  29         b=X5mLXYHzjBKC3t/mk+kJELvnjJQ5YPE495ysXEdNFJ42QIj9ara8snb6z/hfwtsW6Y\r
  30         aP2myNEDWPxZKyK2l3lAVwwqK2NJNVRENhs0KVFq1Ag4Cp5+wtCPE6Jb+IKU4eC2vmUA\r
  31         XnmXm2dCUEbwHHeHdMqpeSiQnTGfocO6ZoPQmUz/7rrGqZX7ncd51qiTPPA0jgK0evTM\r
  32         cxFaWi8yhXKo1oys3jbqA8dPze70gB78WX/UfOrRbr2gFjNshlKRVwhpCyzFSyALX50W\r
  33         RRhg07mcsn/1CGrvLLw5FMBnRYWtiOtGp0C+3D99yV3zZbfglFRuoKnir0LH8Ht1hGva\r
  34         jZVA==\r
  35 X-Received: by 10.112.82.202 with SMTP id k10mr9881137lby.22.1358885116256;\r
  36         Tue, 22 Jan 2013 12:05:16 -0800 (PST)\r
  37 Received: from localhost (dsl-hkibrasgw4-50df51-27.dhcp.inet.fi.\r
  38         [80.223.81.27])\r
  39         by mx.google.com with ESMTPS id hc20sm7441096lab.11.2013.01.22.12.05.14\r
  40         (version=TLSv1.2 cipher=RC4-SHA bits=128/128);\r
  41         Tue, 22 Jan 2013 12:05:15 -0800 (PST)\r
  42 From: Jani Nikula <jani@nikula.org>\r
  43 To: Peter Wang <novalazy@gmail.com>, notmuch@notmuchmail.org\r
  44 Subject: Re: [PATCH v3 00/20] insert command\r
  45 In-Reply-To: <1358643004-14522-1-git-send-email-novalazy@gmail.com>\r
  46 References: <1358643004-14522-1-git-send-email-novalazy@gmail.com>\r
  47 User-Agent: Notmuch/0.14+255~gff3cc55 (http://notmuchmail.org) Emacs/24.2.1\r
  48         (x86_64-pc-linux-gnu)\r
  49 Date: Tue, 22 Jan 2013 22:05:15 +0200\r
  50 Message-ID: <87zk01htl0.fsf@nikula.org>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain\r
  53 X-Gm-Message-State:\r
  54  ALoCoQlD3lFI8b0vG66zA/9T1/QLU73qrqSP7WBQUUR7ejDZKg0v7nuifgNkXJuds2fUl/v4WJmU\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Tue, 22 Jan 2013 20:05:20 -0000\r
  68 \r
  69 On Sun, 20 Jan 2013, Peter Wang <novalazy@gmail.com> wrote:\r
  70 > This mainly addresses review comments from v2\r
  71 > and rebases on top of tag-util.c.\r
  72 \r
  73 Hi Peter, I have some interest in seeing this get merged. To get that\r
  74 done in a timely manner, my suggestion is to do this in two steps. Drop\r
  75 the --folder and --create-folder options and the related code for now,\r
  76 and submit them as another series once the first part has been\r
  77 merged. It will be faster to iterate the first part, and we can dodge\r
  78 some of the issues in the --folder and --create-folder options for now.\r
  79 \r
  80 This is just my personal opinion, and it's up to you whether you do this\r
  81 or not. Let's just call it a hunch saying it'll be faster this way. ;)\r
  82 \r
  83 The patches would be rearranged as follows:\r
  84 \r
  85 > Peter Wang (20):\r
  86 >   tag-util: do not reset list in parse_tag_command_line\r
  87 >   tag-util: move out 'tag' command-line checks\r
  88 \r
  89 Keep the above separate, and make them the first preparatory patches.\r
  90 \r
  91 >   cli: add stub for insert command\r
  92 >   insert: open Maildir tmp file\r
  93 >   insert: copy stdin to Maildir tmp file\r
  94 >   insert: move file from Maildir tmp to new\r
  95 >   insert: add new message to database\r
  96 >   insert: apply default tags to new message\r
  97 >   insert: parse and apply command-line tag operations\r
  98 >   insert: fsync after writing tmp file\r
  99 >   insert: trap SIGINT and clean up\r
 100 >   insert: add copyright line from notmuch-deliver\r
 101 \r
 102 Squash all of the above together into one patch. I will send a review of\r
 103 these based on this version of the series.\r
 104 \r
 105 >   insert: support --folder option\r
 106 >   insert: prevent writes outside Maildir hierarchy\r
 107 >   insert: add --create-folder option\r
 108 >   insert: fsync new directory after rename\r
 109 >   insert: fsync parent directory after mkdir\r
 110 \r
 111 Drop these for now.\r
 112 \r
 113 >   man: document 'insert' command\r
 114 >   man: reference notmuch-insert.1\r
 115 >   test: add tests for insert\r
 116 \r
 117 Keep these separate as they are, but update to reflect the above.\r
 118 \r
 119 Thoughts?\r
 120 \r
 121 \r
 122 BR,\r
 123 Jani.\r
 124 \r
 125 >\r
 126 >  Makefile.local                  |   1 +\r
 127 >  man/Makefile.local              |   1 +\r
 128 >  man/man1/notmuch-config.1       |   4 +-\r
 129 >  man/man1/notmuch-count.1        |   4 +-\r
 130 >  man/man1/notmuch-dump.1         |   4 +-\r
 131 >  man/man1/notmuch-insert.1       |  59 +++++\r
 132 >  man/man1/notmuch-new.1          |   4 +-\r
 133 >  man/man1/notmuch-reply.1        |   3 +-\r
 134 >  man/man1/notmuch-restore.1      |   3 +-\r
 135 >  man/man1/notmuch-search.1       |   3 +-\r
 136 >  man/man1/notmuch-show.1         |   3 +-\r
 137 >  man/man1/notmuch-tag.1          |   3 +-\r
 138 >  man/man1/notmuch.1              |   3 +-\r
 139 >  man/man5/notmuch-hooks.5        |   4 +-\r
 140 >  man/man7/notmuch-search-terms.7 |   3 +-\r
 141 >  notmuch-client.h                |   3 +\r
 142 >  notmuch-insert.c                | 482 ++++++++++++++++++++++++++++++++++++++++\r
 143 >  notmuch-tag.c                   |  10 +\r
 144 >  notmuch.c                       |   3 +\r
 145 >  tag-util.c                      |  12 -\r
 146 >  tag-util.h                      |   2 +\r
 147 >  test/insert                     | 106 +++++++++\r
 148 >  test/notmuch-test               |   1 +\r
 149 >  23 files changed, 692 insertions(+), 29 deletions(-)\r
 150 >  create mode 100644 man/man1/notmuch-insert.1\r
 151 >  create mode 100644 notmuch-insert.c\r
 152 >  create mode 100755 test/insert\r
 153 >\r
 154 > -- \r
 155 > 1.7.12.1\r
 156 >\r
 157 > _______________________________________________\r
 158 > notmuch mailing list\r
 159 > notmuch@notmuchmail.org\r
 160 > http://notmuchmail.org/mailman/listinfo/notmuch\r