86/eb0b9721ce628fd1197060a4bdd23131faa8ed

   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 3AAFC431FAF\r
   6         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 02:32:31 -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 pOTx5xtNI6xP for <notmuch@notmuchmail.org>;\r
  16         Thu, 19 Jan 2012 02:32:30 -0800 (PST)\r
  17 Received: from mail-gw3.nixu.fi (mail-gw3.nixu.fi [193.209.237.7])\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 5EE63431FAE\r
  21         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 02:32:30 -0800 (PST)\r
  22 Received: from pps.filterd (mail-gw3 [127.0.0.1])\r
  23         by mail-gw3.nixu.fi (8.14.4/8.14.4) with SMTP id q0JAJDFI019186;\r
  24         Thu, 19 Jan 2012 12:32:23 +0200\r
  25 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  26         by mail-gw3.nixu.fi with ESMTP id 114cs19sj4-1\r
  27         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  28         Thu, 19 Jan 2012 12:32:23 +0200\r
  29 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  30         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
  31         q0JAWLNI026732; Thu, 19 Jan 2012 12:32:23 +0200\r
  32 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  33 To: David Edmondson <dme@dme.org>, <notmuch@notmuchmail.org>\r
  34 Subject: Re: [PATCH 2/3] test: Add `test_emacs_expect_t'.\r
  35 In-Reply-To: <cunzkdk57yj.fsf@hotblack-desiato.hh.sledj.net>\r
  36 References: <87pqei327k.fsf@gmail.com>\r
  37         <1326809224-22673-1-git-send-email-dme@dme.org>\r
  38         <1326809224-22673-2-git-send-email-dme@dme.org>\r
  39         <yf6zkdlawlc.fsf@taco2.nixu.fi>\r
  40         <cunzkdk57yj.fsf@hotblack-desiato.hh.sledj.net>\r
  41 User-Agent: Notmuch/0.11+71~gefa5d6c (http://notmuchmail.org) Emacs/23.3.1\r
  42         (i686-pc-linux-gnu)\r
  43 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  44         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  45         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  46 Date: Thu, 19 Jan 2012 12:32:21 +0200\r
  47 Message-ID: <yf662g8asp6.fsf@taco2.nixu.fi>\r
  48 MIME-Version: 1.0\r
  49 Content-Type: text/plain; charset=us-ascii\r
  50 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.6.7361,\r
  51  1.0.211,       0.0.0000        definitions=2012-01-19_05:2012-01-19, 2012-01-19,\r
  52         1970-01-01 signatures=0\r
  53 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0\r
  54         ipscore=0 suspectscore=0\r
  55         phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0\r
  56         reason=mlx\r
  57         scancount=1 engine=6.0.2-1012030000 definitions=main-1201190034\r
  58 X-BeenThere: notmuch@notmuchmail.org\r
  59 X-Mailman-Version: 2.1.13\r
  60 Precedence: list\r
  61 List-Id: "Use and development of the notmuch mail system."\r
  62         <notmuch.notmuchmail.org>\r
  63 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  65 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  66 List-Post: <mailto:notmuch@notmuchmail.org>\r
  67 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  68 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  70 X-List-Received-Date: Thu, 19 Jan 2012 10:32:31 -0000\r
  71 \r
  72 On Thu, 19 Jan 2012 09:59:16 +0000, David Edmondson <dme@dme.org> wrote:\r
  73 > On Wed, 18 Jan 2012 16:55:59 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:\r
  74 > > > + # Report success/failure.\r
  75 > > > + if ! test_skip "$test_subtest_name"\r
  76 > > > + then\r
  77 > > > +         if [ "$result" == t ]\r
  78 > > \r
  79 > >             if [ "$result" = t ]\r
  80 > > \r
  81 > > to be compatible with POSIX and consistent with rest code.\r
  82 > \r
  83 > I'm happy to change this.\r
  84 \r
  85 As your older patch has been marked obsolete and this really is the\r
  86 only place where there is == in comparison code please do that change.\r
  87 \r
  88 > > > +                 test_failure_ "$test_subtest_name" "$(eval printf ${result})"\r
  89 > > \r
  90 > > This added 'eval' made me investigate further... running \r
  91 > > \r
  92 > > emacsclient --eval '(print (concat "a" "b" "\t" "c" "\n" "z"))'\r
  93 > > \r
  94 > > outputs "ab c\nz" (tab between 'ab' and 'c', quotes (") around\r
  95 > > the whole output and newlines as "\n" (even '\r' is converted)).\r
  96 > > \r
  97 > > If emacs tests run via test_emacs_expect_t wrote their output \r
  98 > > to ${output} directly above code could be much cleaner in many\r
  99 > > places. Environment variable could be used for the file name.\r
 100 > \r
 101 > I'm reluctant to change this just to avoid calling eval.\r
 102 \r
 103 Consider the following:\r
 104 \r
 105 $ emacsclient --eval '(print "$(echo rm -rf /); echo `date +%Y`")'\r
 106 "$(echo rm -rf /); echo `date +%Y`"\r
 107 \r
 108 x='"$(echo rm -rf /); echo `date +%Y`"'\r
 109 \r
 110 $echo $x\r
 111 "$(echo rm -rf /); echo `date +%Y`"\r
 112 \r
 113 $echo $(eval printf $x)\r
 114 rm -rf /; echo 2012\r
 115 \r
 116 x='"$(echo rm /); echo `date +%Y`"'\r
 117 \r
 118 $(eval printf $x)\r
 119 rm: cannot remove `/;': No such file or directory\r
 120 rm: cannot remove `2012': No such file or directory\r
 121 \r
 122 one just needs to make sure there is no $:s and `: in \r
 123 the output... hmm, nor ';' ...NOR '&':s ...\r
 124 \r
 125 I am not absolutely sure this might actually happen\r
 126 but surely I'm not sure it would not...\r
 127 \r
 128 ... The suggestion having environment variable would\r
 129 now work, but what about\r
 130 \r
 131 test_emacs "(setq test-output-file \"${output}\") $1"\r
 132 \r
 133 \r
 134 Tomi\r
 135 \r
 136 PS:\r
 137 \r
 138 this needs to be changed:\r
 139         emacsclient --socket-name="$EMACS_SERVER" --eval "(progn $@)"\r
 140 to\r
 141         emacsclient --socket-name="$EMACS_SERVER" --eval "(progn $*)"\r
 142 then\r
 143         test_emacs   "(setq test-output-file \"${output}\")"   "$1"\r
 144 \r
 145 would be safe\r