f5/7b479f3ac66e5433da21464e34a6367889cc79

   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 2720F429E26\r
   6         for <notmuch@notmuchmail.org>; Tue, 13 Dec 2011 12:30:17 -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 t7E+l-fyqySY for <notmuch@notmuchmail.org>;\r
  16         Tue, 13 Dec 2011 12:30:15 -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 1ACAD429E25\r
  21         for <notmuch@notmuchmail.org>; Tue, 13 Dec 2011 12:30:14 -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 pBDKU2uZ031736;\r
  24         Tue, 13 Dec 2011 22:30:12 +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 114cs0t4k3-1\r
  27         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  28         Tue, 13 Dec 2011 22:30:12 +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         pBDKUCuA024711; Tue, 13 Dec 2011 22:30:12 +0200\r
  32 From: Tomi Ollila <tomi.ollila@nixu.com>\r
  33 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH] test: optionally print subtest number\r
  35 In-Reply-To: <1323806207-31888-1-git-send-email-david@tethera.net>\r
  36 References: <1323806207-31888-1-git-send-email-david@tethera.net>\r
  37 User-Agent: Notmuch/0.10.2+93~g631d290 (http://notmuchmail.org) Emacs/23.3.1\r
  38         (i686-pc-linux-gnu)\r
  39 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  40         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  41         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  42 Date: Tue, 13 Dec 2011 22:30:11 +0200\r
  43 Message-ID: <yf662hkrz8s.fsf@taco2.nixu.fi>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: text/plain; charset=us-ascii\r
  46 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.5.7110,\r
  47  1.0.211,       0.0.0000        definitions=2011-12-13_03:2011-12-13, 2011-12-13,\r
  48         1970-01-01 signatures=0\r
  49 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0\r
  50         ipscore=0 suspectscore=0\r
  51         phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0\r
  52         reason=mlx\r
  53         scancount=1 engine=6.0.2-1012030000 definitions=main-1112130217\r
  54 X-BeenThere: notmuch@notmuchmail.org\r
  55 X-Mailman-Version: 2.1.13\r
  56 Precedence: list\r
  57 List-Id: "Use and development of the notmuch mail system."\r
  58         <notmuch.notmuchmail.org>\r
  59 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  61 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  62 List-Post: <mailto:notmuch@notmuchmail.org>\r
  63 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  64 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  66 X-List-Received-Date: Tue, 13 Dec 2011 20:30:17 -0000\r
  67 \r
  68 On Tue, 13 Dec 2011 15:56:47 -0400, David Bremner <david@tethera.net> wrote:\r
  69 > From: David Bremner <bremner@debian.org>\r
  70 > \r
  71 > The idea is that $test_count could be used in tests to label\r
  72 > intermediate files. The output enabled by this patch (and --debug)\r
  73 > helps figure out which OUTPUT.nn file belongs to which test in case\r
  74 > several subtests write to OUTPUT.$test_count\r
  75 > ---\r
  76 > \r
  77 > Is there something that depends on the test format? I find it pretty\r
  78 > handy to have the subtest numbers, but I don't want to break some\r
  79 > other tools. I followed the existing style of conditionally defining\r
  80 > functions, but maybe someone with more bash-fu can improve that.\r
  81 \r
  82 +1. I would also say '+1' even non-debug format (and update dependend code,\r
  83 if there is any).\r
  84 \r
  85 Tomi\r
  86 \r
  87 > \r
  88 >  test/test-lib.sh |   12 ++++++++++++\r
  89 >  1 files changed, 12 insertions(+), 0 deletions(-)\r
  90 > \r
  91 > diff --git a/test/test-lib.sh b/test/test-lib.sh\r
  92 > index 6be93fe..aaaaf5c 100644\r
  93 > --- a/test/test-lib.sh\r
  94 > +++ b/test/test-lib.sh\r
  95 > @@ -116,6 +116,16 @@ do\r
  96 >       esac\r
  97 >  done\r
  98 >  \r
  99 > +if test -n "$debug"; then\r
 100 > +    print_subtest () {\r
 101 > +     printf "\t[%d]\t" $(($test_count - 1))\r
 102 > +    }\r
 103 > +else\r
 104 > +    print_subtest () {\r
 105 > +     true\r
 106 > +    }\r
 107 > +fi\r
 108 > +\r
 109 >  if test -n "$color"; then\r
 110 >       say_color () {\r
 111 >               (\r
 112 > @@ -132,6 +142,7 @@ if test -n "$color"; then\r
 113 >               printf " "\r
 114 >                  printf "$@"\r
 115 >               tput sgr0\r
 116 > +             print_subtest\r
 117 >               )\r
 118 >       }\r
 119 >  else\r
 120 > @@ -140,6 +151,7 @@ else\r
 121 >               shift\r
 122 >               printf " "\r
 123 >                  printf "$@"\r
 124 > +             print_subtest\r
 125 >       }\r
 126 >  fi\r
 127 >  \r
 128 > -- \r
 129 > 1.7.5.4\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 -- \r
 138 Tomi Ollila !! tomi.ollila@nixu.fi !! +358 400 888 366\r
 139 Nixu Oy !! http://www.nixu.fi/ !! Keilaranta 15, FI-02150 Espoo, Finland\r