Skip to content

Don't include the cargo target directory or cache directories in wheels built from sdists - #3320

Merged
messense merged 10 commits into
PyO3:mainfrom
ngoldbaum:gh-3319
Oct 8, 2026
Merged

messense merged 10 commits into
PyO3:mainfrom
ngoldbaum:gh-3319

Conversation

@ngoldbaum

Copy link
Copy Markdown
Contributor

Fixes gh-3319.

@ngoldbaum ngoldbaum changed the title Don't include the cargo target directory in wheels built from sdists Don't include the cargo target directory or cache directories in wheels built from sdists Oct 2, 2026
Comment thread tests/common/other.rs
@ngoldbaum

Copy link
Copy Markdown
Contributor Author

There's a version of this PR that leaves the CACHEDIR.TAG stuff out. I included it because it fixes a couple other versions of this bug found by an AI model.

@messense
messense self-requested a review October 8, 2026 07:31
@messense
messense merged commit 62a94db into PyO3:main Oct 8, 2026
45 checks passed
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.

Wheels built from sdists can include the Rust target directory

2 participants