8d/aa76a50dc6484c359025faa899f8ef1e9199f3

   1 Return-Path: <dmitry.kurochkin@gmail.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 F2AEE431FBC\r
   6         for <notmuch@notmuchmail.org>; Fri, 25 May 2012 18:09:48 -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.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] 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 EeM0NtpnAJ6X for <notmuch@notmuchmail.org>;\r
  17         Fri, 25 May 2012 18:09:47 -0700 (PDT)\r
  18 Received: from mail-lb0-f181.google.com (mail-lb0-f181.google.com\r
  19         [209.85.217.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id C125A431FB6\r
  22         for <notmuch@notmuchmail.org>; Fri, 25 May 2012 18:09:46 -0700 (PDT)\r
  23 Received: by lbbgk8 with SMTP id gk8so1123563lbb.26\r
  24         for <notmuch@notmuchmail.org>; Fri, 25 May 2012 18:09:45 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  26         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
  27         :message-id:mime-version:content-type:content-transfer-encoding;\r
  28         bh=M2p9nkOnWj+p4wH0fs8BDa4b/yVYgEN3+LRL6RuuVL0=;\r
  29         b=uA2nrAvDMH2NJofFP41To2CXCuHf9bB1QsQUVwz3GF3zScGv5uyRbojBKYr3OFZNDo\r
  30         H4klSg+XhQP0ztuKIuVxMhyxt4i7OohAmJLMLHWMfLp5/eLT7a8tEmNCHNsb2dAFemfI\r
  31         yjy6Ulw+CNttiw5l/Z8tqlJbxn71uj0bRacbseHz5csEANZ7vONJFzScryTBGAqfqv4/\r
  32         wtIwP6gVV2cWMoooQg/cUullE4kn1d5MmGd7YZ/yn145ngIl3/PmoJAp6uKL/6hD2rd8\r
  33         +or2kRJ7oiz7cD9sxgrUFbuKeZyFVWHN7h4KIMk6PmcAILY2WIp10Y0Z2gPTrEa8jr3S\r
  34         so5w==\r
  35 Received: by 10.152.148.34 with SMTP id tp2mr765889lab.47.1337994585206;\r
  36         Fri, 25 May 2012 18:09:45 -0700 (PDT)\r
  37 Received: from localhost ([91.144.186.21])\r
  38         by mx.google.com with ESMTPS id lv13sm6333203lab.8.2012.05.25.18.09.43\r
  39         (version=TLSv1/SSLv3 cipher=OTHER);\r
  40         Fri, 25 May 2012 18:09:44 -0700 (PDT)\r
  41 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  42 To: Mike Kelly <pioto@pioto.org>\r
  43 Subject: Re: FreeBSD Support Patches\r
  44 In-Reply-To:\r
  45  <CAFb0NsJ7Bv-Lp03WHvZQh_yaimzaj34VmkVQCjr_u5MDrfqAKw@mail.gmail.com>\r
  46 References: <1337953411-21000-1-git-send-email-pioto@pioto.org>\r
  47         <87ipfktkgy.fsf@gmail.com>\r
  48         <CAFb0NsJ7Bv-Lp03WHvZQh_yaimzaj34VmkVQCjr_u5MDrfqAKw@mail.gmail.com>\r
  49 User-Agent: Notmuch/0.13+30~g6d44c5a (http://notmuchmail.org) Emacs/23.4.1\r
  50         (x86_64-pc-linux-gnu)\r
  51 Date: Sat, 26 May 2012 05:07:36 +0400\r
  52 Message-ID: <87aa0vu4vb.fsf@gmail.com>\r
  53 MIME-Version: 1.0\r
  54 Content-Type: text/plain; charset=utf-8\r
  55 Content-Transfer-Encoding: quoted-printable\r
  56 Cc: notmuch@notmuchmail.org\r
  57 X-BeenThere: notmuch@notmuchmail.org\r
  58 X-Mailman-Version: 2.1.13\r
  59 Precedence: list\r
  60 List-Id: "Use and development of the notmuch mail system."\r
  61         <notmuch.notmuchmail.org>\r
  62 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  64 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  65 List-Post: <mailto:notmuch@notmuchmail.org>\r
  66 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  67 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  68         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  69 X-List-Received-Date: Sat, 26 May 2012 01:09:49 -0000\r
  70 \r
  71 Mike Kelly <pioto@pioto.org> writes:\r
  72 \r
  73 > On Fri, May 25, 2012 at 10:15 AM, Dmitry Kurochkin\r
  74 > <dmitry.kurochkin@gmail.com> wrote:\r
  75 >> Why do we need to explicitly declare Emacs dependency for tests? =C2=A0T=\r
  76 here\r
  77 >> should be no need for it. =C2=A0We have "implicit" dependencies which are\r
  78 >> declared once (see test_declare_external_prereq calls at the end of\r
  79 >> test-lib.sh) and are automatically handled when a test tries to use a\r
  80 >> missing binary. =C2=A0Explicit dependencies are hard to maintain (e.g. y=\r
  81 our\r
  82 >> patch adds explicit emacs dependency for crypto test but misses gpg).\r
  83 >> With rare exceptions we should not use explicit dependencies.\r
  84 >\r
  85 > Because not every test actually has those implicit dependencies. For\r
  86 > example, some of the crypto tests depend upon emacs_deliver_message\r
  87 > working correctly for subsequents tests. Those emacs_deliver_message\r
  88 > tests are skipped, but not the ones after it that try to do something\r
  89 > with that injected message.\r
  90 >\r
  91 > For the emacs-* test files, there are some tests that act the same\r
  92 > way.\r
  93 \r
  94 These subtests do not directly depend on Emacs.  They depend on other\r
  95 subtests.  Currently, we do not support such dependencies.  But what you\r
  96 propose is not the solution.  We have two options here: make all\r
  97 subtests independent or introduce proper subtests dependencies.  The\r
  98 former might require many changes to existing tests and may be hard to\r
  99 enforce.  The latter is not trivial as well but is doable.  I planned to\r
 100 implement subtests dependencies but never really got to it (and I do not\r
 101 think I will anytime soon).\r
 102 \r
 103 > However, it is also a minor speed improvement to say that,\r
 104 > obviously, none of the emacs tests are going to work, so just don't\r
 105 > bother.\r
 106 >\r
 107 \r
 108 I do not think maintaining an explicit list of dependencies worth a\r
 109 minor speed improvement.\r
 110 \r
 111 Given all above, I understand that your patches fix a common problem in\r
 112 a simple way.  And it does not look like we would get proper solution\r
 113 anytime soon.  So I am ok with these patches with two comments:\r
 114 \r
 115   * Provide a proper commit message to explain the issue in more detail.\r
 116 \r
 117   * Add an XXX comment for each explicit dependency, something like:\r
 118 \r
 119     // XXX: Workaround for subtests that depend on other subtests (and,\r
 120     // hence, indirectly depend on emacs).  Should be removed when we\r
 121     // have proper subtests dependencies.\r
 122 \r
 123 Regards,\r
 124   Dmitry\r
 125 \r
 126 > --=20\r
 127 > Mike Kelly\r