03/5c58eef7c9ef6014319d01a7045536ba6bad9c

   1 Return-Path: <bremner@tethera.net>\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 AE94B431FBF\r
   6         for <notmuch@notmuchmail.org>; Sat, 19 Jan 2013 10:26:12 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 M3gzGaXvVuuh for <notmuch@notmuchmail.org>;\r
  16         Sat, 19 Jan 2013 10:26:11 -0800 (PST)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id B2902431FAF\r
  21         for <notmuch@notmuchmail.org>; Sat, 19 Jan 2013 10:26:11 -0800 (PST)\r
  22 Received: from fctnnbsc30w-156034082078.dhcp-dynamic.fibreop.nb.bellaliant.net\r
  23         ([156.34.82.78] helo=zancas.localnet)\r
  24         by tesseract.cs.unb.ca with esmtpsa\r
  25         (TLS1.2:DHE_RSA_AES_128_CBC_SHA1:128) (Exim 4.80)\r
  26         (envelope-from <bremner@tethera.net>)\r
  27         id 1Twd7I-0007xI-Ia; Sat, 19 Jan 2013 14:26:09 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.80)\r
  29         (envelope-from <bremner@tethera.net>)\r
  30         id 1Twd7D-0005ae-0a; Sat, 19 Jan 2013 14:26:03 -0400\r
  31 From: david@tethera.net\r
  32 To: notmuch@notmuchmail.org\r
  33 Subject: [PATCH 2/7] CLI: remove alias machinery, and "part",\r
  34         "search-tags" commands\r
  35 Date: Sat, 19 Jan 2013 14:25:53 -0400\r
  36 Message-Id: <1358619958-21209-3-git-send-email-david@tethera.net>\r
  37 X-Mailer: git-send-email 1.7.10.4\r
  38 In-Reply-To: <1358619958-21209-1-git-send-email-david@tethera.net>\r
  39 References: <1358619958-21209-1-git-send-email-david@tethera.net>\r
  40 X-Spam_bar: -\r
  41 Cc: David Bremner <bremner@debian.org>\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Sat, 19 Jan 2013 18:26:15 -0000\r
  55 \r
  56 From: David Bremner <bremner@debian.org>\r
  57 \r
  58 The commands are long deprecated, so removal is probably overdue. The\r
  59 real motivation is to simplify argument handling for notmuch so that\r
  60 we can migrate to the common argument parsing framework.\r
  61 ---\r
  62  NEWS       |    8 ++++++++\r
  63  devel/TODO |    7 -------\r
  64  notmuch.c  |   50 +-------------------------------------------------\r
  65  3 files changed, 9 insertions(+), 56 deletions(-)\r
  66 \r
  67 diff --git a/NEWS b/NEWS\r
  68 index 1cb52dd..2ed472e 100644\r
  69 --- a/NEWS\r
  70 +++ b/NEWS\r
  71 @@ -1,3 +1,11 @@\r
  72 +Notmuch 0.16 (2013-MM-DD)\r
  73 +=========================\r
  74 +\r
  75 +Command-Line Interface\r
  76 +----------------------\r
  77 +\r
  78 +Deprecated commands "part" and "search-tags" are removed.\r
  79 +\r
  80  Notmuch 0.15 (2013-01-18)\r
  81  =========================\r
  82  \r
  83 diff --git a/devel/TODO b/devel/TODO\r
  84 index eb757af..3741f0e 100644\r
  85 --- a/devel/TODO\r
  86 +++ b/devel/TODO\r
  87 @@ -118,13 +118,6 @@ file.\r
  88  Allow configuration for filename patterns that should be ignored when\r
  89  indexing.\r
  90  \r
  91 -Replace the "notmuch part --part=id" command with "notmuch show\r
  92 ---part=id", (David Edmondson wants to rewrite some of "notmuch show" to\r
  93 -provide more MIME-structure information in its output first).\r
  94 -\r
  95 -Replace the "notmuch search-tags" command with "notmuch search\r
  96 ---output=tags".\r
  97 -\r
  98  Fix to avoid this ugly message:\r
  99  \r
 100         (process:17197): gmime-CRITICAL **: g_mime_message_get_mime_part: assertion `GMIME_IS_MESSAGE (message)' failed\r
 101 diff --git a/notmuch.c b/notmuch.c\r
 102 index 4fc0973..f13fd27 100644\r
 103 --- a/notmuch.c\r
 104 +++ b/notmuch.c\r
 105 @@ -31,18 +31,6 @@ typedef struct command {\r
 106      const char *summary;\r
 107  } command_t;\r
 108  \r
 109 -#define MAX_ALIAS_SUBSTITUTIONS 3\r
 110 -\r
 111 -typedef struct alias {\r
 112 -    const char *name;\r
 113 -    const char *substitutions[MAX_ALIAS_SUBSTITUTIONS];\r
 114 -} alias_t;\r
 115 -\r
 116 -alias_t aliases[] = {\r
 117 -    { "part", { "show", "--format=raw"}},\r
 118 -    { "search-tags", {"search", "--output=tags", "*"}}\r
 119 -};\r
 120 -\r
 121  static int\r
 122  notmuch_help_command (void *ctx, int argc, char *argv[]);\r
 123  \r
 124 @@ -260,9 +248,7 @@ main (int argc, char *argv[])\r
 125  {\r
 126      void *local;\r
 127      command_t *command;\r
 128 -    alias_t *alias;\r
 129 -    unsigned int i, j;\r
 130 -    const char **argv_local;\r
 131 +    unsigned int i;\r
 132  \r
 133      talloc_enable_null_tracking ();\r
 134  \r
 135 @@ -285,40 +271,6 @@ main (int argc, char *argv[])\r
 136         return 0;\r
 137      }\r
 138  \r
 139 -    for (i = 0; i < ARRAY_SIZE (aliases); i++) {\r
 140 -       alias = &aliases[i];\r
 141 -\r
 142 -       if (strcmp (argv[1], alias->name) == 0)\r
 143 -       {\r
 144 -           int substitutions;\r
 145 -\r
 146 -           argv_local = talloc_size (local, sizeof (char *) *\r
 147 -                                     (argc + MAX_ALIAS_SUBSTITUTIONS - 1));\r
 148 -           if (argv_local == NULL) {\r
 149 -               fprintf (stderr, "Out of memory.\n");\r
 150 -               return 1;\r
 151 -           }\r
 152 -\r
 153 -           /* Copy all substution arguments from the alias. */\r
 154 -           argv_local[0] = argv[0];\r
 155 -           for (j = 0; j < MAX_ALIAS_SUBSTITUTIONS; j++) {\r
 156 -               if (alias->substitutions[j] == NULL)\r
 157 -                   break;\r
 158 -               argv_local[j+1] = alias->substitutions[j];\r
 159 -           }\r
 160 -           substitutions = j;\r
 161 -\r
 162 -           /* And copy all original arguments (skipping the argument\r
 163 -            * that matched the alias of course. */\r
 164 -           for (j = 2; j < (unsigned) argc; j++) {\r
 165 -               argv_local[substitutions+j-1] = argv[j];\r
 166 -           }\r
 167 -\r
 168 -           argc += substitutions - 1;\r
 169 -           argv = (char **) argv_local;\r
 170 -       }\r
 171 -    }\r
 172 -\r
 173      for (i = 0; i < ARRAY_SIZE (commands); i++) {\r
 174         command = &commands[i];\r
 175  \r
 176 -- \r
 177 1.7.10.4\r
 178 \r