20/241f8f5af7f585df49b25a16cfefe15bf5908e

   1 Return-Path: <tomi.ollila@iki.fi>\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 arlo.cworth.org (Postfix) with ESMTP id 53A236DE0B27\r
   6  for <notmuch@notmuchmail.org>; Thu,  7 Jan 2016 13:50:01 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.666\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.666 tagged_above=-999 required=5 tests=[AWL=0.014, \r
  12  SPF_NEUTRAL=0.652] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id kuFCi-9i9Hes for <notmuch@notmuchmail.org>;\r
  16  Thu,  7 Jan 2016 13:49:58 -0800 (PST)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18  by arlo.cworth.org (Postfix) with ESMTP id EBDFA6DE0B26\r
  19  for <notmuch@notmuchmail.org>; Thu,  7 Jan 2016 13:49:57 -0800 (PST)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21  by guru.guru-group.fi (Postfix) with ESMTP id EC86C1000B3;\r
  22  Thu,  7 Jan 2016 23:50:10 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: J Farkas <jf.hyqohaczlksw4tx6ae@l2015aftruuq.dns007.net>,\r
  25  David Bremner <david@tethera.net>\r
  26 Cc: notmuch@notmuchmail.org\r
  27 Subject: Re: T070 tests portability\r
  28 In-Reply-To: <1452088279.96.560c52fa@201601.l2015aftruuq.dns007.net>\r
  29 References: <1451991305.14.acc64b88@201601.l2015aftruuq.dns007.net>\r
  30  <m2io37agl8.fsf@guru.guru-group.fi> <87ziwj9025.fsf@zancas.localnet>\r
  31  <1452088279.96.560c52fa@201601.l2015aftruuq.dns007.net>\r
  32 User-Agent: Notmuch/0.21+32~g73439f8 (http://notmuchmail.org) Emacs/24.3.1\r
  33  (x86_64-unknown-linux-gnu)\r
  34 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  35  $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  36  !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  37 Date: Thu, 07 Jan 2016 23:50:10 +0200\r
  38 Message-ID: <m260z5jbzh.fsf@guru.guru-group.fi>\r
  39 MIME-Version: 1.0\r
  40 Content-Type: text/plain; charset=utf-8\r
  41 Content-Transfer-Encoding: quoted-printable\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.20\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46  <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  48  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Thu, 07 Jan 2016 21:50:01 -0000\r
  55 \r
  56 On Wed, Jan 06 2016, J Farkas <jf.hyqohaczlksw4tx6ae@l2015aftruuq.dns007.ne=\r
  57 t> wrote:\r
  58 \r
  59 > On 2016-01-05 at 17:41:06, David Bremner wrote:\r
  60 >> Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  61 >>=20\r
  62 >> > The page https://sourceware.org/gdb/onlinedocs/gdb/Returning.html\r
  63 >> > talks something about gdb not knowing the return type if function\r
  64 >> > was compiled without debug info... well, is this is the reason,\r
  65 >> > perhaps we should allow testing w/o debug info compiled in.\r
  66 >>=20\r
  67 >> The test suite overall assumes debug info is present; T000-basic tests\r
  68 >> for this.\r
  69 >\r
  70 > I'm not missing the debug infos (well, except for the thread library if I\r
  71 > understand correctly - and that affects a lot of other tests, but not thi=\r
  72 s).\r
  73 > Perhaps it the version of gdb, but not any compilation flags that can be\r
  74 > changed for the notmuch tree.\r
  75 >\r
  76 > Without the changes, I'm getting these two kind of errors (and as you can\r
  77 > see, the test for the debugging symbols passes too).\r
  78 \r
  79 ack... keep reading...\r
  80 \r
  81 >\r
  82 > $ ./T000-basic.sh\r
  83 >\r
  84 > T000-basic: Testing the test framework itself.\r
  85 >  PASS   success is reported like this\r
  86 > ...\r
  87 >  PASS   PATH is set to build directory\r
  88 >  PASS   notmuch is compiled with debugging symbols\r
  89 > $ ./T070-insert.sh\r
  90 >\r
  91 > T070-insert: Testing "notmuch insert"\r
  92 >  PASS   Insert zero-length file\r
  93 > ...\r
  94 >  FAIL   Insert duplicate message\r
  95 >         --- T070-insert.6.expected      2016-01-06 13:31:01.644676102 +00=\r
  96 00\r
  97 >         +++ T070-insert.6.output        2016-01-06 13:31:01.648675891 +00=\r
  98 00\r
  99 >         @@ -1 +1 @@\r
 100 >         -2\r
 101 >         +      2\r
 102 \r
 103 This is trivial to fix w/ the same\r
 104 \r
 105 output=3D$((`notmuch search --output=3Dfiles "subject:insert-subject" | wc =\r
 106 -l`))\r
 107 \r
 108 construct that is used in T060-count.sh\r
 109 \r
 110 A separate patch doing just this would be appreciated.\r
 111 \r
 112 >  PASS   Duplicate message does not change tags\r
 113 >  PASS   Insert message, add tag\r
 114 > ...\r
 115 >  PASS   Tags starting with '-' in new.tags are forbidden\r
 116 >  PASS   Invalid tags set exit code\r
 117 >  FAIL   error exit when add_message returns OUT_OF_MEMORY\r
 118 >         --- T070-insert.26.expected     2016-01-06 13:31:06.940397013 +00=\r
 119 00\r
 120 >         +++ T070-insert.26.output       2016-01-06 13:31:06.940397013 +00=\r
 121 00\r
 122 >         @@ -1 +1 @@\r
 123 >         -1\r
 124 >         +255\r
 125 > warning: Unable to find libthread_db matching inferior's thread library, =\r
 126 thread debugging will not be available.\r
 127 > index-file-OUT_OF_MEMORY.gdb:7: Error in sourced command file:\r
 128 > Return value type not available for selected stack frame.\r
 129 > Please use an explicit cast of the value to return.\r
 130 >  FAIL   success exit with --keep when add_message returns OUT_OF_MEMORY\r
 131 \r
 132 Your setup is interesting -- you have GNU textutils 2.0.22, released\r
 133 2002-07-20 (i.e. 13.5 years ago) and then gdb 7.4. I just run this\r
 134 test file on Scientific Linux 6.2 (gcc 4.4.6 and gdb 7.2), and the\r
 135 tests passed without problems.\r
 136 \r
 137 Apart from a few special cases, no typecasts should be added lightly\r
 138 to the code -- be it at innocent-looking as it can be. Those impedes\r
 139 the possibility for compiler (etc.) to do type checking.\r
 140 \r
 141 So, in this case, if you're the only one seeing this problem I'd suggest\r
 142 you tolerate the problem (by using private patch) (you could also dig\r
 143 deeper if interested). If this "problem" is (were!) more widespread,\r
 144 *and* unfixable in reasonable manner this thing like this typecast\r
 145 could be applied -- which would also mean this test system would\r
 146 be a bit less robust as it is now...(*)\r
 147 \r
 148 \r
 149 Tomi\r
 150 \r
 151 (*) imo these are hard facts, but someone(tm) may consider these as mere\r
 152 opinions...\r
 153 \r
 154 \r
 155 > ...\r
 156 >\r
 157 > (The above is then repeated for all gdb using `insert` tests.)\r
 158 >\r
 159 > J=C3=A1nos\r