Re: T070 tests portability
authorTomi Ollila <tomi.ollila@iki.fi>
Thu, 7 Jan 2016 21:50:10 +0000 (23:50 +0200)
committerW. Trevor King <wking@tremily.us>
Sat, 20 Aug 2016 23:20:48 +0000 (16:20 -0700)
20/241f8f5af7f585df49b25a16cfefe15bf5908e [new file with mode: 0644]

diff --git a/20/241f8f5af7f585df49b25a16cfefe15bf5908e b/20/241f8f5af7f585df49b25a16cfefe15bf5908e
new file mode 100644 (file)
index 0000000..23311de
--- /dev/null
@@ -0,0 +1,159 @@
+Return-Path: <tomi.ollila@iki.fi>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+ by arlo.cworth.org (Postfix) with ESMTP id 53A236DE0B27\r
+ for <notmuch@notmuchmail.org>; Thu,  7 Jan 2016 13:50:01 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at cworth.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: 0.666\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=0.666 tagged_above=-999 required=5 tests=[AWL=0.014, \r
+ SPF_NEUTRAL=0.652] autolearn=disabled\r
+Received: from arlo.cworth.org ([127.0.0.1])\r
+ by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
+ with ESMTP id kuFCi-9i9Hes for <notmuch@notmuchmail.org>;\r
+ Thu,  7 Jan 2016 13:49:58 -0800 (PST)\r
+Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
+ by arlo.cworth.org (Postfix) with ESMTP id EBDFA6DE0B26\r
+ for <notmuch@notmuchmail.org>; Thu,  7 Jan 2016 13:49:57 -0800 (PST)\r
+Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
+ by guru.guru-group.fi (Postfix) with ESMTP id EC86C1000B3;\r
+ Thu,  7 Jan 2016 23:50:10 +0200 (EET)\r
+From: Tomi Ollila <tomi.ollila@iki.fi>\r
+To: J Farkas <jf.hyqohaczlksw4tx6ae@l2015aftruuq.dns007.net>,\r
+ David Bremner <david@tethera.net>\r
+Cc: notmuch@notmuchmail.org\r
+Subject: Re: T070 tests portability\r
+In-Reply-To: <1452088279.96.560c52fa@201601.l2015aftruuq.dns007.net>\r
+References: <1451991305.14.acc64b88@201601.l2015aftruuq.dns007.net>\r
+ <m2io37agl8.fsf@guru.guru-group.fi> <87ziwj9025.fsf@zancas.localnet>\r
+ <1452088279.96.560c52fa@201601.l2015aftruuq.dns007.net>\r
+User-Agent: Notmuch/0.21+32~g73439f8 (http://notmuchmail.org) Emacs/24.3.1\r
+ (x86_64-unknown-linux-gnu)\r
+X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
+ $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
+ !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
+Date: Thu, 07 Jan 2016 23:50:10 +0200\r
+Message-ID: <m260z5jbzh.fsf@guru.guru-group.fi>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=utf-8\r
+Content-Transfer-Encoding: quoted-printable\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.20\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+ <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Thu, 07 Jan 2016 21:50:01 -0000\r
+\r
+On Wed, Jan 06 2016, J Farkas <jf.hyqohaczlksw4tx6ae@l2015aftruuq.dns007.ne=\r
+t> wrote:\r
+\r
+> On 2016-01-05 at 17:41:06, David Bremner wrote:\r
+>> Tomi Ollila <tomi.ollila@iki.fi> writes:\r
+>>=20\r
+>> > The page https://sourceware.org/gdb/onlinedocs/gdb/Returning.html\r
+>> > talks something about gdb not knowing the return type if function\r
+>> > was compiled without debug info... well, is this is the reason,\r
+>> > perhaps we should allow testing w/o debug info compiled in.\r
+>>=20\r
+>> The test suite overall assumes debug info is present; T000-basic tests\r
+>> for this.\r
+>\r
+> I'm not missing the debug infos (well, except for the thread library if I\r
+> understand correctly - and that affects a lot of other tests, but not thi=\r
+s).\r
+> Perhaps it the version of gdb, but not any compilation flags that can be\r
+> changed for the notmuch tree.\r
+>\r
+> Without the changes, I'm getting these two kind of errors (and as you can\r
+> see, the test for the debugging symbols passes too).\r
+\r
+ack... keep reading...\r
+\r
+>\r
+> $ ./T000-basic.sh\r
+>\r
+> T000-basic: Testing the test framework itself.\r
+>  PASS   success is reported like this\r
+> ...\r
+>  PASS   PATH is set to build directory\r
+>  PASS   notmuch is compiled with debugging symbols\r
+> $ ./T070-insert.sh\r
+>\r
+> T070-insert: Testing "notmuch insert"\r
+>  PASS   Insert zero-length file\r
+> ...\r
+>  FAIL   Insert duplicate message\r
+>         --- T070-insert.6.expected      2016-01-06 13:31:01.644676102 +00=\r
+00\r
+>         +++ T070-insert.6.output        2016-01-06 13:31:01.648675891 +00=\r
+00\r
+>         @@ -1 +1 @@\r
+>         -2\r
+>         +      2\r
+\r
+This is trivial to fix w/ the same\r
+\r
+output=3D$((`notmuch search --output=3Dfiles "subject:insert-subject" | wc =\r
+-l`))\r
+\r
+construct that is used in T060-count.sh\r
+\r
+A separate patch doing just this would be appreciated.\r
+\r
+>  PASS   Duplicate message does not change tags\r
+>  PASS   Insert message, add tag\r
+> ...\r
+>  PASS   Tags starting with '-' in new.tags are forbidden\r
+>  PASS   Invalid tags set exit code\r
+>  FAIL   error exit when add_message returns OUT_OF_MEMORY\r
+>         --- T070-insert.26.expected     2016-01-06 13:31:06.940397013 +00=\r
+00\r
+>         +++ T070-insert.26.output       2016-01-06 13:31:06.940397013 +00=\r
+00\r
+>         @@ -1 +1 @@\r
+>         -1\r
+>         +255\r
+> warning: Unable to find libthread_db matching inferior's thread library, =\r
+thread debugging will not be available.\r
+> index-file-OUT_OF_MEMORY.gdb:7: Error in sourced command file:\r
+> Return value type not available for selected stack frame.\r
+> Please use an explicit cast of the value to return.\r
+>  FAIL   success exit with --keep when add_message returns OUT_OF_MEMORY\r
+\r
+Your setup is interesting -- you have GNU textutils 2.0.22, released\r
+2002-07-20 (i.e. 13.5 years ago) and then gdb 7.4. I just run this\r
+test file on Scientific Linux 6.2 (gcc 4.4.6 and gdb 7.2), and the\r
+tests passed without problems.\r
+\r
+Apart from a few special cases, no typecasts should be added lightly\r
+to the code -- be it at innocent-looking as it can be. Those impedes\r
+the possibility for compiler (etc.) to do type checking.\r
+\r
+So, in this case, if you're the only one seeing this problem I'd suggest\r
+you tolerate the problem (by using private patch) (you could also dig\r
+deeper if interested). If this "problem" is (were!) more widespread,\r
+*and* unfixable in reasonable manner this thing like this typecast\r
+could be applied -- which would also mean this test system would\r
+be a bit less robust as it is now...(*)\r
+\r
+\r
+Tomi\r
+\r
+(*) imo these are hard facts, but someone(tm) may consider these as mere\r
+opinions...\r
+\r
+\r
+> ...\r
+>\r
+> (The above is then repeated for all gdb using `insert` tests.)\r
+>\r
+> J=C3=A1nos\r