UTF-8 decode error

[futureweihk]

UTF-8 decode error

[aspiers]

This is most likely a bug in the approach to UTF-8 decoding. Perhaps the data being decoded is not actually UTF-8. Either way it will be related to the use of Python 3, as I haven't gotten around to doing heavy testing on Python 3 yet. See also #98 and #87 which are both related to Python 3.

[aspiers]

Are you able to share the repository which caused this bug, so we can try to reproduce?

[futureweihk]

This is most likely a bug in the approach to UTF-8 decoding. Perhaps the data being decoded is not actually UTF-8. Either way it will be related to the use of Python 3, as I haven't gotten around to doing heavy testing on Python 3 yet. See also #98 and #87 which are both related to Python 3.

Thanks, so do you mean that if we use Pyhon 2.7 can avoid such error?

[futureweihk]

How can we change git-deps Python engine? need reinstall git-deps, or any parameter changing can acheive this? thx

[aspiers]

@futureweihk commented on September 22, 2020 4:54 AM:

Thanks, so do you mean that if we use Pyhon 2.7 can avoid such error?

Possibly - I'd say there is a good chance but I can't guarantee it.

@futureweihk commented on September 22, 2020 4:56 AM:

How can we change git-deps Python engine? need reinstall git-deps, or any parameter changing can acheive this? thx

That depends very much on your OS and how you normally install Python. Please just follow standard Python installation documentation, as I do not have time to provide general support for Python. git-deps does not do anything significantly different to other Python programs, so standard procedures work as normal. If you are not too familiar with Python then you could alternatively find a Python consultant to help. It is not hard.

[futureweihk]

Sir，

We change the detector.py line 173 from:
blame = subprocess.check_output(cmd, universal_newlines=True)
to:
blame = subprocess.check_output(cmd, encoding="utf-8", errors="replace", universal_newlines=True)
Now seems the git-deps can run without utf-8 error, do you have any suggestions for the approach?

[aspiers]

Thanks, that's very helpful. I still need this though:

@aspiers commented on September 21, 2020 5:14 PM:

Are you able to share the repository which caused this bug, so we can try to reproduce?

so that I can reproduce and test the fix. Please can you share it?