-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Pruning manifest to those items we need #1656
Conversation
@hugovk I've looked at the output file manifest from this change, and I think it's actually correct. Check-manifest is freaking out, but I don't think it's actually calculating the files correctly. Can you take a look at this? |
I also see a discrepancy between what For example, the former complains Tests/*.py are missing, but the latter includes them. Python 2.7.10 With this branch, MANIFEST.in contains:
I get this output locally:
Running
and Pillow.egg-info/SOURCES.txt contains:
|
Reported: mgedmin/check-manifest#65 |
I could have sworn that the tests were actually included, but looking at Pillow.egg-info/SOURCES.txt, (edit -- after a clean checkout and running make sdist) they weren't there. I've pushed an update for that, and am going to rebase to catch the recently added added lines. |
I've force pushed the rebase, so now anything that the original issue w/ check-manifest is going to be off in the ether. |
155b0f1
to
e2f7652
Compare
Right, and now rebased to fix merge errors from the changes I just merged to master. |
Pruning manifest to those items we need
Pruning manifest to those items we need and in larger chunks to reduce thrashing.
However. Check-manifest barfs mightily, and complains that there are a hell of a long list of things that aren't included. If you check in the actual generated manifest,
SOURCES.txt
, they are actually there, or intentionally ignored (like the source control cruft). Adding the intentional ignores back in causes warnings from the manifest generation. I'd prefer to fix or ignore check-manifest, because it's clearly wrong.