4f/2affa28b653deda837c822cc354cc24d8185d1

   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 57B6C431FD0\r
   6         for <notmuch@notmuchmail.org>; Fri, 11 Nov 2011 00:09:29 -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 69qWO+h7tBwk for <notmuch@notmuchmail.org>;\r
  16         Fri, 11 Nov 2011 00:09:28 -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 2C878431FB6\r
  21         for <notmuch@notmuchmail.org>; Fri, 11 Nov 2011 00:09:27 -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         pAB89HW7003584; Fri, 11 Nov 2011 10:09:18 +0200\r
  25 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  26 To: Pieter Praet <pieter@praet.org>,\r
  27         Jameson Graef Rollins <jrollins@finestructure.net>,\r
  28         Notmuch Mail <notmuch@notmuchmail.org>\r
  29 Subject: Re: [PATCH] test: use dtach(1) instead of screen(1) in emacs tests.\r
  30 In-Reply-To: <87hb2btsv2.fsf@praet.org>\r
  31 References: <yf6mxc3ab4t.fsf@taco2.nixu.fi>\r
  32         <1320963737-1666-1-git-send-email-jrollins@finestructure.net>\r
  33         <87hb2btsv2.fsf@praet.org>\r
  34 User-Agent: Notmuch/0.9+23~g556c5fa (http://notmuchmail.org) Emacs/23.3.1\r
  35         (i686-pc-linux-gnu)\r
  36 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  37         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  38         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  39 Date: Fri, 11 Nov 2011 10:09:17 +0200\r
  40 Message-ID: <yf6ipmrdqs2.fsf@taco2.nixu.fi>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: text/plain; charset=us-ascii\r
  43 X-BeenThere: notmuch@notmuchmail.org\r
  44 X-Mailman-Version: 2.1.13\r
  45 Precedence: list\r
  46 List-Id: "Use and development of the notmuch mail system."\r
  47         <notmuch.notmuchmail.org>\r
  48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  51 List-Post: <mailto:notmuch@notmuchmail.org>\r
  52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  55 X-List-Received-Date: Fri, 11 Nov 2011 08:09:29 -0000\r
  56 \r
  57 On Fri, 11 Nov 2011 01:17:37 +0100, Pieter Praet <pieter@praet.org> wrote:\r
  58 > On Thu, 10 Nov 2011 14:22:17 -0800, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  59 > > From: Tomi Ollila <tomi.ollila@iki.fi>\r
  60 > > \r
  61 > > dtach is lighter than screen and is not setuid/setgid program so\r
  62 > > TMPDIR does not get reset by dynamic loader when executed.\r
  63 > > \r
  64 > \r
  65 > `dtach' may be lighter than `screen', but contrary to my expectations,\r
  66 > it appears to be far less efficient:\r
  67 > \r
  68 > |      | original [1] | w/ screen [2] | w/ dtach [3] |\r
  69 > |------+--------------+---------------+--------------|\r
  70 > | real |    0m17.570s |     0m20.132s |    0m28.567s |\r
  71 > | user |     0m2.453s |      0m2.593s |     0m2.753s |\r
  72 > | sys  |     0m1.700s |      0m1.577s |     0m1.753s |\r
  73 \r
  74 This sounds interesting; how were these runs measured?\r
  75 Just using shell builtin time command ?\r
  76 \r
  77 Does time measure the time of the process and it's subprocesses\r
  78 or just the process. In any way 20 vs 28 second 'real' time difference\r
  79 could indicate that there are more other processes running and\r
  80 the context switch time affects user and system time.\r
  81 \r
  82 Or, dtach does some stupid io things. Or screen does some advanced\r
  83 optimizations. Or emacs works more effectively when TERM=screen.\r
  84 Or the 'virtual' window size emacs sees is unsuitable...\r
  85 \r
  86 \r
  87 > \r
  88 > > Signed-off-by: Jameson Graef Rollins <jrollins@finestructure.net>\r
  89 > > ---\r
  90 > > This tweaks the original patch slightly by removing the no-longer needed\r
  91 > > screenrc variables.\r
  92 > > \r
  93 > >  test/test-lib.sh |    5 ++---\r
  94 > >  1 files changed, 2 insertions(+), 3 deletions(-)\r
  95 > > \r
  96 > > diff --git a/test/test-lib.sh b/test/test-lib.sh\r
  97 > > index c81c709..c0fe037 100755\r
  98 > > --- a/test/test-lib.sh\r
  99 > > +++ b/test/test-lib.sh\r
 100 > > @@ -50,8 +50,6 @@ TZ=UTC\r
 101 > >  TERM=dumb\r
 102 > >  export LANG LC_ALL PAGER TERM TZ\r
 103 > >  GIT_TEST_CMP=${GIT_TEST_CMP:-diff -u}\r
 104 > > -export SCREENRC=/dev/null\r
 105 > > -export SYSSCREENRC=/dev/null\r
 106 > >  \r
 107 > >  # Protect ourselves from common misconfiguration to export\r
 108 > >  # CDPATH into the environment\r
 109 > > @@ -844,7 +842,8 @@ test_emacs () {\r
 110 > >     if [ -z "$EMACS_SERVER" ]; then\r
 111 > >             EMACS_SERVER="notmuch-test-suite-$$"\r
 112 > >             # start a detached screen session with an emacs server\r
 113 > > -           screen -S "$EMACS_SERVER" -d -m "$TMP_DIRECTORY/run_emacs" \\r
 114 > > +           TERM=xterm dtach -n "$TMP_DIRECTORY/dtach-emacs-socket-$$" \\r
 115 > \r
 116 > It's worth noting that $TERM is only set to "xterm" to deceive `dtach',\r
 117 > so thankfully, xterm (and all its deps) aren't actually required to run\r
 118 > the tests.\r
 119 \r
 120 Actually, emacs SIGSEGVs if there is unsuitable TERM in use. I don't know\r
 121 what it originally was.\r
 122 \r
 123 > \r
 124 > > +                   "$TMP_DIRECTORY/run_emacs" \\r
 125 > >                     --no-window-system \\r
 126 > >                     --eval "(setq server-name \"$EMACS_SERVER\")" \\r
 127 > >                     --eval '(server-start)' \\r
 128 > > -- \r
 129 > > 1.7.7.1\r
 130 > > \r
 131 > > _______________________________________________\r
 132 > > notmuch mailing list\r
 133 > > notmuch@notmuchmail.org\r
 134 > > http://notmuchmail.org/mailman/listinfo/notmuch\r
 135 > \r
 136 > \r
 137 > Peace\r
 138 > \r
 139 > -- \r
 140 > Pieter\r
 141 > \r
 142 > [1] @ commit 749abb74\r
 143 > [2] @ commit 760b311b\r
 144 > [3] @ commit 760b311b + id:"1320963737-1666-1-git-send-email-jrollins@finestructure.net"\r
 145 > _______________________________________________\r
 146 > notmuch mailing list\r
 147 > notmuch@notmuchmail.org\r
 148 > http://notmuchmail.org/mailman/listinfo/notmuch\r
 149 > \r
 150 \r
 151 Tomi\r