Skip to content

Conversation

@davidhewitt
Copy link
Member

Following up from #5343

This updates documentation across the codebase to discourage use of the extension-module feature and to instead rely on the PYO3_BUILD_EXTENSION_MODULE environment variable when relevant.

Comment on lines 74 to 76
PyO3 has some Cargo features to configure projects for building Python extension modules:

- The `extension-module` feature, which must be enabled when building Python extension modules.
- The `PYO3_BUILD_EXTENSION_MODULE` environment variable, which must be set when building Python extension modules.
Copy link
Contributor

Choose a reason for hiding this comment

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

The bullet doesn't match the intro :-)

It's probably worth saying that people don't generally need to set this themselves, because maturin/setuptools-rust set it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, done in 7e8882e and 2e65a8f

@davidhewitt davidhewitt enabled auto-merge November 8, 2025 18:04
@davidhewitt davidhewitt added this pull request to the merge queue Nov 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 8, 2025
@davidhewitt
Copy link
Member Author

Looks like this failed to merge because ffi-check is linking when it doesn't need to; I'll try to rework that as a separate step first.

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.

3 participants