search

wjakob / nanobind

nanobind: tiny and efficient C++/Python bindings
BSD 3-Clause "New" or "Revised" License
2.14k stars 161 forks source link

fix: add MANIFEST.in for SDist generation #619

Open henryiii opened 2 weeks ago

henryiii commented 2 weeks ago

This is missing. Mostly generated automatically by running pipx run check-manifest -c, then minor touchup to make pipx run check-manifest pass. (I used embarrassingly used check-manifest instead of check-sdist, which is my tool that works with all backends, due to the fact it can generate a MANIFEST.in due to it's setuptools integration and I'm lazy. Don't tell anyone ;) ).

This fixes pipx run build, and building SDists and wheels from SDists in general.

This makes the code saved in #618 even bigger. :D

henryiii commented 2 weeks ago

PS: I'd normally add check-manifest or check-sdist into your pre-commit (check-sdist is designed to be fast enough to do this!) to make sure it never misses anything, but you don't have one.