Skip to content

Adjust LANG note, drop LANG for 3.13+ #895

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

yosifkit
Copy link
Member

@yosifkit yosifkit commented Dec 21, 2023

This is mostly to update the LANG note to clarify why it still cannot be removed. The also reattempts #570 but in a much more limited way (only 3.13+). 3.13 is still only an alpha so it should be safer to change there.

Happy to not remove LANG in 3.13+ if we only want to update the note. This can wait until the new year so that any users testing 3.13.0a2 don't break over the holidays.

 Fixes #887

@tianon tianon merged commit 1b7a110 into docker-library:master Mar 21, 2024
@tianon tianon deleted the utf-note branch March 21, 2024 23:32
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Mar 22, 2024
Changes:

- docker-library/python@1b7a110: Merge pull request docker-library/python#895 from infosiftr/utf-note
- docker-library/python@656d5af: Merge pull request docker-library/python#914 from infosiftr/3.10-key
- docker-library/python@44def75: Swap 3.10 release key back to Pablo
martin-g pushed a commit to martin-g/docker-official-images that referenced this pull request Apr 3, 2024
Changes:

- docker-library/python@1b7a110: Merge pull request docker-library/python#895 from infosiftr/utf-note
- docker-library/python@656d5af: Merge pull request docker-library/python#914 from infosiftr/3.10-key
- docker-library/python@44def75: Swap 3.10 release key back to Pablo
edmorley added a commit to heroku/buildpacks-python that referenced this pull request Dec 16, 2024
Previously the buildpack would set the `LANG` env var at build
and run time to the value `C.UTF-8`, to match the official Docker
Python images.

However, the env var should not be needed for modern Python,
and so the official Docker Python images no longer set it:
docker-library/python#887
docker-library/python#895

In addition, the older issues typically only affected other distros
such as alpine, or environments where someone had overridden
the system locale, which doesn't apply to the Heroku base images.
edmorley added a commit to heroku/buildpacks-python that referenced this pull request Dec 16, 2024
Previously the buildpack would set the `LANG` env var at build
and run time to the value `C.UTF-8`, to match the official Docker
Python images.

However, the env var should not be needed for modern Python,
and so the official Docker Python images no longer set it:
docker-library/python#887
docker-library/python#895

In addition, the older issues typically only affected other distros
such as alpine, or environments where someone had overridden
the system locale, which doesn't apply to the Heroku base images.

GUS-W-17443565.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LANG C.UTF-8 issue status?
2 participants