[issue18492] Allow all resources if not running under regrtest.py

Terry J. Reedy added the comment: Just in time, as I will be reviewing and committing new Idle unittest module starting this week (new GSOC student). -- ___ Python tracker ___ _

[issue18492] Allow all resources if not running under regrtest.py

Zachary Ware added the comment: You convinced me too, Serhiy :). Committed, without the regrtest_run flag. Thanks for review. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue18492] Allow all resources if not running under regrtest.py

Roundup Robot added the comment: New changeset eabff2a97b5b by Zachary Ware in branch '2.7': Issue #18492: Allow all resources when tests are not run by regrtest.py. http://hg.python.org/cpython/rev/eabff2a97b5b New changeset 8519576b0dc5 by Zachary Ware in branch '3.4': Issue #18492: Allow all

[issue18492] Allow all resources if not running under regrtest.py

Changes by Serhiy Storchaka : -- assignee: -> zach.ware ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:

[issue18492] Allow all resources if not running under regrtest.py

Serhiy Storchaka added the comment: You have convinced me. In general the approach and the patch LGTM. But I agree with Terry that flag support.regrtest_run is redundant, it duplicates a bit of information which provides support.use_resources. Instead we can rewrite support.is_resource_enabled

[issue18492] Allow all resources if not running under regrtest.py

Zachary Ware added the comment: Here's a new and better patch. This patch keeps the regrtest_run global in support and moves the regrtest-or-not check into is_resource_enabled to make is_resource_enabled, requires, and requires_resource consistent with each other and in a way that still allow