Skip to content

Ignore progress lines from the git http backend that start w/ POST ... #42

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

Merged
merged 1 commit into from
May 17, 2012

Conversation

gsoltis
Copy link
Contributor

@gsoltis gsoltis commented Jan 11, 2012

A better way to implement this might be to enumerate the things you do want to count, rather than skipping ones you don't. In the meantime, this fixes the error I ran into on my server.

Byron added a commit that referenced this pull request May 17, 2012
Ignore progress lines from the git http backend that start w/ POST ...

-- gsoltis -- 
A better way to implement this might be to enumerate the things you do want to count, rather than skipping ones you don't. In the meantime, this fixes the error I ran into on my server.
@Byron Byron merged commit a7c4b32 into gitpython-developers:0.3 May 17, 2012
devdattakulkarni pushed a commit to devdattakulkarni/zuul_messaging that referenced this pull request Feb 10, 2014
GitPython when fetching refs may improperly read in git progress info
and interpret it as fetch info. When this happens an AssertionError is
thrown. However, since fetching seems to cache the ref locally
subsequent fetches work without running into the progress info problem.
So fetch again if an AssertionError is caught.

Upstream bugfix
gitpython-developers/GitPython#42

Change-Id: I1768b836aade2853d4dd3b2f4cd1fce7ec035524
Reviewed-on: https://review.openstack.org/15219
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants