Title: bpo-29585: Fix sysconfig.get_config_var("PYTHONFRAMEWORK") by methane · Pull Request #2483 · python/cpython · GitHub
Open Graph Title: bpo-29585: Fix sysconfig.get_config_var("PYTHONFRAMEWORK") by methane · Pull Request #2483 · python/cpython
X Title: bpo-29585: Fix sysconfig.get_config_var("PYTHONFRAMEWORK") by methane · Pull Request #2483 · python/cpython
Description: After GH-136 merged, PYTHONFRAMEWORK config variable are in Makefile and pyconfig.h. When reading string variable from Makefile, quote is removed. But when reading them from pyconfig.h, quote is not removed. And config variables in pyconfig.h override config variables in Makefile. So sysconfig.get_config_var("PYTHONFRAMEWORK") was changed '' to '""' in non framework build, and 'PYTHON' to '"PYTHON"' in framework build. I thought about unquoting string config variables in pyconfig.h, but there is one string variables already and it may cause unintentional backward incompatibility. (sysconfig.get_config_var('PY_FORMAT_SIZE_T') is '"z"'). So this pull request just rename PYTHONFRAMEWORK to _PYTHONFRAMEWORK in pyconfig.h. Variables which name starts with _ is not exported to sysconfig.
Open Graph Description: After GH-136 merged, PYTHONFRAMEWORK config variable are in Makefile and pyconfig.h. When reading string variable from Makefile, quote is removed. But when reading them from pyconfig.h, quote is no...
X Description: After GH-136 merged, PYTHONFRAMEWORK config variable are in Makefile and pyconfig.h. When reading string variable from Makefile, quote is removed. But when reading them from pyconfig.h, quote is no...
Opengraph URL: https://github.com/python/cpython/pull/2483
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/checks(.:format) |
| route-controller | pull_requests |
| route-action | checks |
| fetch-nonce | v2:3351a035-2225-9ee7-2bc2-e79f39074d8b |
| current-catalog-service-hash | 87dc3bc62d9b466312751bfd5f889726f4f1337bdff4e8be7da7c93d6c00a25a |
| request-id | E9C8:21E6C4:79D4EA:A9B72C:696A2AB9 |
| html-safe-nonce | 8e2bdae2cc86410841e6b6e14d5378aa92e84949eb652bb2535ccb0fc5a1b4b2 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFOUM4OjIxRTZDNDo3OUQ0RUE6QTlCNzJDOjY5NkEyQUI5IiwidmlzaXRvcl9pZCI6IjcxNDYyNzE4Nzk0MjM3MzI0MDkiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 0f3a523c05fc05bfafe4b0f395b65143110d5a4e6a8c7f095c718325cef5e393 |
| hovercard-subject-tag | pull_request:128074538 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,checks,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/2483/checks |
| twitter:image | https://avatars.githubusercontent.com/u/199592?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/199592?s=400&v=4 |
| og:image:alt | After GH-136 merged, PYTHONFRAMEWORK config variable are in Makefile and pyconfig.h. When reading string variable from Makefile, quote is removed. But when reading them from pyconfig.h, quote is no... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | a1022f03e4f0d91ea173e4e5dac892c982e0588c62f1ce56121d755a320a3569 |
| turbo-cache-control | no-preview |
| 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 full-width-p-0 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | f472b8e6c7b3fdd5d0354972a3f4c516289bf0be |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width