Title: gh-128690: temporarily do not use test_embed in PGO profile builds by ned-deily · Pull Request #128771 · python/cpython · GitHub
Open Graph Title: gh-128690: temporarily do not use test_embed in PGO profile builds by ned-deily · Pull Request #128771 · python/cpython
X Title: gh-128690: temporarily do not use test_embed in PGO profile builds by ned-deily · Pull Request #128771 · python/cpython
Description: As documented in #128690, under certain configurations test case test_init_pyvenv_cfg of test_embed fails apparently since 34e840f. This behavior was not seen in 3.14.0a3, the most recent alpha before this change. Unfortunately, test_embed is included in the subset of tests used for PGO profile builds when Python is built with --enable-optimizations. If any of the tests included in the PGO profile build fails, including test_embed, the whole Python build fails. Besides the originally reported case, I have now seen this failure in two other macOS build configurations, both rather complex; one is for the build of the python.org installer package. (At the moment, I haven't had time to produce a manageable reproducible test case.) It could be that this is just an existing test case issue that was brought to the fore by the above change. Since the failure involves sys._base_executable, it's possible that the behavior documented in #128670 might be the root cause. If necessary for 3.14.0a4, I could apply this PR manually to avoid using test_embed in the profile builds. But I would prefer to either have the test failure resolved prior to a4 or merge this PR to temporarily avoid using test_embed. Marking as release-blocker for release manager decision. cc: @hugovk @FFY00 Issue: gh-128690
Open Graph Description: As documented in #128690, under certain configurations test case test_init_pyvenv_cfg of test_embed fails apparently since 34e840f. This behavior was not seen in 3.14.0a3, the most recent alpha bef...
X Description: As documented in #128690, under certain configurations test case test_init_pyvenv_cfg of test_embed fails apparently since 34e840f. This behavior was not seen in 3.14.0a3, the most recent alpha bef...
Opengraph URL: https://github.com/python/cpython/pull/128771
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:e8dade10-bf3d-66cf-387a-0885430af3da |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | E89E:13DDC4:A2B23D6:E486328:6A50DCB5 |
| html-safe-nonce | 499ba04deaa5e0c0d4a9c3b4d55d832dba59d73b99617cbdcfd06ad37515de7f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFODlFOjEzRERDNDpBMkIyM0Q2OkU0ODYzMjg6NkE1MERDQjUiLCJ2aXNpdG9yX2lkIjoiNzYwNzA1MzAzNDg1ODc5ODI2MSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 8cc6d71e9b29e32b7e19e96ee5d0750fd2c0ac3efc34cd7df65063029cb76cab |
| hovercard-subject-tag | pull_request:2273238957 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/128771/files |
| twitter:image | https://avatars.githubusercontent.com/u/5833005?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/5833005?s=400&v=4 |
| og:image:alt | As documented in #128690, under certain configurations test case test_init_pyvenv_cfg of test_embed fails apparently since 34e840f. This behavior was not seen in 3.14.0a3, the most recent alpha bef... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 72b13e0e8d0319acdd27a145cfe73f4f06c791713d0ac4690e41fb68ad28cac0 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 17290b65caabb92b271e0949d1bf147b63deab41 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width