94/5f4ddc7d59fb54a421de1619ad4ef1f70814a0

   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 33450431FD2\r
   6         for <notmuch@notmuchmail.org>; Sat,  1 Nov 2014 04:30:57 -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 0bykIjZrjRNr for <notmuch@notmuchmail.org>;\r
  16         Sat,  1 Nov 2014 04:30:53 -0700 (PDT)\r
  17 Received: from mail-la0-f44.google.com (mail-la0-f44.google.com\r
  18         [209.85.215.44]) (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 A63B6431FAF\r
  21         for <notmuch@notmuchmail.org>; Sat,  1 Nov 2014 04:30:52 -0700 (PDT)\r
  22 Received: by mail-la0-f44.google.com with SMTP id gf13so7518393lab.3\r
  23         for <notmuch@notmuchmail.org>; Sat, 01 Nov 2014 04:30:50 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\r
  27         :references;\r
  28         bh=zZsqh5Nz8uol2/SuV1QivkDVHs8L+uZNjvI0FdMbWyg=;\r
  29         b=NZEuDYpawmG1ATP0jXp1bymr7i339m8jHR8k68QctihNVQi0LTNkZfjmRa4GzundtF\r
  30         K8A8+CFlJTOc7ai+CNJrpENw+IIPoRk+vsWSzXxlWgywe8Zw+SJvUP1EVHsCpBTrYLfY\r
  31         PyOpQzIMITq1FFbDuDHP1Ue0hX7zabT4C3YdHNSBcfevaUgTONPJSc6ph2DUYcQSs1D5\r
  32         14eAQ4mO+2T4CmeqiV+gHEXezHQSkw+a/E8Qt8nXg/AXXb5hLLEW6muOBCGzsyguMLz/\r
  33         16G6ZXQFyRPqnJzKMXsDyLdhOaz8ElrycmEsNxzIaKVJcc2h68FnjZAVwYZ/tEpZkXqR\r
  34         lveA==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQk7RwTgB1PXElx+Sr6SDrgCUEca4GijUL2WTDDWwyMkVkYSkuiTxkB+pG2wM1drlVBdbgdP\r
  37 X-Received: by 10.113.5.7 with SMTP id ci7mr33664147lbd.9.1414841450887;\r
  38         Sat, 01 Nov 2014 04:30:50 -0700 (PDT)\r
  39 Received: from localhost (dsl-hkibrasgw2-58c36d-48.dhcp.inet.fi.\r
  40         [88.195.109.48])\r
  41         by mx.google.com with ESMTPSA id g5sm5528456lam.1.2014.11.01.04.30.49\r
  42         for <multiple recipients>\r
  43         (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\r
  44         Sat, 01 Nov 2014 04:30:50 -0700 (PDT)\r
  45 From: Jani Nikula <jani@nikula.org>\r
  46 To: Michal Sojka <sojkam1@fel.cvut.cz>\r
  47 Subject: [RFC PATCH 0/2] cli: split 'notmuch address' from 'notmuch search'\r
  48 Date: Sat,  1 Nov 2014 13:30:46 +0200\r
  49 Message-Id: <cover.1414839970.git.jani@nikula.org>\r
  50 X-Mailer: git-send-email 2.1.1\r
  51 In-Reply-To: <87egtn2s4z.fsf@steelpick.2x.cz>\r
  52 References: <87egtn2s4z.fsf@steelpick.2x.cz>\r
  53 Cc: notmuch@notmuchmail.org\r
  54 X-BeenThere: notmuch@notmuchmail.org\r
  55 X-Mailman-Version: 2.1.13\r
  56 Precedence: list\r
  57 List-Id: "Use and development of the notmuch mail system."\r
  58         <notmuch.notmuchmail.org>\r
  59 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  61 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  62 List-Post: <mailto:notmuch@notmuchmail.org>\r
  63 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  64 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  66 X-List-Received-Date: Sat, 01 Nov 2014 11:30:57 -0000\r
  67 \r
  68 On Sat, 01 Nov 2014, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  69 > On Sat, Nov 01 2014, Jani Nikula wrote:\r
  70 >> On Fri, 31 Oct 2014, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  71 >>> This option allows to configure the criterion for duplicate address\r
  72 >>> filtering. Without this option, all unique combinations of name and\r
  73 >>> address parts are printed. This option allows to filter the output\r
  74 >>> more, for example to only contain unique address parts.\r
  75 >>\r
  76 >> This patch finally makes me think we should have a separate 'notmuch\r
  77 >> address' command for all of this. We are starting to have two orthogonal\r
  78 >> sets of 'notmuch search' options, one set for search and another for\r
  79 >> addresses. I regret not following the series and then making the\r
  80 >> observation so late.\r
  81 >\r
  82 > To some extent it is true. The question is whether it would make sense\r
  83 > for 'notmuch address' to have options like --offset and --limit.\r
  84 > Probably not.\r
  85 >\r
  86 > I can prepare a new patchset implementing 'notmuch address' over the\r
  87 > weekend. It would be necessary to revert the last (or two) already\r
  88 > merged patch. What do others think?\r
  89 \r
  90 Here's a couple of draft patches to make the search/address command\r
  91 split happen with minimal changes.\r
  92 \r
  93 In my opinion the problem is primarily in the user interface;\r
  94 notmuch-search man page is growing intimidating to the user. I also\r
  95 think splitting the implementation completely would lead to too much\r
  96 code duplication, as there is plenty of common code. The idea here is to\r
  97 split the user interface for address searching now when we haven't made\r
  98 a release with the functionality, and we can more easily change the code\r
  99 behind the interface going forward. Plus we can more comfortably add\r
 100 address management commands to 'notmuch address' than 'notmuch search'\r
 101 as the need arises.\r
 102 \r
 103 I do have to say in the end I'm also fine with having everything in the\r
 104 notmuch search command if there's opposition to having two very similar,\r
 105 even if diverging, commands.\r
 106 \r
 107 BR,\r
 108 Jani.\r
 109 \r
 110 \r
 111 Jani Nikula (2):\r
 112   cli: add support for hierarchical command line option arrays\r
 113   cli: add notmuch address command\r
 114 \r
 115  command-line-arguments.c |  11 ++--\r
 116  command-line-arguments.h |   1 +\r
 117  notmuch-client.h         |   3 ++\r
 118  notmuch-search.c         | 128 ++++++++++++++++++++++++++++++-----------------\r
 119  notmuch.c                |   2 +\r
 120  5 files changed, 95 insertions(+), 50 deletions(-)\r
 121 \r
 122 -- \r
 123 2.1.1\r
 124 \r