GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-158001: Remove global configuration variables from C API by vstinner · Pull Request #158020 · python/cpython

Conversation

@vstinner

@vstinner vstinner commented Sep 23, 2026 •
edited by bedevere-app Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

Keep 4 variables in the stable ABI:

  • Py_FileSystemDefaultEncodeErrors
  • Py_FileSystemDefaultEncoding
  • Py_HasFileSystemDefaultEncoding
  • Py_UTF8Mode
  • Issue: C API: Remove deprecated API to configure Python initialization #158001

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@vstinner
pythongh-158001: Remove global configuration variables from C API
eb053c3
Keep 4 variables in the stable ABI:

* Py_FileSystemDefaultEncodeErrors
* Py_FileSystemDefaultEncoding
* Py_HasFileSystemDefaultEncoding
* Py_UTF8Mode
@vstinner
vstinner requested review from AA-Turner, FFY00, ZeroIntensity and ericsnowcurrently as code owners September 23, 2026 19:32
@bedevere-app bedevere-app Bot mentioned this pull request Sep 23, 2026
C API: Remove deprecated API to configure Python initialization #158001
Closed
@bedevere-app bedevere-app Bot added the awaiting core review label Sep 23, 2026
@vstinner

vstinner commented Sep 23, 2026

Copy link
Copy Markdown
Member Author

This PR is based on #158014 which prepared the code base for this large removal.

I leave Doc/deprecations/c-api-pending-removal-in-3.16.rst unchanged, since this file is include by What's New in Python 3.12/13/14.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@read-the-docs-community

read-the-docs-community Bot commented Sep 23, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #34736919 | 📁 Comparing dcfba00 against main (2f5791d)

  🔍 Preview build  

8 files changed · ± 8 modified

± Modified

  • builtins/functions.html
  • c-api/interp-lifecycle.html
  • c-api/stable.html
  • c-api/threads.html
  • library/code.html
  • library/symtable.html
  • whatsnew/3.16.html
  • whatsnew/changelog.html
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

vstinner added 4 commits September 23, 2026 21:49
@vstinner
Update tests
dbe5cae
* test_capi.test_config: Remove test_config_set_global_vars().
* test_ctypes: Get Py_Version variable instead of Py_OptimizeFlag.
@vstinner
Update make check-limited-abi
42963a7
Mark the 4 variables as abi_only in Misc/stable_abi.toml.
@vstinner
doc: Remove links to removed variables
316cb16
@vstinner
Try to fix build on Windows
27cbdd6
@vstinner
vstinner requested a review from encukou as a code owner September 23, 2026 20:49
vstinner added 2 commits September 23, 2026 22:58
@vstinner
Update Doc/tools/removed-ids.txt
8b1af46
@vstinner
Fix make check-c-globals
301c87c
@vstinner vstinner added the topic-C-API label Sep 23, 2026
vstinner added 3 commits September 24, 2026 12:54
@vstinner
Merge branch 'main' into remove_global_config
ef87ce6
@vstinner
Update doc to point to PyInitConfig
46fa563
@vstinner
Update comment
dcfba00
@vstinner
vstinner merged commit 7242fea into python:main Sep 24, 2026
55 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@vstinner
vstinner deleted the remove_global_config branch September 24, 2026 14:42
@bedevere-app bedevere-app Bot removed the awaiting core review label Sep 24, 2026
@hugovk

hugovk commented Sep 25, 2026

Copy link
Copy Markdown
Member

Looks like this might be causing s390x buildbots to fail.

vstinner reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vstinner

vstinner commented Sep 25, 2026

Copy link
Copy Markdown
Member Author

Looks like this might be causing s390x buildbots to fail.

Ooops, correct. I introduced a bug in test_ctypes.test_version_var() which is visible on big endian machines such as s390x.

I wrote PR gh-158193 to fix test_ctypes.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ozeranskii ozeranskii mentioned this pull request Oct 18, 2026
Avoid removed Py_FileSystemDefaultEncoding in load_library python-cffi/cffi#287
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericsnowcurrently ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently is a code owner
@FFY00 FFY00 Awaiting requested review from FFY00 FFY00 is a code owner
@AA-Turner AA-Turner Awaiting requested review from AA-Turner AA-Turner is a code owner
@ZeroIntensity ZeroIntensity Awaiting requested review from ZeroIntensity ZeroIntensity is a code owner
@encukou encukou Awaiting requested review from encukou encukou is a code owner

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

topic-C-API

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@vstinner @hugovk
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.