Skip to content

Fix encoding issue with stderr_value and kill_after_timeout #682

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
Oct 7, 2017

Conversation

pabelanger
Copy link
Contributor

We don't properly encode our error message under python3.

Signed-off-by: Paul Belanger pabelanger@redhat.com

We don't properly encode our error message under python3.

Signed-off-by: Paul Belanger <pabelanger@redhat.com>
@codecov-io
Copy link

Codecov Report

Merging #682 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #682   +/-   ##
=======================================
  Coverage   92.95%   92.95%           
=======================================
  Files          61       61           
  Lines       10065    10065           
=======================================
  Hits         9356     9356           
  Misses        709      709
Impacted Files Coverage Δ
git/cmd.py 83.4% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95ff827...1dbfd29. Read the comment docs.

@Byron
Copy link
Member

Byron commented Oct 7, 2017

Thanks a lot for your contribution!

@Byron Byron merged commit f237620 into gitpython-developers:master Oct 7, 2017
openstack-gerrit pushed a commit to openstack-infra/zuul that referenced this pull request Oct 10, 2017
Timeout remote git operations after 300 seconds.

Because it could be in an invalid state, delete the local repo if a
timeout occurs (subsequent operations will recreate it).

This replaces our use of the clone_from() and fetch() methods from
GitPython with lower-level equivalents.  The high-level methods
do not currently permit the hard timeout.

The GitPython requirement is changed to a temporary fork until both

gitpython-developers/GitPython#682
and
gitpython-developers/GitPython#686

end up in a release.

Change-Id: I7f680472a8d67ff2dbe7956a8585fb3714119e65
riley-martine pushed a commit to riley-martine/GitPython that referenced this pull request Dec 7, 2023
…ding

Fix encoding issue with stderr_value and kill_after_timeout
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants