From 4c05cbe07201728a180c210fd3ec50d98d05fe95 Mon Sep 17 00:00:00 2001 From: Elijah Newren Date: Sat, 27 Apr 2019 14:34:59 -0700 Subject: [PATCH] filter-repo (python3): bytes() instead of chr() or string join Signed-off-by: Elijah Newren --- git-filter-repo | 5 +++-- t/t9391/splice_repos.py | 2 +- t/t9391/unusual.py | 2 +- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/git-filter-repo b/git-filter-repo index bd4682a..b694e8b 100755 --- a/git-filter-repo +++ b/git-filter-repo @@ -157,7 +157,8 @@ class PathQuoting: '"': '"', '\\':'\\'} _unescape_re = re.compile(r'\\([a-z"\\]|[0-9]{3})') - _escape = [chr(x) for x in range(127)]+['\\'+oct(x)[2:] for x in range(127,256)] + _escape = [bytes([x]) for x in range(127)]+[ + '\\'+bytes(ord(c) for c in oct(x)[2:]) for x in range(127,256)] _reverse = dict(map(reversed, _unescape.items())) for x in _reverse: _escape[ord(x)] = '\\'+_reverse[x] @@ -166,7 +167,7 @@ class PathQuoting: @staticmethod def unescape_sequence(orig): seq = orig.group(1) - return PathQuoting._unescape[seq] if len(seq) == 1 else chr(int(seq, 8)) + return PathQuoting._unescape[seq] if len(seq) == 1 else bytes([int(seq, 8)]) @staticmethod def dequote(quoted_string): diff --git a/t/t9391/splice_repos.py b/t/t9391/splice_repos.py index c7834c7..133044e 100755 --- a/t/t9391/splice_repos.py +++ b/t/t9391/splice_repos.py @@ -34,7 +34,7 @@ class InterleaveRepositories: def weave_commit(self, commit): letter = re.match('Commit (.)', commit.message).group(1) - prev_letter = chr(ord(letter)-1) + prev_letter = bytes([ord(letter)-1]) # Splice in any extra commits needed if prev_letter in self.commit_map: diff --git a/t/t9391/unusual.py b/t/t9391/unusual.py index c7a2c57..da0cf89 100755 --- a/t/t9391/unusual.py +++ b/t/t9391/unusual.py @@ -21,7 +21,7 @@ import textwrap import git_filter_repo as fr def handle_progress(progress): - print("Decipher this: "+''.join(reversed(progress.message))) + print("Decipher this: "+bytes(reversed(progress.message))) def handle_checkpoint(checkpoint_object): # Flip a coin; see if we want to pass the checkpoint through.