87/7b71c6d20c6bb17adf8b15896f1c33e4c99758

   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 7C201431FBD\r
   6         for <notmuch@notmuchmail.org>; Thu,  2 Feb 2012 09:41:06 -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 mxHBNkZt8KAc for <notmuch@notmuchmail.org>;\r
  16         Thu,  2 Feb 2012 09:41:06 -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 0A44A431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu,  2 Feb 2012 09:41:06 -0800 (PST)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id 9FB363280FA;\r
  23         Thu,  2 Feb 2012 09:40:52 -0800 (PST)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-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 fire-doxen-submit (Postfix) with ESMTP id 5382A3280F3;\r
  28         Thu,  2 Feb 2012 09:40:44 -0800 (PST)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id B9EFA291; Thu,  2 Feb 2012 09:40:43 -0800 (PST)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH] test: make test_expect_equal_file() arguments flexible\r
  34 In-Reply-To: <1328080794-24670-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  35 References: <1328080794-24670-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  36 User-Agent: Notmuch/0.11+128~g6f388fa (http://notmuchmail.org) Emacs/23.3.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Thu, 02 Feb 2012 09:40:40 -0800\r
  39 Message-ID: <87aa51qgkn.fsf@servo.finestructure.net>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: multipart/signed; boundary="=-=-=";\r
  42         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  43 X-BeenThere: notmuch@notmuchmail.org\r
  44 X-Mailman-Version: 2.1.13\r
  45 Precedence: list\r
  46 List-Id: "Use and development of the notmuch mail system."\r
  47         <notmuch.notmuchmail.org>\r
  48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  51 List-Post: <mailto:notmuch@notmuchmail.org>\r
  52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  55 X-List-Received-Date: Thu, 02 Feb 2012 17:41:06 -0000\r
  56 \r
  57 --=-=-=\r
  58 \r
  59 On Wed,  1 Feb 2012 11:19:54 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  60 > Before the change, test_expect_equal_file() function treated the first\r
  61 > argument as "actual output file" and the second argument as "expected\r
  62 > output file".  When the test fails, the files are copied for later\r
  63 > inspection.  The first files was copied to "$testname.output" and the\r
  64 > second file to "$testname.expected".  The argument order for\r
  65 > test_expect_equal_file() is often wrong which results in confusing\r
  66 > diff output and incorrectly named files.\r
  67 \r
  68 After thinking about this some more, I'm taking it all back.  I think\r
  69 this is a fine solution, since it just goes with whatever name the test\r
  70 is invoked with.  Since this is usually just OUTPUT and EXPECTED, it\r
  71 should all be clear, and the diffs should be fine, even if the ordering\r
  72 is permuted some places.  Sorry about all the chatter.\r
  73 \r
  74 +1\r
  75 \r
  76 jamie.\r
  77 \r
  78 --=-=-=\r
  79 Content-Type: application/pgp-signature\r
  80 \r
  81 -----BEGIN PGP SIGNATURE-----\r
  82 Version: GnuPG v1.4.11 (GNU/Linux)\r
  83 \r
  84 iQIcBAEBCAAGBQJPKsqYAAoJEO00zqvie6q8PEcP/ihx7cEc+0bcpUG6Is8n1+GL\r
  85 44xxyrCZU1oQ/vE5MiZHJB9njQgTDRyaIdNruPnKdx5H6FJ3nKqFByoJLAYSgwIa\r
  86 NmA3TzdD9N00p0u6+Yt5qiXjn9zopjJ8KkKUr+3sZ3x8qJvsnJyHJaWWFkepz4NU\r
  87 ob/qeQ8cT2ZArXjIGqmVFgU9peul3vEOK6kaMpRWc639fnn0jNI9xQp4Y/ao2lEJ\r
  88 0llrPpGtxnoMHvKz/oTZOJ9jmhoBPwynmWgHcltnawQm5mQfQtOqGSwMX3Tq0eh6\r
  89 OERcir+fFlmJyzLnWzSyyck77cvD8lWdxlEfFir9hrEGOxHoGNMCL6OLrQ8dBJc+\r
  90 02aY9/pun9Oe5d34OAGHw9A85+hF14lYXG6MRYSRjp2Ox10Z2u1gLF2qY766uFGn\r
  91 wH6DZdqF87sm1NmCz9PuJCc3Osi2iiwKWZFA+hAf0lCtsb3YNvbixKwHIbfletXB\r
  92 aUD17WeunOytjW4vB1Pztk7l/2W9/33oZacRJLQAQmRC8jX0dYSBtsmUgE7reIbV\r
  93 de8U1Ke2JBr4mwyZDOoRBtwXiMwLFa1GQd09h/BdfBu9pFul29+P54l2mtTQimte\r
  94 zejc0dux8CJ+q4pENDRy0CsJvHc9mx30NW5dPD1dSRbbblegtYmvbkHjBorVOCFw\r
  95 TWKtxGMfnVifjvS1yBqi\r
  96 =gS+h\r
  97 -----END PGP SIGNATURE-----\r
  98 --=-=-=--\r