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

Directives in MANIFEST.in may cause pip install to fail #184

Closed
thunderk opened this issue Feb 2, 2017 · 5 comments
Closed

Directives in MANIFEST.in may cause pip install to fail #184

thunderk opened this issue Feb 2, 2017 · 5 comments

Comments

@thunderk
Copy link

thunderk commented Feb 2, 2017

In MANIFEST.in file, there are directives about a "debian" folder that is not present. In some python/setuptools configuration, this leads to an error on pip install.

Quick test in a docker:
`
$ docker run --rm -it python:3.4 pip install nodeenv
Collecting nodeenv
Downloading nodeenv-1.1.1.tar.gz
Complete output from command python setup.py egg_info:
running egg_info
creating pip-egg-info/nodeenv.egg-info
writing pip-egg-info/nodeenv.egg-info/PKG-INFO
writing entry points to pip-egg-info/nodeenv.egg-info/entry_points.txt
writing dependency_links to pip-egg-info/nodeenv.egg-info/dependency_links.txt
writing top-level names to pip-egg-info/nodeenv.egg-info/top_level.txt
writing manifest file 'pip-egg-info/nodeenv.egg-info/SOURCES.txt'
warning: manifest_maker: standard file '-c' not found

reading manifest file 'pip-egg-info/nodeenv.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
error: [Errno 2] No such file or directory: 'debian'

`

@thunderk
Copy link
Author

thunderk commented Feb 2, 2017

Thank you for the quick fix !
But there are other lines with "debian", shouldn't they be changed too ? (prune and recursive-exclude)

ekalinin added a commit that referenced this issue Feb 2, 2017
@ekalinin
Copy link
Owner

ekalinin commented Feb 2, 2017

Oops. Sorry. Fixed all rows :)

@thunderk
Copy link
Author

thunderk commented Feb 3, 2017

Thanks !
Could you please make a new release with the fix ?

@ekalinin
Copy link
Owner

ekalinin commented Feb 3, 2017

@thunderk
Copy link
Author

thunderk commented Feb 3, 2017

Nice, thanks :)

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

No branches or pull requests

2 participants