c5/7ebf7d24cc1b3fa73071747be2e8dd390d43d1

   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 D5D57431FAF\r
   6         for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 05:54:23 -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 TPfSTLgNC-9E for <notmuch@notmuchmail.org>;\r
  16         Wed, 18 Jan 2012 05:54:23 -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 0013E431FAE\r
  21         for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 05:54:22 -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 q0IDs0Ok021887;\r
  24         Wed, 18 Jan 2012 15:54:00 +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 114cs194ts-1\r
  27         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  28         Wed, 18 Jan 2012 15:53:59 +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         q0IDrx6t022606; Wed, 18 Jan 2012 15:53:59 +0200\r
  32 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  33 To: Pieter Praet <pieter@praet.org>,\r
  34         Dmitry Kurochkin <dmitry.kurochkin@gmail.com>,\r
  35         Ethan Glasser-Camp <glasse@cs.rpi.edu>\r
  36 Subject: Re: [PATCH] test: always report missing prereqs,\r
  37         independent of `--verbose' option\r
  38 In-Reply-To: <1326889181-32311-1-git-send-email-pieter@praet.org>\r
  39 References: <87d3aj8rcl.fsf@gmail.com>\r
  40         <1326889181-32311-1-git-send-email-pieter@praet.org>\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: Wed, 18 Jan 2012 15:53:59 +0200\r
  47 Message-ID: <yf64nvtce14.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.5.7110,\r
  51  1.0.211,       0.0.0000        definitions=2012-01-18_06:2012-01-17, 2012-01-18,\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-1201180093\r
  58 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  59 X-BeenThere: notmuch@notmuchmail.org\r
  60 X-Mailman-Version: 2.1.13\r
  61 Precedence: list\r
  62 List-Id: "Use and development of the notmuch mail system."\r
  63         <notmuch.notmuchmail.org>\r
  64 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  66 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  67 List-Post: <mailto:notmuch@notmuchmail.org>\r
  68 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  69 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  71 X-List-Received-Date: Wed, 18 Jan 2012 13:54:24 -0000\r
  72 \r
  73 On Wed, 18 Jan 2012 13:19:41 +0100, Pieter Praet <pieter@praet.org> wrote:\r
  74 > When tests are skipped due to missing prereqs, those prereqs are only\r
  75 > displayed when running with the `--verbose' option.  This is essential\r
  76 > information when troubleshooting, so always send to stdout.\r
  77 > \r
  78 > ---\r
  79 \r
  80 Instead of this the test suite could be enchanged so that\r
  81 full log of the execution is stored somewhere and path\r
  82 to that file is displayed at the end of test. Generally,\r
  83 all output could stay available after tests exit so that\r
  84 those could be investigated furtner.\r
  85 \r
  86 (I tried to grep 'missing prerequisites' after running test\r
  87 but did not find the info. the file opened for >&3 has been\r
  88 deleted already?)\r
  89 \r
  90 Now I just see huge amounts of missing prerequisites: emacs(1)\r
  91 in my output (I presume that doesn't happen due to my\r
  92 hack to "hide" emacs).\r
  93 \r
  94 But well, I'm not against applying this patch -- just that\r
  95 the test system starts to get so rotten touching it gives\r
  96 shivers...\r
  97 \r
  98 Tomi\r
  99 \r
 100 > \r
 101 >  test/test-lib.sh |    4 ++--\r
 102 >  1 files changed, 2 insertions(+), 2 deletions(-)\r
 103 > \r
 104 > diff --git a/test/test-lib.sh b/test/test-lib.sh\r
 105 > index d1fbc05..6560628 100644\r
 106 > --- a/test/test-lib.sh\r
 107 > +++ b/test/test-lib.sh\r
 108 > @@ -673,8 +673,8 @@ test_skip () {\r
 109 >  \r
 110 >  test_check_missing_external_prereqs_ () {\r
 111 >       if test -n "$test_subtest_missing_external_prereqs_"; then\r
 112 > -             say_color skip >&3 "missing prerequisites:"\r
 113 > -             echo "$test_subtest_missing_external_prereqs_" >&3\r
 114 > +             say_color skip >&1 "missing prerequisites:"\r
 115 > +             echo "$test_subtest_missing_external_prereqs_" >&1\r
 116 >               test_report_skip_ "$@"\r
 117 >       else\r
 118 >               false\r
 119 > -- \r
 120 > 1.7.8.1\r