-
Notifications
You must be signed in to change notification settings - Fork 200
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
Update go-ipfs version to support < 0.10 #294
base: master
Are you sure you want to change the base?
Conversation
Tests are passing locally for me, and I'm not sure how to go about debugging on Travis. Let me know if anyone has any ideas as to why it might be failing! |
I checked with IPFS version 0.9.1 and I get the same problem with the OSX tests. It seems there is a Python issue: https://bugs.python.org/issue32958 |
I can't find a way to test this besides running the pipeline all the time. I don't have a Mac. So maybe someone else will be able to have a look into this issue :/ |
The tests are passing on my MacBook. platform darwin -- Python 3.9.7, pytest-6.2.5, py-1.10.0, pluggy-1.0.0 -- /py-ipfs-http-client/.tox/py3/bin/python
---
test/unit/test_http.py::test_successful_request_uds PASSED [ 34%]
test/unit/test_http.py::test_generic_failure_uds PASSED [ 35%]
---
Success: no issues found in 30 source files
_____________ summary_____________
py3: commands succeeded
py3-httpx: commands succeeded
styleck: commands succeeded
typeck: commands succeeded
congratulations :) There is no reason they should not pass. Apple macOS is Unix compliant and supports Unix Domain Sockets (UDS). Something is either wrong with CI or Python 3.9.0, most likely the former. |
Interesting. I tried it again but the error remains: https://app.travis-ci.com/github/ipfs-shipyard/py-ipfs-http-client/jobs/538855219 |
@jacekv I'm not familiar with Travis CI. Can you try with a version of macOS 11.x? The current release is 11.6. If this continues to fail I would recommend the project maintainers remove the UDS test for macOS specific builds. |
@hexcowboy Good point regarding the os version. I changed it to 11.3, since this is what seems to be currently supported by Travis and the error is still there. |
Is this ever going to get worked on? IPFS is now on 0.13.0 and this only supports up to version 0.8.0 :-| |
No description provided.