This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Always run regrtest.py with -bb
Type: enhancement Stage:
Components: Tests Versions: Python 3.1, Python 3.2
process
Status: closed Resolution: wont fix
Dependencies: Superseder:
Assigned To: Nosy List: flox, r.david.murray, vstinner
Priority: normal Keywords: patch

Created on 2010-04-28 23:17 by vstinner, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
regrtest_bb.patch vstinner, 2010-04-28 23:16
Messages (4)
msg104474 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2010-04-28 23:16
-bb option helps debug because it detects errors earlier (comparing byte and unicode strings looks strange, it should be a bug).

Attached patch re-exec regrtest.py with -bb if -bb was not used.
msg104480 - (view) Author: Florent Xicluna (flox) * (Python committer) Date: 2010-04-29 07:02
+1 to force -bb on buildbots
-1 to hack regrtest.

IMHO it should be added to ./Makefile.pre.in and PCbuild/rt.bat, like other warning switches ( "-Wd" "-tt" "-3").
msg104481 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2010-04-29 08:09
> +1 to force -bb on buildbots

I think that it's already enabled, but I'm not sure.

> -1 to hack regrtest

Does someone use runtests.sh? We should maybe improve this script instead of regrtest.py.
msg104536 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2010-04-29 16:43
There are currently (at least) two places where regrtest is invoked by the buildbots, one for unix and one for windows.  The flags used are not always in sync, but probably should be.

I think it is probably better to not hack regrtest for this, since that would actually reduce its flexibility, but I'm open to argument on that. I'm going to set myself up an alias for running the tests:

   alias pytest="./python -E -tt -bb -Wd -3 -m test.regrtest"

That form for running regrtest should probably be added to the dev docs as a recommendation.
History
Date User Action Args
2022-04-11 14:57:00adminsetgithub: 52811
2010-04-30 16:15:26vstinnersetstatus: open -> closed
resolution: wont fix
2010-04-29 16:43:58r.david.murraysetnosy: + r.david.murray
messages: + msg104536
2010-04-29 08:09:20vstinnersetmessages: + msg104481
2010-04-29 07:02:02floxsettype: enhancement

messages: + msg104480
nosy: + flox
2010-04-28 23:17:00vstinnercreate