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:24 p.m.
Message ID <6cf719958719c08fc19fd788d8f12c0b@localhost.localdomain>
Download mbox | patch
Permalink /patch/22791/
State Not Applicable
Headers show

Comments

phabricator - Aug. 9, 2017, 2:24 p.m.
durin42 updated this revision to Diff 675.
durin42 retitled this revision from "obsolete: use bytestr() instead of str() so the node is bytes on py3" to "obsolete: use bytes() instead of str() so the node is bytes on py3".

REPOSITORY
  rHG Mercurial

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

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):