Patchwork D271: obsolete: use bytes() instead of str() so the node is bytes on py3

login
register
mail settings
Submitter phabricator
Date Aug. 9, 2017, 2:51 p.m.
Message ID <2f7f8e882b1141867b52ace0797c5bd8@localhost.localdomain>
Download mbox | patch
Permalink /patch/22802/
State Not Applicable
Headers show

Comments

phabricator - Aug. 9, 2017, 2:51 p.m.
This revision was automatically updated to reflect the committed changes.
durin42 marked an inline comment as done.
Closed by commit rHG388901a15bfa: obsolete: use bytes() instead of str() so the node is bytes on py3 (authored by durin42).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D271?vs=675&id=686

REVISION DETAIL
  https://phab.mercurial-scm.org/D271

AFFECTED FILES
  mercurial/obsolete.py

CHANGE DETAILS




To: durin42, #hg-reviewers, quark
Cc: yuja, mercurial-devel

Patch

diff --git a/mercurial/obsolete.py b/mercurial/obsolete.py
--- a/mercurial/obsolete.py
+++ b/mercurial/obsolete.py
@@ -583,7 +583,7 @@ 
 
         metadata = tuple(sorted(metadata.iteritems()))
 
-        marker = (str(prec), tuple(succs), int(flag), metadata, date, parents)
+        marker = (bytes(prec), tuple(succs), int(flag), metadata, date, parents)
         return bool(self.add(transaction, [marker]))
 
     def add(self, transaction, markers):