Implement missing METADATA parsing via pkginfo #198
Conversation
@brandjon Can you take a look? I think this patch is an important fix. |
Hi @brandjon. Please let us know if this project is maintained, or we should move to something else. There are multiple PRs and issues that are not answered for months. Importing pip packages doesn't work, using python3 is even in a worst state. |
Any updates on this pull request? Using Bazel in conjunction with python3/pip seems pretty painful without this. @brandjon |
I second @cgordoncarroll. Is there a way we could help to push this PR forward? |
Would be very interested in this being merged as well. Trying to use Bazel with Python3 deps. |
Have you tried any of these?
They all support Python 3. We are using the 1st with Python 3.6 at work. It was created after |
@thundergolfer Thanks I ended up using https://github.com/dillon-giacoppo/rules_python_external and it works perfectly. |
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. |
@googlebot I fixed it. |
CLAs look good, thanks! |
|
My only context for this code change is a close reading of the diff (unfortunately |
Thanks a lot @andyscott! Would you like me to do anything else here? (The build of your merge failed because of an unrelated issue already fixed at head, afaict). |
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. |
CLAs look good, thanks! |
Hi! @thundergolfer @andyscott I removed the .par files from the PR, and squashed the commits. Is there anything else I should do here? Thank you! |
Hi |
Replaces the incomplete METADATA parsing with pkginfo. Adds tests using real-world packages.
Fixes #102