Message ID | 1265340805-25653-1-git-send-email-madduck@madduck.net |
---|---|
State | Accepted |
Headers | show |
Am Freitag, den 05.02.2010, 04:33 +0100 schrieb martin f. krafft: > Patchwork (rightfully) hashes patches using UTF-8: > > parser.py: > def hash_patch(str): > […] > hash.update(line.encode('utf-8') + '\n') > > This patch makes sure that it treats the incoming data as UTF-8 as well. > > Signed-off-by: martin f. krafft <madduck@madduck.net> I think this patch needs to be reapplied, after the branch change(?) in March 2010 [1]. Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net> [1] http://lists.ozlabs.org/pipermail/patchwork/2010-March/000251.html > --- > apps/patchwork/parser.py | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/apps/patchwork/parser.py b/apps/patchwork/parser.py > index 16cc308..de81ca7 100644 > --- a/apps/patchwork/parser.py > +++ b/apps/patchwork/parser.py > @@ -63,8 +63,7 @@ def parse_patch(text): > lc = (0, 0) > hunk = 0 > > - > - for line in text.split('\n'): > + for line in text.decode('utf-8').split('\n'): > line += '\n' > > if state == 0:
Hi Paul, > I think this patch needs to be reapplied, after the branch change(?) in > March 2010 [1]. Thanks, good catch, I've reapplied it. Cheers, Jeremy
diff --git a/apps/patchwork/parser.py b/apps/patchwork/parser.py index 16cc308..de81ca7 100644 --- a/apps/patchwork/parser.py +++ b/apps/patchwork/parser.py @@ -63,8 +63,7 @@ def parse_patch(text): lc = (0, 0) hunk = 0 - - for line in text.split('\n'): + for line in text.decode('utf-8').split('\n'): line += '\n' if state == 0:
Patchwork (rightfully) hashes patches using UTF-8: parser.py: def hash_patch(str): […] hash.update(line.encode('utf-8') + '\n') This patch makes sure that it treats the incoming data as UTF-8 as well. Signed-off-by: martin f. krafft <madduck@madduck.net> --- apps/patchwork/parser.py | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-)