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

Wheel tool doesn't resolve whl for some versions #19

Open
samwestmoreland opened this issue Oct 12, 2021 · 3 comments · Fixed by #21
Open

Wheel tool doesn't resolve whl for some versions #19

samwestmoreland opened this issue Oct 12, 2021 · 3 comments · Fixed by #21
Assignees

Comments

@samwestmoreland
Copy link
Contributor

E.g. it's unable to find any wheels for version 3.12.2 of protobuf, even though it looks on PyPI like there are wheels available.

python_wheel(
     name = "protobuf",
     outs = ["google"],
-    version = "3.12.2",
+    version = "3.18.1",
     deps = [":six"],
 )
@samwestmoreland samwestmoreland self-assigned this Oct 12, 2021
@samwestmoreland
Copy link
Contributor Author

This seems to only be an issue with protobuf - distlib doesn't pick up any urls for any version before 3.18.1. I can't reproduce with any other libraries however (tried tensorflow and cryptography), so for now I'll say this is just something going on between distlib and protobuf and not our concern. And I guess we should update the version of protobuf we're using in the plugin just to be safe.

@tiagovtristao
Copy link
Contributor

Interesting... So distlib can't find older versions of protobuf, did you find anything online about this? Do you know what might be the issue?

@samwestmoreland
Copy link
Contributor Author

This appears to still be an issue. @tiagovtristao noticed a problem in the last couple of days with the wheel resolver tool not able to find wheels in the index for protobuf 3.18.1, and bumping the version again to 3.19.3 fixed it. So if this is going to keep happening we probably ought to get to the bottom of it.

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 a pull request may close this issue.

2 participants