search

freelawproject / courts-db

A database of courts, tests and other experiments
BSD 2-Clause "Simplified" License
58 stars 15 forks source link

Rename Manifest.in to MANIFEST.in #35

Closed step21 closed 2 years ago

step21 commented 2 years ago

So as it wasn't fixed yet - turns out it has to be MANIFEST.in. I did test it locally (python setup.py sdist), but as I use a case-insensitive file system on my desktop machine, it worked fine. However when I run it on another machine with linux and a case sensitive file name, it failed. And then, renaming it did indeed fix it. So sorry for the inconvenience, but this should really work now.

mlissner commented 2 years ago

No prob. Windows strikes again!

step21 commented 2 years ago

(It was macOS in this case, but still works ;) )

mlissner commented 2 years ago

Huh. I thought Mac was case sensitive. Shows what I know.

step21 commented 2 years ago

It can be both. I do not know what the default is these days, but at least when I set this up the usual one was case-insensitive.

mlissner commented 2 years ago

Well, that's just terrifying.

step21 commented 2 years ago

Sorry for the repeated drama, but the file size of the source dist is still way too small, and predictable it is not in there. I even reviewed the github action you use and I do not get why it is not working :/ (commenting here instead of making a new issue for now, hope that is ok)