65/7a14a08cee4327f6da530794618f5f59568d66

   1 Return-Path: <amdragon@mit.edu>\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 8EF4C431FBD\r
   6         for <notmuch@notmuchmail.org>; Mon, 22 Oct 2012 01:28:22 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 gZs8CbDPp6sB for <notmuch@notmuchmail.org>;\r
  16         Mon, 22 Oct 2012 01:28:21 -0700 (PDT)\r
  17 Received: from dmz-mailsec-scanner-3.mit.edu (DMZ-MAILSEC-SCANNER-3.MIT.EDU\r
  18         [18.9.25.14])\r
  19         by olra.theworths.org (Postfix) with ESMTP id CD323431FAE\r
  20         for <notmuch@notmuchmail.org>; Mon, 22 Oct 2012 01:28:21 -0700 (PDT)\r
  21 X-AuditID: 1209190e-b7f756d000000904-0b-508503a587bf\r
  22 Received: from mailhub-auth-4.mit.edu ( [18.7.62.39])\r
  23         by dmz-mailsec-scanner-3.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 9D.45.02308.5A305805; Mon, 22 Oct 2012 04:28:21 -0400 (EDT)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-4.mit.edu (8.13.8/8.9.2) with ESMTP id q9M8SKv1019666; \r
  27         Mon, 22 Oct 2012 04:28:21 -0400\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q9M8SJsq025081\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Mon, 22 Oct 2012 04:28:20 -0400 (EDT)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1TQDMw-0003Wh-Ne; Mon, 22 Oct 2012 04:28:18 -0400\r
  37 Date: Mon, 22 Oct 2012 04:28:17 -0400\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Ethan Glasser-Camp <ethan.glasser.camp@gmail.com>\r
  40 Subject: Re: [PATCH] test: new: Fix intermittent test failures with --debug\r
  41 Message-ID: <20121022082817.GN14861@mit.edu>\r
  42 References: <1350825298-8261-1-git-send-email-ethan@betacantrips.com>\r
  43 MIME-Version: 1.0\r
  44 Content-Type: text/plain; charset=us-ascii\r
  45 Content-Disposition: inline\r
  46 In-Reply-To: <1350825298-8261-1-git-send-email-ethan@betacantrips.com>\r
  47 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  48 X-Brightmail-Tracker:\r
  49  H4sIAAAAAAAAA+NgFuphleLIzCtJLcpLzFFi42IRYrdT113K3Bpg8Ou/qMWdnl0sFrsmz2Wz\r
  50         uH5zJrMDs8f/63+ZPHbOusvu8WzVLeYA5igum5TUnMyy1CJ9uwSujBnLJQq2i1e82/GTuYHx\r
  51         jnAXIyeHhICJxKe751ggbDGJC/fWs3UxcnEICexjlDhw9hgjhLOBUaLh+zFWCOckk8Tf+Yuh\r
  52         MksYJQ7/nMMM0s8ioCox4fUMRhCbTUBDYtv+5UA2B4eIgKnE34PqIGFmAQ+JPb3TwNYJC3hL\r
  53         3Nv1jRXE5hXQkZjYsBrMFhJwlfjatQkqLihxcuYTFoheLYkb/14ygYxkFpCWWP6PAyTMKeAm\r
  54         cfHwY7CtogIqElNObmObwCg0C0n3LCTdsxC6FzAyr2KUTcmt0s1NzMwpTk3WLU5OzMtLLdI1\r
  55         1svNLNFLTSndxAgKdE5Jvh2MXw8qHWIU4GBU4uFtuN4SIMSaWFZcmXuIUZKDSUmU994foBBf\r
  56         Un5KZUZicUZ8UWlOavEhRgkOZiURXleG1gAh3pTEyqrUonyYlDQHi5I475WUm/5CAumJJanZ\r
  57         qakFqUUwWRkODiUJ3glMQI2CRanpqRVpmTklCGkmDk6Q4TxAw/VBaniLCxJzizPTIfKnGBWl\r
  58         xHlXgyQEQBIZpXlwvbBE9IpRHOgVYd4lIFU8wCQG1/0KaDAT0GBz7kaQwSWJCCmpBsZY9ocT\r
  59         dpxfZ99ge9it6xdHxrT+z/VCD38omn/d7f+s6NMzRZfclNtxllGTNPpPFcg5flqzM3Ta7Ivb\r
  60         VrNx8MQYnqxVMFroV12T1yGb0rC/YcGmdCMRxkouFlYtVbZP0tH3Vu4Mush58fSkps+eP+ao\r
  61         MaVtW/Dvpr7txIMbBPL1Gh+JSPjEpCixFGckGmoxFxUnAgCCFHs9HwMAAA==\r
  62 Cc: notmuch@notmuchmail.org, Ethan Glasser-Camp <ethan@betacantrips.com>\r
  63 X-BeenThere: notmuch@notmuchmail.org\r
  64 X-Mailman-Version: 2.1.13\r
  65 Precedence: list\r
  66 List-Id: "Use and development of the notmuch mail system."\r
  67         <notmuch.notmuchmail.org>\r
  68 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  70 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  71 List-Post: <mailto:notmuch@notmuchmail.org>\r
  72 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  73 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  74         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  75 X-List-Received-Date: Mon, 22 Oct 2012 08:28:22 -0000\r
  76 \r
  77 LGTM.\r
  78 \r
  79 This is closely related to another problem in the test suite that\r
  80 causes frequent failures for me when I run on tmpfs: generate_message\r
  81 creates messages with the same time stamp by default, which means that\r
  82 the output of date-sorted notmuch search is technically\r
  83 non-deterministic.  I suspect this doesn't cause problems most of the\r
  84 time or for most people because messages get added in the order\r
  85 they're read from the file system (of course, this isn't guaranteed in\r
  86 general and doesn't hold on tmpfs), which assigns them monotonically\r
  87 increasing docids, which stabilizes the search results (though I don't\r
  88 think this is guaranteed either).\r
  89 \r
  90 Quoth Ethan Glasser-Camp on Oct 21 at  9:14 am:\r
  91 > Although messages are created in a particular order, it seems that\r
  92 > when they are created on a tmpfs, they do not always come back in the\r
  93 > same order, leading to the same files being ignored but being output\r
  94 > in a different order. This causes the test to fail because the outputs\r
  95 > being compared are the same.\r
  96 > \r
  97 > Fix the failures by sorting the output of notmuch --debug and\r
  98 > comparing this to a hand-sorted version of its output.\r
  99 > \r
 100 > Signed-off-by: Ethan Glasser-Camp <ethan@betacantrips.com>\r
 101 > ---\r
 102 >  test/new |   10 +++++-----\r
 103 >  1 file changed, 5 insertions(+), 5 deletions(-)\r
 104 > \r
 105 > diff --git a/test/new b/test/new\r
 106 > index cc2af72..587aa11 100755\r
 107 > --- a/test/new\r
 108 > +++ b/test/new\r
 109 > @@ -189,17 +189,17 @@ touch "${MAIL_DIR}"/.git # change .git's mtime for notmuch new to rescan.\r
 110 >  mkdir -p "${MAIL_DIR}"/one/two/three/.git\r
 111 >  notmuch new > /dev/null # ensure that files/folders will be printed in ASCII order.\r
 112 >  touch "${MAIL_DIR}"/{one,one/two,one/two/three}/ignored_file\r
 113 > -output=$(NOTMUCH_NEW --debug 2>&1)\r
 114 > +output=$(NOTMUCH_NEW --debug 2>&1 | sort)\r
 115 >  test_expect_equal "$output" \\r
 116 >  "(D) add_files_recursive, pass 1: explicitly ignoring ${MAIL_DIR}/.git\r
 117 >  (D) add_files_recursive, pass 1: explicitly ignoring ${MAIL_DIR}/one/two/three/.git\r
 118 > -(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/three/.git\r
 119 > -(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/three/ignored_file\r
 120 > -(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/ignored_file\r
 121 > -(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/ignored_file\r
 122 >  (D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/.git\r
 123 >  (D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/.ignored_hidden_file\r
 124 >  (D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/ignored_file\r
 125 > +(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/ignored_file\r
 126 > +(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/ignored_file\r
 127 > +(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/three/.git\r
 128 > +(D) add_files_recursive, pass 2: explicitly ignoring ${MAIL_DIR}/one/two/three/ignored_file\r
 129 >  No new mail."\r
 130 >  \r
 131 >  \r