30/321af7bf16eae227fd239818acf4054834833d

   1 Return-Path: <dmitry.kurochkin@gmail.com>\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 15906431FB6\r
   6         for <notmuch@notmuchmail.org>; Sat, 28 Jan 2012 01:50:42 -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.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id bg++2gGWqGFi for <notmuch@notmuchmail.org>;\r
  17         Sat, 28 Jan 2012 01:50:38 -0800 (PST)\r
  18 Received: from mail-bk0-f53.google.com (mail-bk0-f53.google.com\r
  19         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id EAAF5431FAE\r
  22         for <notmuch@notmuchmail.org>; Sat, 28 Jan 2012 01:50:37 -0800 (PST)\r
  23 Received: by bkbzt19 with SMTP id zt19so2384438bkb.26\r
  24         for <notmuch@notmuchmail.org>; Sat, 28 Jan 2012 01:50:36 -0800 (PST)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
  27         :message-id:mime-version:content-type;\r
  28         bh=tp69KnoQYVYlTdiwQp8i76mXUPK33h48+NEFE8hSlJA=;\r
  29         b=hPzgB7dESbplhIp1Nyj4U7r+36ZMZyA7Q2cfsnb6s1DVgRVu14eMoChFqYtI76BHxd\r
  30         FaRmYRDxIo64ZISocaluPVq0y8eHg3yELmshvQurcnd27kqA5yZbppID9cwr8R/Zsof5\r
  31         EAeCwAY0wT+gq5ciTJd4kDNzGeC6ey9SwHHZI=\r
  32 Received: by 10.204.129.220 with SMTP id p28mr4884693bks.8.1327744236644;\r
  33         Sat, 28 Jan 2012 01:50:36 -0800 (PST)\r
  34 Received: from localhost ([91.144.186.21])\r
  35         by mx.google.com with ESMTPS id cg2sm22178901bkb.12.2012.01.28.01.50.35\r
  36         (version=TLSv1/SSLv3 cipher=OTHER);\r
  37         Sat, 28 Jan 2012 01:50:35 -0800 (PST)\r
  38 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  39 To: Jani Nikula <jani@nikula.org>\r
  40 Subject: Re: [PATCH 7/6] emacs: relax tag syntax check in `notmuch-tag'\r
  41         function\r
  42 In-Reply-To:\r
  43  <CAB+hUn834oJ+XGx-YyYSGxSnzrBYCMvcu4Vd73ws28qTS2riuA@mail.gmail.com>\r
  44 References: <1327725684-5887-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  45         <1327727143-24455-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  46         <CAB+hUn834oJ+XGx-YyYSGxSnzrBYCMvcu4Vd73ws28qTS2riuA@mail.gmail.com>\r
  47 User-Agent: Notmuch/0.11+134~g7ddba9d (http://notmuchmail.org) Emacs/23.3.1\r
  48         (x86_64-pc-linux-gnu)\r
  49 Date: Sat, 28 Jan 2012 13:49:27 +0400\r
  50 Message-ID: <87ty3g88d4.fsf@gmail.com>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain; charset=us-ascii\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, 28 Jan 2012 09:50:42 -0000\r
  67 \r
  68 On Sat, 28 Jan 2012 10:56:21 +0200, Jani Nikula <jani@nikula.org> wrote:\r
  69 > On Jan 28, 2012 7:06 AM, "Dmitry Kurochkin" <dmitry.kurochkin@gmail.com>\r
  70 > wrote:\r
  71 > >\r
  72 > > The tag syntax check in `notmuch-tag' function was too strict and did\r
  73 > > not allow nmbug tags with "::".  Since the check is done for all\r
  74 > > tagging operations in Emacs UI, this basically means that no nmbug\r
  75 > > tags can be changed.  The patch relaxes the tag syntax check to allow\r
  76 > > any tag names that do not include whitespace characters.\r
  77 > \r
  78 > Imho the syntax check should be in cli, or lib even. I posted a patch to\r
  79 > cli some time ago when I realized it's possible to add tag "-" but you\r
  80 > can't remove it with the current cli. (On the road, can't find the message\r
  81 > id now.)\r
  82 > \r
  83 \r
  84 I agree that this is an issue (a general issue for notmuch, not just\r
  85 Emacs UI).  But it is outside of scope of this patch.  This patch\r
  86 purpose is just to solve an immediate issue with a broken regexp.\r
  87 \r
  88 Regards,\r
  89   Dmitry\r
  90 \r
  91 > > ---\r
  92 > >  emacs/notmuch.el |    2 +-\r
  93 > >  1 files changed, 1 insertions(+), 1 deletions(-)\r
  94 > >\r
  95 > > diff --git a/emacs/notmuch.el b/emacs/notmuch.el\r
  96 > > index 9813e0a..0de6123 100644\r
  97 > > --- a/emacs/notmuch.el\r
  98 > > +++ b/emacs/notmuch.el\r
  99 > > @@ -555,7 +555,7 @@ notmuch-after-tag-hook will be run."\r
 100 > >   ;; Perform some validation\r
 101 > >   (when (null tags) (error "No tags given"))\r
 102 > >   (mapc (lambda (tag)\r
 103 > > -         (unless (string-match-p "^[-+][-+_.[:word:]]+$" tag)\r
 104 > > +         (unless (string-match-p "^[-+]\\S-+$" tag)\r
 105 > >            (error "Tag must be of the form `+this_tag' or `-that_tag'")))\r
 106 > >        tags)\r
 107 > >   (run-hooks 'notmuch-before-tag-hook)\r
 108 > > --\r
 109 > > 1.7.8.3\r
 110 > >\r
 111 > > _______________________________________________\r
 112 > > notmuch mailing list\r
 113 > > notmuch@notmuchmail.org\r
 114 > > http://notmuchmail.org/mailman/listinfo/notmuch\r
 115 Non-text part: text/html\r