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

Travis CI tests fail when dist: xenial is used instead of dist: trusty #827

Closed
cclauss opened this issue Jan 19, 2019 · 3 comments
Closed

Travis CI tests fail when dist: xenial is used instead of dist: trusty #827

cclauss opened this issue Jan 19, 2019 · 3 comments

Comments

@cclauss
Copy link
Contributor

@cclauss cclauss commented Jan 19, 2019

Why does git.test.test_remote.TestRemote fail in all versions of Python on Travis CI when dist: xenial is used instead of the Travis CI default of dist: trusty? For Python >= 3.7 it is required that Travis runs on Xenial: https://github.com/gitpython-developers/GitPython/blob/master/.travis.yml#L11-L14

https://travis-ci.org/gitpython-developers/GitPython/builds/481862926

@Byron
Copy link
Member

@Byron Byron commented Jan 20, 2019

Even though I am as puzzled as you are, that particular test uses the git daemon to verify remote operation, without leaving the host. Possibly the git-daemon provided by this distro operates differently enough to make the test fail.

Something I could imagine is to just disable it, for now, for the python versions that require it. I say this under the assumption that nobody has the time to figure what's really going on though.

@cclauss
Copy link
Contributor Author

@cclauss cclauss commented Jan 20, 2019

OK. I will add a PR for a @pytest.mark.xfail()

cclauss added a commit to cclauss/GitPython that referenced this issue Jan 21, 2019
@cclauss cclauss closed this Feb 5, 2019
@jeking3
Copy link
Contributor

@jeking3 jeking3 commented Feb 6, 2019

The root cause and fix is buried in #826 right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.