D296: extensions: if on py3 and propname is a bytestr, convert to sysstr

2017-08-15 Thread durin42 (Augie Fackler)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG38a3767975a7: extensions: if on py3 and propname is a bytestr, convert to sysstr (authored by durin42). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE

D296: extensions: if on py3 and propname is a bytestr, convert to sysstr

2017-08-15 Thread durin42 (Augie Fackler)
durin42 added a comment. Take another look? REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D296 To: durin42, #hg-reviewers Cc: yuja, mercurial-devel ___ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org

D296: extensions: if on py3 and propname is a bytestr, convert to sysstr

2017-08-15 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 945. durin42 marked 2 inline comments as done. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D296?vs=799=945 REVISION DETAIL https://phab.mercurial-scm.org/D296 AFFECTED FILES mercurial/extensions.py CHANGE

D296: extensions: if on py3 and propname is a bytestr, convert to sysstr

2017-08-12 Thread yuja (Yuya Nishihara)
yuja added inline comments. INLINE COMMENTS > yuja wrote in extensions.py:388 > `pycompat.sysstr()` ? The "if" shouldn't be necessary. That's what `sysstr()` is for. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D296 To: durin42, #hg-reviewers Cc: yuja,

D296: extensions: if on py3 and propname is a bytestr, convert to sysstr

2017-08-11 Thread durin42 (Augie Fackler)
durin42 added inline comments. INLINE COMMENTS > yuja wrote in extensions.py:401 > Perhaps this shouldn't be bytes. Needs `r''` and drop `_()`. Done in a followup change. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D296 To: durin42, #hg-reviewers Cc: yuja,

D296: extensions: if on py3 and propname is a bytestr, convert to sysstr

2017-08-11 Thread durin42 (Augie Fackler)
durin42 updated this revision to Diff 799. durin42 marked 2 inline comments as done. durin42 retitled this revision from "extensions: if on py3 and propname is a bytestr, decode as ascii" to "extensions: if on py3 and propname is a bytestr, convert to sysstr". REPOSITORY rHG Mercurial