73/b01d4411df98d31c7b739b4a9204561d2621cb

   1 Return-Path: <tomi.ollila@nixu.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 B7ED7429E25\r
   6         for <notmuch@notmuchmail.org>; Thu, 27 Oct 2011 06:52:29 -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\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 zOf1Btg2BQXv for <notmuch@notmuchmail.org>;\r
  16         Thu, 27 Oct 2011 06:52:29 -0700 (PDT)\r
  17 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id E89B4431FB6\r
  21         for <notmuch@notmuchmail.org>; Thu, 27 Oct 2011 06:52:28 -0700 (PDT)\r
  22 Received: from taco2.nixu.fi (localhost [127.0.0.1])\r
  23         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
  24         p9RDqJbt015415\r
  25         (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT);\r
  26         Thu, 27 Oct 2011 16:52:19 +0300\r
  27 Received: (from too@localhost)\r
  28         by taco2.nixu.fi (8.14.3/8.14.3/Submit) id p9RDqJJW015414;\r
  29         Thu, 27 Oct 2011 16:52:19 +0300\r
  30 X-Authentication-Warning: taco2.nixu.fi: too set sender to\r
  31         tomi.ollila@nixu.com using -f\r
  32 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  33 To: Jameson Graef Rollins <jrollins@finestructure.net>\r
  34 Subject: Re: [PATCH] emacs: Modify notmuch-show-get-message-id to return\r
  35         message-id unprefixed with "id:".\r
  36 References: <87mxdabyy8.fsf@washington.ligo-wa.caltech.edu>\r
  37         <1318199748-25058-1-git-send-email-jrollins@finestructure.net>\r
  38         <87ehykflzw.fsf@kepler.schwinge.homeip.net>\r
  39         <87ehxzb0xg.fsf@servo.finestructure.net>\r
  40 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  41         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  42         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  43 Date: Thu, 27 Oct 2011 16:52:19 +0300\r
  44 In-Reply-To: <87ehxzb0xg.fsf@servo.finestructure.net> (Jameson Graef Rollins's\r
  45         message of "Wed, 26 Oct 2011 19:52:59 -0700")\r
  46 Message-ID: <yf6obx25yp8.fsf@taco2.nixu.fi>\r
  47 User-Agent: Gnus/5.110014 (No Gnus v0.14) Emacs/22.2 (gnu/linux)\r
  48 MIME-Version: 1.0\r
  49 Content-Type: text/plain\r
  50 Cc: Notmuch Mail <notmuch@notmuchmail.org>,\r
  51         Thomas Schwinge <thomas@schwinge.name>\r
  52 X-BeenThere: notmuch@notmuchmail.org\r
  53 X-Mailman-Version: 2.1.13\r
  54 Precedence: list\r
  55 List-Id: "Use and development of the notmuch mail system."\r
  56         <notmuch.notmuchmail.org>\r
  57 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  59 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  60 List-Post: <mailto:notmuch@notmuchmail.org>\r
  61 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  62 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  64 X-List-Received-Date: Thu, 27 Oct 2011 13:52:29 -0000\r
  65 \r
  66 On Thu 27 Oct 2011 05:52, Jameson Graef Rollins <jrollins@finestructure.net> writes:\r
  67 \r
  68 > On Tue, 11 Oct 2011 01:46:43 +0200, Thomas Schwinge\r
  69 > <thomas@schwinge.name> wrote:\r
  70 >> On Sun, 9 Oct 2011 15:35:48 -0700, Jameson Graef Rollins\r
  71 >> <jrollins@finestructure.net> wrote:\r
  72 >> >  (defun notmuch-show-get-message-id ()\r
  73 >> >    "Return the message id of the current message."\r
  74 >> > -  (concat "id:\"" (notmuch-show-get-prop :id) "\""))\r
  75 >> > +  (concat "\"" (notmuch-show-get-prop :id) "\""))\r
  76 >> \r
  77 >> Shouldn't the double quotes be removed here, too?  (And be re-added in\r
  78 >> the other places where id: is added.)\r
  79 >\r
  80 > In point of fact, do we need these quotes around message IDs at all?\r
  81 > Can message IDs have characters that would need to be escaped for the\r
  82 > shell?\r
  83 \r
  84 Is there intermediate shell run when emacs run these commands, if not,\r
  85 then shell expansion isn't happen. If there is, then the "simplest" way\r
  86 to avoid shell expansion is to put strings in '' -quotes and replace\r
  87 all intermediate ':s with '\'' (meaning single ' expands to ''\''' ;)\r
  88 \r
  89 >\r
  90 > This seems to also be related to the issue brought up by Erlend Simonsen\r
  91 > in [0], where emacs 24 seems to be shell-quoting things as needed.\r
  92 >\r
  93 > So maybe these quotes aren't actually needed at all?...\r
  94 \r
  95 These quotes are needed if emacs uses intermediate shell and emacs \r
  96 doesn't quote itself. if there is functionality change between emacs\r
  97 23 and emacs 24 here, then that's interesting.\r
  98 \r
  99 >\r
 100 > jamie.\r
 101 >\r
 102 \r
 103 Tomi\r