D7504: py3: replace %s by %r on binary format string when needed

2019-11-22 Thread matclab (Mathieu Clabaut)
matclab added a comment. matclab abandoned this revision. INLINE COMMENTS > dlax wrote in localrepo.py:1571 > The first `%s` was correct I think because `changeid` can be a bytes. > > For the second one, if `%r` is the way to go (I'm not sure), maybe we can > drop `pycompat.sysstr()`? I'm

D7504: py3: replace %s by %r on binary format string when needed

2019-11-22 Thread matclab (Mathieu Clabaut)
matclab created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY For some error, mercurial displays the type of a variable. However the formatting string used was "%s" which is not compatible with `type` and lead to an