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

Fix pip install errors. #726

Merged
merged 2 commits into from
Nov 4, 2020
Merged

Fix pip install errors. #726

merged 2 commits into from
Nov 4, 2020

Conversation

rchen152
Copy link
Contributor

@rchen152 rchen152 commented Nov 4, 2020

I was getting errors about cfg_logging.h being missing when trying to
test a new release; this change seems to fix the issue.

I'm still seeing some odd dynamic import failure, but python3.7 on my
machine also appears to be half-broken right now, so I have no idea if
the issue is with the pytype code or my environment.

I was getting errors about cfg_logging.h being missing when trying to
test a new release; this change seems to fix the issue.

I'm still seeing some odd dynamic import failure, but python3.7 on my
machine also appears to be half-broken right now, so I have no idea if
the issue is with the pytype code or my environment.
@rchen152 rchen152 requested a review from Solumin November 4, 2020 04:02
@google-cla google-cla bot added the cla: yes label Nov 4, 2020
For reasons mysterious to me, using Pybind11Extension leads to the error
"ImportError: dynamic module does not define module export function
(PyInit_parser_ext)", even if I set include_pybind11=False.
@rchen152
Copy link
Contributor Author

rchen152 commented Nov 4, 2020

I fixed my Python 3.7 installation, and the pytype install was still broken, so I've pushed a commit to also fix the dynamic import failure.

@rchen152 rchen152 changed the title Add pytype/typegraph/* to MANIFEST.in. Fix pip install errors. Nov 4, 2020
@rchen152 rchen152 merged commit 70215e1 into master Nov 4, 2020
@rchen152 rchen152 deleted the manifest branch November 4, 2020 23:32
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

Successfully merging this pull request may close these issues.

2 participants