bd/310ed43ee5116f296d0d9b1fb2565e7a321a85

   1 Return-Path: <jrollins@finestructure.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 BCCE1421192\r
   6         for <notmuch@notmuchmail.org>; Wed,  1 Feb 2012 09:24:46 -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: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 o5OOtEwJL9Yf for <notmuch@notmuchmail.org>;\r
  16         Wed,  1 Feb 2012 09:24:46 -0800 (PST)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 09A6E429E5B\r
  20         for <notmuch@notmuchmail.org>; Wed,  1 Feb 2012 09:24:46 -0800 (PST)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id EAFD266E0030;\r
  23         Wed,  1 Feb 2012 09:24:43 -0800 (PST)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
  25 Received: from finestructure.net (cpe-76-174-137-84.socal.res.rr.com\r
  26         [76.174.137.84]) (Authenticated sender: jrollins)\r
  27         by earth-doxen-submit (Postfix) with ESMTP id 1580B66E01A1;\r
  28         Wed,  1 Feb 2012 09:24:37 -0800 (PST)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id DEFCC291; Wed,  1 Feb 2012 09:24:35 -0800 (PST)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>,\r
  33         Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH] test: make test_expect_equal_file() arguments flexible\r
  35 In-Reply-To: <87k446n8ji.fsf@gmail.com>\r
  36 References: <1328080794-24670-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  37         <87r4yfszx9.fsf@servo.finestructure.net>\r
  38         <87pqdync64.fsf@gmail.com> <m262fqc0wv.fsf@guru.guru-group.fi>\r
  39         <87k446n8ji.fsf@gmail.com>\r
  40 User-Agent: Notmuch/0.11+128~g6f388fa (http://notmuchmail.org) Emacs/23.3.1\r
  41         (x86_64-pc-linux-gnu)\r
  42 Date: Wed, 01 Feb 2012 09:24:32 -0800\r
  43 Message-ID: <87ehuetqjz.fsf@servo.finestructure.net>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: multipart/signed; boundary="=-=-=";\r
  46         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  47 X-BeenThere: notmuch@notmuchmail.org\r
  48 X-Mailman-Version: 2.1.13\r
  49 Precedence: list\r
  50 List-Id: "Use and development of the notmuch mail system."\r
  51         <notmuch.notmuchmail.org>\r
  52 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  54 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  55 List-Post: <mailto:notmuch@notmuchmail.org>\r
  56 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  57 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  59 X-List-Received-Date: Wed, 01 Feb 2012 17:24:46 -0000\r
  60 \r
  61 --=-=-=\r
  62 Content-Transfer-Encoding: quoted-printable\r
  63 \r
  64 On Wed, 01 Feb 2012 14:37:53 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmai=\r
  65 l.com> wrote:\r
  66 > On Wed, 01 Feb 2012 12:18:08 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrot=\r
  67 e:\r
  68 > >=20\r
  69 > > There are at least these options here\r
  70 > >=20\r
  71 > > 1) go through all ~100 places where test_expect_equal_file is used\r
  72 > >    and fix the call order: quick look tells that the offending uses\r
  73 > >    are in dump-restore, hooks, search-limiting and symbol-hiding.\r
  74 > >=20\r
  75 > > 2) enforce "expected" filename has some format *and* fix all current\r
  76 > >    uses of it. Add testbed_error () function which yells loudly ane exi=\r
  77 ts...\r
  78 > >=20\r
  79 > > 3) guess which is output and which is expected from args so that=20\r
  80 > >    machine helps tester here (for both diff output & copied files)a\r
  81 > >=20\r
  82 > > 4) just copy compared files to some directory, those are named as\r
  83 > >    basename of the original -- diff order still inconsistent.\r
  84 > >=20\r
  85 > >=20\r
  86 > > I'd just go with option 1 and fix new *violations* when stumble upon on=\r
  87 e.\r
  88 > >=20\r
  89 >=20\r
  90 > Option 1 does not solve the problem.  New violations would apper and\r
  91 > need to be fixed again.  I am for option 2.\r
  92 \r
  93 How is enforcing use of a particular filename better and more robust\r
  94 than enforcing argument order?  You'll still have to force an arbitrary\r
  95 heuristic.  And you'll still be vulnerable to people messing up the file\r
  96 names (which actually seems easier to get wrong than messing up the\r
  97 order).  And you'll have to have more code to parse the argument\r
  98 strings.  And you'll still get inconsistent diffs.\r
  99 \r
 100 If this is really a problem, I vote for 1.  In general, I am not in\r
 101 favor of making the test suite more complicated than it needs to be.\r
 102 \r
 103 jamie.\r
 104 \r
 105 --=-=-=\r
 106 Content-Type: application/pgp-signature\r
 107 \r
 108 -----BEGIN PGP SIGNATURE-----\r
 109 Version: GnuPG v1.4.11 (GNU/Linux)\r
 110 \r
 111 iQIcBAEBCAAGBQJPKXVQAAoJEO00zqvie6q8/L0P/3zAXYtLqisL3wl5Zk2urhcH\r
 112 6kQI2EFKcdPqRO1aKPvjT7dlnkXHeztxovwAYNAl2fj7TRbnnIzsGmm1H49fDi1b\r
 113 Sqa/aHwm9d8GFiHP981YV9cgLfyvHQdRjr7OXBangZTKzIoIs7s/uV9OAHXAGmN3\r
 114 SbX4f5el9MVYMQXxbVs4B6yGP4qMi1LsIyFxoI3crrYqJIykbedOMfglFPzbb3vD\r
 115 0ivtMphws9mutnDyXJNCF+HzRPDjnI7ac3/u2KqJ67++WOOJ76hfHRa0f4tcrGC8\r
 116 XNakXDDiUhirvBn7+0usG+iwoEd8SwC1IsbTPrNywAE/WjCi78sjmRHHpBHr+qAE\r
 117 T7tDudL3tctuI4c3MW95GbbVWSLGsOvWcXR64O12gEyhFs5P9wnNMbx6KhdG8WxM\r
 118 vbwIixdSCU1CJUAKOFs9MVbBRbtddrvSwWlL9JEwig+/41mH5HiuvIngddKyOYkP\r
 119 84NLTzY9cG9FJzrVs5ud++N3D5bjhPcZbeUQ/ZD0mjsq1AKtzY/fZqVytZywthCH\r
 120 gVobPgz50mCNnkKrod9oMmHrqkVXBpTrguvRuO4P4Cek6aILxk92O7uxtZd3wQB6\r
 121 BQFTOh8+331zIkoXuDkzSArMfrb8+IVKDdy1WB7zWiq9qFUOYXjzV/WNQqKGU8Rz\r
 122 QttiXiBdZIDBkfbQb6Vw\r
 123 =xYxA\r
 124 -----END PGP SIGNATURE-----\r
 125 --=-=-=--\r