-
Notifications
You must be signed in to change notification settings - Fork 611
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
fake_libc_include not included in distributed version of library #224
Comments
s/consider/consider accepting/ |
It's listed here: https://github.com/eliben/pycparser/blob/master/MANIFEST.in#L4 And I do see it in the distribution tarball -- are you sure pip doesn't install it? |
I'm pretty sure... I've
I'm using pip 9.0.1 and Python 3.6.0:
This SO answer suggests that pip ignores top-level "non-package" directories, and that a solution would be to move the |
But moving it would break compatibility for projects that rely on it being there :) I'd say I'm not too excited about making this change, to be completely honest. It's easy enough to distribute this directory with pycparser in some way, I don't know if I'd force it to be installed since it's not strictly necessary for the package (pycparser really wants to see preprocessed source code -- this dir is used by the preprocessor, not pycparser) |
Sounds reasonable! Thanks for taking the time to respond. |
If I submit a Pull Request for this feature, would you be willing to add it? I feel like I can add it without breaking anything since its not there to begin with. You'd have to make additions to setup.py as my understanding, the |
@drebbe-intrepid I think I prefer not installing it. pycparser itself doesn't need or use it. |
For anyone still strugling I found this to work for me:
Hope this helps. Also this only worked on Linux (Mac C libraries seemed to conflict |
I don't know what A quick solution for me was this: |
The
fake_libc_include
directory is not included in the version of the library installed from PyPI with pip.This means that I cannot do something like:
Instead I have to vendor
fake_libc_include
(which is not the end of the world!).Would you consider a patch to fix this?
The text was updated successfully, but these errors were encountered: