-
-
Notifications
You must be signed in to change notification settings - Fork 934
test failures #674
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
Comments
Related to #673 . I presume this worked before? |
I hadn't run the self-tests for GitPython before, no. Is there any additional information I can provide to help you in debugging this? |
I think for the person tempted to pick it up it would be great to know your motivation. Right now, this issue describes something that is not working, but one might wonder which problem this causes, and the benefit a fix would provide. |
Sure, that is a valid question. |
Does it work for you if you do this instead?
|
I had to switch the order around, the
So the tests assume that the user under which the tests run has git When I configure name and email, I get:
which looks quite respectable already :-) |
I think it will get better once you have run this script. For reference - all things that have to be done to make it work are contained in the |
Thanks for the pointer. I suggest documenting the necessary setup in a human-readable location, and perhaps providing a single script that does all the necessary changes. Right now it's hard to guess what is needed, even you needed multiple messages to describe all the necessary stuff. |
I totally agree that it’s not so obvious what the test-suite expects and where to find it. On the other hand, looking at the CI system is always a good starting point. Do I assume correctly that this is working for you now? If so, you could close this issue and possibly provide a paragraph in the README file that would have helped you solve the problems you encountered. |
I think it would be sufficient to document the results of this issue in the
It would help if the init* script could also do the Other than that, I see
which is good enough for me :) (not sure what the problem is:
) |
@0-wiz-0 Thanks a lot! I think if you run |
I tried again today and I couldn't reproduce it. I saw you updated the documentation, thank you! |
See gitpython-developers#674 for reference. [skip CI]
When running the self tests for today's git head of GitPython as of today (ddb828e) on NetBSD with python-3.6.2, ddt-1.1.1, mock-2.0.0, nose-1.3.7, smmap2-2.0.3, gitdb2-2.0.3 and git-2.14.2, there are many test failures:
gives me:
The text was updated successfully, but these errors were encountered: