Skip to content

Update the bootstrap guide for tomllib#739

Merged
takluyver merged 4 commits intopypa:mainfrom
AA-Turner:bootstrap-tomllib
Mar 27, 2025
Merged

Update the bootstrap guide for tomllib#739
takluyver merged 4 commits intopypa:mainfrom
AA-Turner:bootstrap-tomllib

Conversation

@AA-Turner
Copy link
Contributor

No description provided.

Copy link
Member

@cdce8p cdce8p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems overly complicated to me. I'd assume that redistributors will need to check how tomli themselves anyway, so I don't see a particular need or improvement with this change.

Comment on lines 29 to 30
In Python 3.11 and newer, ``flit_core`` uses the standard library :mod:`tomllib`
module, meaning that ``flit_core`` has no third-party dependencies.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's vendored for all Python versions, just not used in >=3.11.

@takluyver
Copy link
Member

I think it is worth updating this document, because for Python 3.11+ the bootstrapping story is pretty easy; it's only downstreams that are still on 3.10 or older (and care about avoiding bundling) that need to figure out the dependency cycle one way or another. But we should make sure the wording is clear that tomli is bundled regardless of your Python version, but only used on older versions.

I think we can also get rid of the 'as of version 3.6' qualifier now; version 3.6 was more than 3 years ago, so that's just a distraction.

@AA-Turner
Copy link
Contributor Author

How's this rephrasing? It's a little more verbose but hopefully clearer.

A

Co-authored-by: Thomas Kluyver <takowl@gmail.com>
@takluyver takluyver added this to the 3.13 milestone Mar 27, 2025
@takluyver
Copy link
Member

Thanks, that looks fine to me. 👍

@takluyver takluyver merged commit a254b8a into pypa:main Mar 27, 2025
16 checks passed
@AA-Turner AA-Turner deleted the bootstrap-tomllib branch March 27, 2025 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants