setuptools-rust icon indicating copy to clipboard operation
setuptools-rust copied to clipboard

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

Open tiran opened this issue 3 years ago • 1 comments

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.

tiran avatar Aug 15 '22 08:08 tiran

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 🙂

davidhewitt avatar Aug 16 '22 07:08 davidhewitt

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

?

tiran avatar Aug 17 '22 07:08 tiran

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 avatar Aug 17 '22 07:08 davidhewitt

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 avatar Sep 18 '22 10:09 davidhewitt

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.

davidhewitt avatar Sep 19 '22 12:09 davidhewitt