b2/6d6a64583881b39044948b77a59e315747ed14

   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 8626B429E21\r
   6         for <notmuch@notmuchmail.org>; Thu, 17 Nov 2011 01:43:43 -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 p0VAmz9VGSQ4 for <notmuch@notmuchmail.org>;\r
  16         Thu, 17 Nov 2011 01:43:41 -0800 (PST)\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 E685A431FD0\r
  21         for <notmuch@notmuchmail.org>; Thu, 17 Nov 2011 01:43:40 -0800 (PST)\r
  22 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  23         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
  24         pAH9ha5w008495; Thu, 17 Nov 2011 11:43:37 +0200\r
  25 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  26 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  27 Subject: Re: [PATCH 8/9] test: check if emacs is available in the beginning of\r
  28         test_emacs\r
  29 In-Reply-To: <1321494986-18998-9-git-send-email-dmitry.kurochkin@gmail.com>\r
  30 References: <1321494986-18998-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  31         <1321494986-18998-9-git-send-email-dmitry.kurochkin@gmail.com>\r
  32 User-Agent: Notmuch/0.9+73~ged20210 (http://notmuchmail.org) Emacs/23.3.1\r
  33         (i686-pc-linux-gnu)\r
  34 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  35         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  36         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  37 Date: Thu, 17 Nov 2011 11:43:36 +0200\r
  38 Message-ID: <yf67h2zgk3b.fsf@taco2.nixu.fi>\r
  39 MIME-Version: 1.0\r
  40 Content-Type: text/plain; charset=us-ascii\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.13\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Thu, 17 Nov 2011 09:43:43 -0000\r
  54 \r
  55 On Thu, 17 Nov 2011 05:56:25 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  56 > Unfortunately, this is needed to avoid the emacs waiting loop.\r
  57 > ---\r
  58 \r
  59 >  test/test-lib.sh |    4 ++++\r
  60 >  1 files changed, 4 insertions(+), 0 deletions(-)\r
  61 \r
  62 [ context reduced ]\r
  63 \r
  64 > diff --git a/test/test-lib.sh b/test/test-lib.sh\r
  65 > index 840c86c..5bd5bd6 100755\r
  66 > --- a/test/test-lib.sh\r
  67 > +++ b/test/test-lib.sh\r
  68 \r
  69 I have to patch the full set to my clone of the repo and do \r
  70 fuller review, but until that...\r
  71 \r
  72 >  test_emacs () {\r
  73 > +     # test dependencies beforehand to avoid the waiting loop below\r
  74 > +     which emacs >/dev/null || emacs || return\r
  75 > +     which emacsclient >/dev/null || emacsclient || return\r
  76 > +\r
  77 \r
  78 If emacs not found using which (what happened to hash), then try\r
  79 to execute emacs (??) and if that fails return.\r
  80 \r
  81 Same thing with emacsclient.\r
  82 \r
  83 In case of emacs: DISPLAY set, TERM=dumb -- emacs starts interactively\r
  84 on my desktop; grep DISPLAY test/* yields nothing. (maybe we should unset\r
  85 DISPLAY for tests ?). And, if DISPLAY unset, emacs exits with nonzero\r
  86 value, making || return happen anyway.\r
  87 \r
  88 ... and this is tested every time test_emacs() is executed...\r
  89 \r
  90 >       if [ -z "$EMACS_SERVER" ]; then\r
  91 >               server_name="notmuch-test-suite-$$"\r
  92 > -- \r
  93 > 1.7.7.2\r
  94 \r
  95 Tomi\r