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.

Author Arfrever
Recipients Arfrever, alecsandru.patrascu, brett.cannon, python-dev
Date 2015-10-11.12:42:11
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1444567331.49.0.621529280618.issue25188@psf.upfronthosting.co.za>
In-reply-to
Content
> New changeset 136ad559fa4f by Brett Cannon in branch '2.7':
> Issue #25188: Add -P/--pgo to test.regrtest for PGO building.
> https://hg.python.org/cpython/rev/136ad559fa4f
> 
> +                    # required to permit running tests with PGO flag on/off
> +                    if pgo:
> +                        args_tuple[1]['pgo']=pgo

This looks ugly.
The following change should work and would be consistent with handling of huntrleaks and use_resources:

--- Lib/test/regrtest.py
+++ Lib/test/regrtest.py
@@ -508,7 +508,8 @@
             for test in tests:
                 args_tuple = (
                     (test, verbose, quiet),
-                    dict(huntrleaks=huntrleaks, use_resources=use_resources)
+                    dict(huntrleaks=huntrleaks, use_resources=use_resources,
+                         pgo=pgo)
                 )
                 yield (test, args_tuple)
         pending = tests_and_args()
@@ -526,9 +527,6 @@
                     except StopIteration:
                         output.put((None, None, None, None))
                         return
-                    # required to permit running tests with PGO flag on/off
-                    if pgo:
-                        args_tuple[1]['pgo']=pgo
                     # -E is needed by some tests, e.g. test_import
                     popen = Popen(base_cmd + ['--slaveargs', json.dumps(args_tuple)],
                                    stdout=PIPE, stderr=PIPE,
History
Date User Action Args
2015-10-11 12:42:11Arfreversetrecipients: + Arfrever, brett.cannon, python-dev, alecsandru.patrascu
2015-10-11 12:42:11Arfreversetmessageid: <1444567331.49.0.621529280618.issue25188@psf.upfronthosting.co.za>
2015-10-11 12:42:11Arfreverlinkissue25188 messages
2015-10-11 12:42:11Arfrevercreate