search

SimplyKnownAsG / yamlize

Python YAML serializing library
Apache License 2.0
11 stars 5 forks source link

LICENSE not included in PyPI sdist #7

Closed jayvdb closed 2 years ago

jayvdb commented 3 years ago

The PyPI artifacts needs to include the LICENSE. Recent setuptools will include this automatically. Otherwise a MANIFEST.in is needed.

jayvdb commented 3 years ago

btw, https://build.opensuse.org/package/show/home:jayvdb:py-new/python-yamlize created

SimplyKnownAsG commented 3 years ago

Sorry, I'm a bit naive on this. Does this mean I just need to update setuptools before building/releasing? I haven't really liked the MANIFEST thing. Any opinions of your own are welcome!

jayvdb commented 3 years ago

You can use a more recent setuptools, and it should do the trick, but IMO explicit (MANIFEST.in) is better than implicit. If using a recent setuptools works for you, add setup_requires=['setuptools>=..'] to avoid regressions.

SimplyKnownAsG commented 2 years ago

I think this is done now.