43/a3f19846fcd5969689e1046f44d25754b10b50

   1 Return-Path: <david@tethera.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 18F6E431FAE\r
   6         for <notmuch@notmuchmail.org>; Tue, 28 Oct 2014 11:16:05 -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\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 9PUTMgNXjQIn for <notmuch@notmuchmail.org>;\r
  16         Tue, 28 Oct 2014 11:15:57 -0700 (PDT)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id BD476431FBC\r
  21         for <notmuch@notmuchmail.org>; Tue, 28 Oct 2014 11:15:57 -0700 (PDT)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1XjBJD-0002t7-Cw; Tue, 28 Oct 2014 15:15:55 -0300\r
  25 Received: (nullmailer pid 30371 invoked by uid 1000); Tue, 28 Oct 2014\r
  26         18:15:50 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH] test: fix test_require_external_prereq gdb in insert test\r
  30 In-Reply-To: <1414518754-16344-1-git-send-email-jani@nikula.org>\r
  31 References: <1414518754-16344-1-git-send-email-jani@nikula.org>\r
  32 User-Agent: Notmuch/0.18.1+131~gdd8373f (http://notmuchmail.org) Emacs/24.4.1\r
  33         (x86_64-pc-linux-gnu)\r
  34 Date: Tue, 28 Oct 2014 19:15:50 +0100\r
  35 Message-ID: <87a94g2h7t.fsf@maritornes.cs.unb.ca>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Tue, 28 Oct 2014 18:16:05 -0000\r
  51 \r
  52 Jani Nikula <jani@nikula.org> writes:\r
  53 \r
  54 > You need to actually check the return value. Limit the missing deps\r
  55 > part to subtests requiring gdb.\r
  56 \r
  57 For me, on a system without gdb, the current version looks like\r
  58 \r
  59 T070-insert: Testing "notmuch insert"\r
  60  missing prerequisites: gdb(1)\r
  61  SKIP   all tests in T070-insert\r
  62 \r
  63 This seems OK to me?\r
  64 \r
  65 Of course you're right that moving it forward to only skip the gdb\r
  66 requiring tests is the right thing to do, and in that setting having a\r
  67 skipped message for all 10 skipped tests is a bit ugly. OTOHO your\r
  68 version doesn't give any output at all, which seems not perfect either.\r
  69 \r