Skip to content
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

1.5.x sdist no longer contains examples, docs, and tests #285

Closed
tiran opened this issue Aug 15, 2022 · 5 comments · Fixed by #291
Closed

1.5.x sdist no longer contains examples, docs, and tests #285

tiran opened this issue Aug 15, 2022 · 5 comments · Fixed by #291

Comments

@tiran
Copy link
Contributor

tiran commented Aug 15, 2022

The source distributions for 1.5.0 and 1.5.1 no longer contain docs, examples, and tests directories. The tests and examples are useful for downstream packagers. For example I use them in Fedora packages to verify that setuptools-rust is working correctly.

Could you please re-add them? I'm not sure why they are now omitted from sdist. You may have to add the directories to MANIFEST.in.

@davidhewitt
Copy link
Member

Ungh, must be something different in the build environment. I'm prioritising the PyO3 0.17 release first, I'll investigate this in a week or two and push 1.5.3. If you need it sooner a PR to help things along is appreciated 🙂

@tiran
Copy link
Contributor Author

tiran commented Aug 17, 2022

Which MANIFEST.in do you prefer,

graft examples
recursive-include tests *.py

or

recursive-include examples *.py *.rs *.toml *.ini *.cfg *.sh *.lock *.in *.txt *.html *.md *.yml Dockerfile
recursive-include tests *.py

?

@davidhewitt
Copy link
Member

Probably better to go for recursive-include so that we don't accidentally ship build products etc? Has the downside of needing to opt-in to example contents though.

@davidhewitt
Copy link
Member

I'm not sure why they are now omitted from sdist.

On further reading, I think what happened is we stopped using setuptools_scm - I'll see if I can make it used only in sdist step.

@davidhewitt davidhewitt linked a pull request Sep 19, 2022 that will close this issue
@davidhewitt
Copy link
Member

Yes, it looks like just installing setuptools-scm for the sdist step is enough to fix the sdist. Thanks for reporting, I'll push 1.5.2 later today.

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 a pull request may close this issue.

2 participants