Skip to content
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

--create and --update exit 1 when version control and sdist do not match #82

Open
wmedlar opened this issue Apr 5, 2017 · 2 comments
Open

Comments

@wmedlar
Copy link

wmedlar commented Apr 5, 2017

For example, with files tracked in VCS (git) and no manifest:

~/package $ check-manifest --create
lists of files in version control and sdist do not match!
missing from sdist:
  LICENSE.txt
  Makefile
  requirements.txt
  tests
  tests/test_a.py
  tests/test_b.py
no MANIFEST.in found
suggested MANIFEST.in rules:
  include *.txt
  include Makefile
  recursive-include tests *.py
creating MANIFEST.in
~/package $ echo $?
1

or when updating the manifest:

~/package $ check-manifest --update
lists of files in version control and sdist do not match!
missing from sdist:
  Makefile
suggested MANIFEST.in rules:
  include Makefile
updating MANIFEST.in
~/package $ echo $?
1

I would expect these flags to cause the program to exit 0, indicating a successful creation/update.

@mgedmin
Copy link
Owner

mgedmin commented Apr 5, 2017

That makes sense, as long as check-manifest didn't encounter any files it doesn't know how to add.

I'm not likely to get around to this soon (new baby in the house), but feel free to ping me in a month or so. Or you could provide a pull request, if you've time.

@wmedlar
Copy link
Author

wmedlar commented Apr 15, 2017

Congratulations! 😄 I'm sure I'll be able to find the time to work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants