9e/5b49f013f1a3706e817226803fe696cf499a19

   1 Return-Path: <tomi.ollila@iki.fi>\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 arlo.cworth.org (Postfix) with ESMTP id E48866DE1772\r
   6  for <notmuch@notmuchmail.org>; Mon, 28 Dec 2015 14:29:05 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.686\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.686 tagged_above=-999 required=5 tests=[AWL=0.034, \r
  12  SPF_NEUTRAL=0.652] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id RrWdihuCe3qW for <notmuch@notmuchmail.org>;\r
  16  Mon, 28 Dec 2015 14:29:03 -0800 (PST)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18  by arlo.cworth.org (Postfix) with ESMTP id 111B16DE176C\r
  19  for <notmuch@notmuchmail.org>; Mon, 28 Dec 2015 14:29:02 -0800 (PST)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21  by guru.guru-group.fi (Postfix) with ESMTP id 3E88E10008E;\r
  22  Tue, 29 Dec 2015 00:29:20 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Michal Sojka <sojkam1@fel.cvut.cz>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH] test: Unset ALTERNATE_EDITOR before running emacsclient\r
  26 In-Reply-To: <1451337395-14196-1-git-send-email-sojkam1@fel.cvut.cz>\r
  27 References: <1451337395-14196-1-git-send-email-sojkam1@fel.cvut.cz>\r
  28 User-Agent: Notmuch/0.21+32~g73439f8 (http://notmuchmail.org) Emacs/24.3.1\r
  29  (x86_64-unknown-linux-gnu)\r
  30 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  31  $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  32  !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  33 Date: Tue, 29 Dec 2015 00:29:19 +0200\r
  34 Message-ID: <m28u4e2on4.fsf@guru.guru-group.fi>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.20\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41  <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  43  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Mon, 28 Dec 2015 22:29:06 -0000\r
  50 \r
  51 On Mon, Dec 28 2015, Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
  52 \r
  53 > ALTERNATE_EDITOR causes emacsclient to run an alternate editor if the\r
  54 > emacs server is not ready. This can collide with intended\r
  55 > functionality in test-lib.sh.\r
  56 >\r
  57 > If the ALTERNATE_EDITOR is set but empty, emacsclient runs emacs\r
  58 > daemon and tries to connect to it. When this happens the emacs run by\r
  59 > test-lib.sh fails to start the server and the subsequent attempts to\r
  60 > use the server fail because the daemon started by emacsclient does not\r
  61 > know about notmuch-test-progn. This leads to test suite failure due to\r
  62 > time out on any emacs test.\r
  63 > ---\r
  64 \r
  65 Looks good to me -- just that should the unsetting be done in the same\r
  66 place as (most) other environment variables are handled... in the\r
  67 region starting from line 64 of that same file.\r
  68 \r
  69 Tomi\r
  70 \r
  71 \r
  72 >  test/test-lib.sh | 1 +\r
  73 >  1 file changed, 1 insertion(+)\r
  74 >\r
  75 > diff --git a/test/test-lib.sh b/test/test-lib.sh\r
  76 > index 126911f..0f6a6cf 100644\r
  77 > --- a/test/test-lib.sh\r
  78 > +++ b/test/test-lib.sh\r
  79 > @@ -1152,6 +1152,7 @@ test_emacs () {\r
  80 >       rm -f OUTPUT\r
  81 >       touch OUTPUT\r
  82 >  \r
  83 > +     unset ALTERNATE_EDITOR\r
  84 >       ${TEST_EMACSCLIENT} --socket-name="$EMACS_SERVER" --eval "(notmuch-test-progn $@)"\r
  85 >  }\r
  86 >  \r
  87 > -- \r
  88 > 2.6.4\r
  89 >\r
  90 > _______________________________________________\r
  91 > notmuch mailing list\r
  92 > notmuch@notmuchmail.org\r
  93 > https://notmuchmail.org/mailman/listinfo/notmuch\r