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

vcrpy 1.13 #735

Closed
rapgro opened this issue Nov 1, 2018 · 3 comments · Fixed by #736
Closed

vcrpy 1.13 #735

rapgro opened this issue Nov 1, 2018 · 3 comments · Fixed by #736

Comments

@rapgro
Copy link

rapgro commented Nov 1, 2018

We've python-vcrpy 1.13 in Fedora 29 and have python3-vcrpy as an enforced build dependency.
Try to start tests fail with an error:

+ cd pyvmomi-6.7.1
+ /usr/bin/python3 setup.py test
running test
Searching for vcrpy==1.12.0
Reading https://pypi.org/simple/vcrpy/
Download error on https://pypi.org/simple/vcrpy/: [Errno -2] Name or service not known -- Some packages may not be found!
Couldn't retrieve index page for 'vcrpy'
Scanning index of all packages (this may take a while)
Reading https://pypi.org/simple/
Download error on https://pypi.org/simple/: [Errno -2] Name or service not known -- Some packages may not be found!
No local packages or working download links found for vcrpy==1.12.0
error: Could not find suitable distribution for Requirement.parse('vcrpy==1.12.0')
@rapgro
Copy link
Author

rapgro commented Nov 1, 2018

Further, when try to build with python 2.7:

…
  File "/usr/lib/python2.7/site-packages/pkg_resources.py", line 630, in resolve
    raise VersionConflict(dist,req) # XXX put more info here
pkg_resources.VersionConflict: (vcrpy 1.5.2 (/usr/lib/python2.7/site-packages), Requirement.parse('vcrpy==1.12.0'))

@jeking3
Copy link
Contributor

jeking3 commented Nov 5, 2018

Interesting. I thought I changed it to vcrpy<2.0.0 - try that locally if you can. I'll submit a PR.

@rapgro
Copy link
Author

rapgro commented Nov 6, 2018

OK, thanks. I fixed with a downstream patch to temporarily drop the version checks. Looking forward to see the PR in the next official release.

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