Re: T050-new and T060-count fail if gdb prints a warning

Subject: Re: T050-new and T060-count fail if gdb prints a warning

Date: Wed, 11 Mar 2020 10:22:09 +0100

To: Tomi Ollila

Cc: notmuch@notmuchmail.org

From: Dan Čermák


Hi Tomi,

Tomi Ollila <tomi.ollila@iki.fi> writes:

> On Tue, Mar 10 2020, Dan Čermák wrote:
>
>> Hi list,
>>
>> I have encountered a test failure caused by the T050-new and T060-count
>> tests in the latest openSUSE-Tumbleweed rebuild with Python 3.8 instead
>> of Python 3.7. Both tests die with:
>>
>> --8<---------------cut here---------------start------------->8---
>> T050-new: Testing "notmuch new" in several variations
>>  FAIL   Handle files vanishing between scandir and add_file
>>        --- T050-new.36.EXPECTED        2020-03-10 12:49:34.388407058 +0000
>>        +++ T050-new.36.OUTPUT  2020-03-10 12:49:34.388407058 +0000
>>        @@ -1,3 +1,7 @@
>>        +/usr/share/gdb/python/gdb/command/prompt.py:48: SyntaxWarning: "is not" with a literal. Did you mean "!="?
>>        +  if self.value is not '':
>>        +/usr/share/gdb/python/gdb/command/prompt.py:60: SyntaxWarning: "is not" with a literal. Did you mean "!="?
>>        +  if self.value is not '':
>>         Unexpected error with file /home/abuild/rpmbuild/BUILD/notmuch-0.29.3/test/tmp.T050-new/mail/vanish
>>         add_file: Something went wrong trying to read or write a file
>>         Error opening /home/abuild/rpmbuild/BUILD/notmuch-0.29.3/test/tmp.T050-new/mail/vanish: No such file or directory
>> --8<---------------cut here---------------end--------------->8---
>>
>> which I think is caused by gdb 8 using some odd Python syntax and is
>> completely unrelated to notmuch.
>>
>> The solution is of course to fix this in gdb (e.g. by updating to gdb
>> 9), but the more general question is: can notmuch's testsuite do
>> something to avoid issues like this altogether?
>
> setting PYTHONWARNINGS environment variable could probably do it.

TIL about this environment variable! Thanks a lot for the hint, that
fixed the build in openSUSE and is a far better temporary workaround
than skipping two tests.

>
> IIRC the -count test is something that is harder to replace with
> LD_PRELOAD shim...
>
> (btw: thanks for this report -- i learned python3 -bb option
> (been hit comparing strings and bytes with == ...)

Well, thanks for sharing this too, thereby I learned another thing today ;-)


Cheers,

Dan
signature.asc (application/pgp-signature)
_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch

Thread: