4e/801402b3f56ed2c9e5ec06133f8cef722db842

   1 Return-Path: <schnouki@schnouki.net>\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 235C7431FD0\r
   6         for <notmuch@notmuchmail.org>; Wed, 20 Jul 2011 11:13:11 -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.09\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.09 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         T_MIME_NO_TEXT=0.01] autolearn=disabled\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 pEViGbu2L7kX for <notmuch@notmuchmail.org>;\r
  17         Wed, 20 Jul 2011 11:13:09 -0700 (PDT)\r
  18 Received: from ks3536.kimsufi.com (schnouki.net [87.98.217.222])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 61D99431FB6\r
  20         for <notmuch@notmuchmail.org>; Wed, 20 Jul 2011 11:13:09 -0700 (PDT)\r
  21 Received: from odin.local (nancy.schnouki.net [78.238.0.45])\r
  22         by ks3536.kimsufi.com (Postfix) with ESMTPSA id 4D2256A06A9;\r
  23         Wed, 20 Jul 2011 20:21:54 +0200 (CEST)\r
  24 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=schnouki.net;\r
  25         s=key-schnouki; t=1311186114;\r
  26         bh=0y5HG3RAA8EUpYiQtz+QEHxq6TyHUWB35qmUnwovjGE=;\r
  27         h=From:To:Subject:In-Reply-To:References:Date:Message-ID:\r
  28         MIME-Version:Content-Type;\r
  29         b=SjUpu/rmFc2w+zcVb1I110YmB61lq3nKRUXt0/Bfnd6Iw3bLzCdjTZeHuMekM1SeE\r
  30         cOcAcknBOe3YWeJuOxLw5P62a/u9eCsjlGXgq9Hjr+JnaaSqVLEz3h4EtsmjM8K7cP\r
  31         NpEXgoEHiCQMSO7bnGS1sx9hF3qevIzDbIaNSbcE=\r
  32 From: Thomas Jost <schnouki@schnouki.net>\r
  33 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
  34         Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  35 Subject: Re: [PATCH] test: run emacs inside tmux if screen is not available\r
  36 In-Reply-To: <87k4bc275l.fsf@servo.factory.finestructure.net>\r
  37 References: <1309496122-4965-2-git-send-email-dmitry.kurochkin@gmail.com>\r
  38         <1311165396-12268-1-git-send-email-schnouki@schnouki.net>\r
  39         <87d3h5ul7k.fsf@gmail.com>\r
  40         <87k4bc275l.fsf@servo.factory.finestructure.net>\r
  41 User-Agent: Notmuch/0.6.1-84-gc2f6c66 (http://notmuchmail.org) Emacs/23.3.1\r
  42         (x86_64-unknown-linux-gnu)\r
  43 Date: Wed, 20 Jul 2011 20:12:55 +0200\r
  44 Message-ID: <8762mw7rdk.fsf@schnouki.net>\r
  45 MIME-Version: 1.0\r
  46 Content-Type: multipart/signed; boundary="=-=-=";\r
  47         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  48 X-BeenThere: notmuch@notmuchmail.org\r
  49 X-Mailman-Version: 2.1.13\r
  50 Precedence: list\r
  51 List-Id: "Use and development of the notmuch mail system."\r
  52         <notmuch.notmuchmail.org>\r
  53 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  55 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  56 List-Post: <mailto:notmuch@notmuchmail.org>\r
  57 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  58 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  59         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  60 X-List-Received-Date: Wed, 20 Jul 2011 18:13:11 -0000\r
  61 \r
  62 --=-=-=\r
  63 Content-Transfer-Encoding: quoted-printable\r
  64 \r
  65 On Wed, 20 Jul 2011 10:28:38 -0700, Jameson Graef Rollins <jrollins@finestr=\r
  66 ucture.net> wrote:\r
  67 Non-text part: multipart/signed\r
  68 > On Wed, 20 Jul 2011 17:37:35 +0400, Dmitry Kurochkin <dmitry.kurochkin@gm=\r
  69 ail.com> wrote:\r
  70 > > On Wed, 20 Jul 2011 14:36:36 +0200, Thomas Jost <schnouki@schnouki.net>=\r
  71  wrote:\r
  72 > > > Before this change, the test suite reported many failed tests on mach=\r
  73 ines where\r
  74 > > > screen is not installed (which is the case of many *BSD systems). Thi=\r
  75 s patch\r
  76 > > > makes the test suite try to use tmux, another terminal multiplexer, i=\r
  77 f screen is\r
  78 > > > not available.\r
  79 > >=20\r
  80 > > This is a nice improvement.  But I think we should make it even better =\r
  81 :)\r
  82 > > There are more terminal emulators besides screen and tmux (dtach comes\r
  83 > > to mind).  We need a more general mechanism for trying them.  There\r
  84 > > should be a list of commands for running terminal emulators in the order\r
  85 > > of preference (I think that is dtach, tmux, screen), and we should try\r
  86 > > each one in a loop (this would also avoid repeating the long emacs\r
  87 > > command).\r
  88 >=20\r
  89 > Hi, guys.  I suppose you can go down this route if you want, but I worry\r
  90 > that it just adds a lot of extra code to the test suite that will\r
  91 > ultimately make it less robust.  What's wrong with just depending on\r
  92 > screen for the tests?  I'm sure screen is available on every operating\r
  93 > system that we currently support.\r
  94 >=20\r
  95 > It makes me wary that we would be starting a precedent for making a\r
  96 > bunch of conditionals for all of the possible alternative tools we use\r
  97 > in the test suite.  Maybe that's an overblown concern, though.\r
  98 >=20\r
  99 > jamie.\r
 100 \r
 101 You're right, just depending on screen would be enough (the package is\r
 102 just 864 kB on Arch Linux...), and supporting alternative tools\r
 103 would make the test suite more complicated and less robust. But the\r
 104 dependency on screen should be more *explicit*: if it's not available,\r
 105 the test suite should skip all the emacs tests and display a message\r
 106 about screen.\r
 107 \r
 108 Apparently test-lib.sh includes some things about prerequisites. I'll\r
 109 try to use these to implement the explicit dependency on screen.\r
 110 \r
 111 Regards,\r
 112 \r
 113 =2D-=20\r
 114 Thomas/Schnouki\r
 115 \r
 116 --=-=-=\r
 117 Content-Type: application/pgp-signature\r
 118 \r
 119 -----BEGIN PGP SIGNATURE-----\r
 120 Version: GnuPG v1.4.11 (GNU/Linux)\r
 121 \r
 122 iQEcBAEBAgAGBQJOJxqnAAoJEMPdciX+bh5IMyQH/iOqvAMk3LiOT1QYRiW63mSp\r
 123 NRHo1RIdZaDHhahnQdVC2e3pV+bu4j8E5EFvs2xcJw9w1kaSnZcvWHJ14K/B7bL4\r
 124 LlPXMSCV+hgrGvVubSUC3S5usz4N6XAELGoLCb3yKG0RtCtmicBaCnoCgOGvYTA7\r
 125 04pOalWTcAargk1/JvQhlI7pqMpzWRDHQdd6AnPkZtMkmq+VbRpfcIuZ3q8wrFlA\r
 126 Kq4MuNxuRo51SXLAygbOPFQMYXcc7Vz7KxZURWSwrcg/kljxWBJxrlOcXguiXK7y\r
 127 fiWmiOnGMoGc8pVaW+++c4WcH+J4vpCKWzcC+vxQd+t5FyTBGWVuBX8iDBXj1Uw=\r
 128 =e0rz\r
 129 -----END PGP SIGNATURE-----\r
 130 --=-=-=--\r