90/8838b7db00cd579681e07d57efbd7b722a4ef9

   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 CC1E2429E21\r
   6         for <notmuch@notmuchmail.org>; Thu, 17 Nov 2011 01:47:02 -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.1\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.1 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1]\r
  13         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 2UqiOw8qxeRj for <notmuch@notmuchmail.org>;\r
  17         Thu, 17 Nov 2011 01:47:02 -0800 (PST)\r
  18 Received: from ks3536.kimsufi.com (schnouki.net [87.98.217.222])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 2DBDF431FD0\r
  20         for <notmuch@notmuchmail.org>; Thu, 17 Nov 2011 01:47:02 -0800 (PST)\r
  21 Received: from thor.loria.fr (thor.loria.fr [152.81.12.250])\r
  22         by ks3536.kimsufi.com (Postfix) with ESMTPSA id 095A26A0026;\r
  23         Thu, 17 Nov 2011 10:47:01 +0100 (CET)\r
  24 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=schnouki.net;\r
  25         s=key-schnouki; t=1321523221;\r
  26         bh=PDFUe1hLjaqsY3n4YpaR0pW2NfiMA+faymbBEjVniTM=;\r
  27         h=From:To:Subject:In-Reply-To:References:Date:Message-ID:\r
  28         MIME-Version:Content-Type;\r
  29         b=XI5S4LVMniYtK2FHZ1aYFWlmXg1Z5XWNwcARX1F7XYSqXdBybA7Z09vo8C/NpMVnB\r
  30         SkWvZgAT5m3VV1Dz/1U9UUAfnVhIP7p9ZYGlX5ax5IYick1OWDPtU19x/mhqAwBpQW\r
  31         NTJ8rFRcoVhpm2paz5x+qSvh7FaA1EozkNXIW0jY=\r
  32 From: Thomas Jost <schnouki@schnouki.net>\r
  33 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH 0/9] test: (hopefully) better test prerequisites\r
  35 In-Reply-To: <1321494986-18998-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  36 References: <1321494986-18998-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  37 User-Agent: Notmuch/0.10_rc1+21~gaf76c55 (http://notmuchmail.org)\r
  38         Emacs/24.0.90.1 (x86_64-unknown-linux-gnu)\r
  39 Date: Thu, 17 Nov 2011 10:46:58 +0100\r
  40 Message-ID: <87d3craxnx.fsf@thor.loria.fr>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: multipart/signed; boundary="=-=-=";\r
  43         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Thu, 17 Nov 2011 09:47:02 -0000\r
  57 \r
  58 --=-=-=\r
  59 Content-Type: text/plain\r
  60 Content-Transfer-Encoding: quoted-printable\r
  61 \r
  62 On Thu, 17 Nov 2011 05:56:17 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmai=\r
  63 l.com> wrote:\r
  64 > Hi all.\r
  65 >=20\r
  66 > The following patch series is an attempt to introduce proper\r
  67 > dependencies for external binaries in a less intrusive way than\r
  68 > [1].  The primary aim was to avoid changing every subtest that\r
  69 > uses external binaries.\r
  70 >=20\r
  71 > There are still failing tests if a dependency is\r
  72 > missing (e.g. "Verify that sent messages are\r
  73 > saved/searchable (via FCC)" fails if there is no emacs).  It\r
  74 > happens because such tests depend on others which are skipped.\r
  75 > This issues are not addressed by this patch series.\r
  76 >=20\r
  77 > If others do like the approach and it is pushed, I will work on\r
  78 > updating tests that use the old style prerequisites (atomicity).\r
  79 >=20\r
  80 > A careful review is needed!\r
  81 >=20\r
  82 > Regards,\r
  83 >   Dmitry\r
  84 >=20\r
  85 > [1] id:"1321454035-22023-1-git-send-email-schnouki@schnouki.net"\r
  86 \r
  87 Hi Dmitry,\r
  88 \r
  89 This series looks quite good to me. It's a good approach, cleaner than\r
  90 explicitely adding the prereqs to each test as in my previous patches\r
  91 (and Pieter's).\r
  92 \r
  93 Now, a few questions:\r
  94 \r
  95 =2D same as Jamie: emacs_deliver_message hangs if dtach is not installed.\r
  96   In my patches I had to do this: "test_have_prereq EMACS &&\r
  97   emacs_deliver_message ...". Any idea how to handle this?\r
  98 \r
  99 =2D what about indirect, "hidden" dependencies? The crypto test need to\r
 100   have a signed message delivered by emacs, so actually *all* the crypto\r
 101   tests depend on emacs. Right now, when dtach is not installed, the\r
 102   first test ("emacs delivery of signed message") is skipped and all the\r
 103   others fail. Would it be possible to handle this case without having\r
 104   to add explicit prereqs?\r
 105 \r
 106 =2D right now functions like test_expect_success can be used as\r
 107   "test_expect_success COMMAND" or "test_expect_success PREREQ COMMAND".\r
 108   If we use your approach (and I hope we will!), do we need to keep that\r
 109   second syntax available in test-lib.sh too, or should we do some\r
 110   cleanup to get rid of it?\r
 111 \r
 112 Thanks,\r
 113 \r
 114 =2D-=20\r
 115 Thomas/Schnouki\r
 116 \r
 117 --=-=-=\r
 118 Content-Type: application/pgp-signature\r
 119 \r
 120 -----BEGIN PGP SIGNATURE-----\r
 121 Version: GnuPG v1.4.11 (GNU/Linux)\r
 122 \r
 123 iQEcBAEBAgAGBQJOxNgSAAoJEMPdciX+bh5I0i0IAKOUvatel+QffjVM8ic22wZk\r
 124 Pkp8Vkq9mGIvEqreJgzmG5FdYC5d3TnC2/31f2xcRxDycdXw3QlHjCT6nu+S40fs\r
 125 f2n3Bp7AQtvW3ofXLGdMNxxoDMtLtYURCT6y3a8peAMudZ4us2hyHwnoJkG7+VUh\r
 126 lXWyemG010edMDCRscTl4jyh+w0g7rSDdAKbXDewMCYwujhE9O6yPqvqda2BYBgM\r
 127 faJ5B2upW5cOmcmtkdQwAhgvyTsV3xs4k4ldIl7wjOO7uX7wa3W8bbzeMHbde6+j\r
 128 GuF8N33pv1NeOmSfcblicJ0aG3PxohgBWSOoxywtyV/kI5gNrP1JQAzZ3O57oMA=\r
 129 =sKM1\r
 130 -----END PGP SIGNATURE-----\r
 131 --=-=-=--\r