7b/60b2d9e346b4588bfdb903cee38a75a1b153a0

   1 Return-Path: <cworth@cworth.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 CF3D94196F0\r
   6         for <notmuch@notmuchmail.org>; Tue, 27 Apr 2010 08:29:04 -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: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\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 2jgtsGZqhBbd; Tue, 27 Apr 2010 08:29:04 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 13DC6431FC1;\r
  19         Tue, 27 Apr 2010 08:29:04 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id AFE2C568DEB; Tue, 27 Apr 2010 08:29:03 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: Jesse Rosenthal <jrosenthal@jhu.edu>,\r
  24         Notmuch developer list <notmuch@notmuchmail.org>\r
  25 Subject: Re: [PATCH 1/2] emacs: Add auto-tagging for replied messages.\r
  26 In-Reply-To: <87d3xlkpsc.fsf@jhu.edu>\r
  27 References: <87d3xlkpsc.fsf@jhu.edu>\r
  28 User-Agent: Notmuch/0.2-251-g8514e59 (http://notmuchmail.org) Emacs/23.1.1\r
  29         (i486-pc-linux-gnu)\r
  30 Date: Tue, 27 Apr 2010 08:29:03 -0700\r
  31 Message-ID: <87fx2gnccg.fsf@yoom.home.cworth.org>\r
  32 MIME-Version: 1.0\r
  33 Content-Type: multipart/signed; boundary="=-=-=";\r
  34         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  35 X-BeenThere: notmuch@notmuchmail.org\r
  36 X-Mailman-Version: 2.1.13\r
  37 Precedence: list\r
  38 List-Id: "Use and development of the notmuch mail system."\r
  39         <notmuch.notmuchmail.org>\r
  40 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  41         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  42 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  43 List-Post: <mailto:notmuch@notmuchmail.org>\r
  44 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  45 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  47 X-List-Received-Date: Tue, 27 Apr 2010 15:29:05 -0000\r
  48 \r
  49 --=-=-=\r
  50 Content-Transfer-Encoding: quoted-printable\r
  51 \r
  52 On Tue, 27 Apr 2010 09:06:59 -0400, Jesse Rosenthal <jrosenthal@jhu.edu> wr=\r
  53 ote:\r
  54 > Add `notmuch-message-mark-replied', a function for automatically tagging\r
  55 > replied messages with user-defined tags.\r
  56 \r
  57 A nice, feature. Thanks!\r
  58 \r
  59 A couple of points on the patch:\r
  60 \r
  61 > +(defcustom notmuch-message-replied-tags '("replied")\r
  62 > +  "A set of tags to be added or, if prefaced with a `-', removed. For=20\r
  63 > +example, \(\"replied\" \"-inbox\" \"-todo\"\)"\r
  64 \r
  65 This documentation string needs to be improved a bit. Look in customize\r
  66 and see if only the first line appears by default. If so, we don't want\r
  67 a widowed "For" at the end of the line.\r
  68 \r
  69 Also, this documentation string doesn't actually give a new user enough\r
  70 information to learn what this actually does.\r
  71 \r
  72 Should say something like:\r
  73 \r
  74 "Tags to be added/removed from messages when sending a reply to that messag=\r
  75 e."\r
  76 \r
  77 as the first sentence. That tells the user what the feature does and\r
  78 whether it's interesting. If so, the user can expand the documentation\r
  79 and subsequent sentences can explain things like '-', etc.\r
  80 \r
  81 Finally, since this is a new feature it needs to add a new blurb to the\r
  82 NEWS file.\r
  83 \r
  84 Thanks,\r
  85 \r
  86 =2DCarl\r
  87 \r
  88 --=-=-=\r
  89 Content-Type: application/pgp-signature\r
  90 \r
  91 -----BEGIN PGP SIGNATURE-----\r
  92 Version: GnuPG v1.4.10 (GNU/Linux)\r
  93 \r
  94 iD8DBQFL1wK/6JDdNq8qSWgRAvR8AJwPhGh2Js5N6wu4ydvopQhCnKemqwCfcdQc\r
  95 cBc5vnzMpb7IKOUJFSDhd4s=\r
  96 =UWhW\r
  97 -----END PGP SIGNATURE-----\r
  98 --=-=-=--\r