Mercurial > public > mercurial-scm > hg
diff mercurial/pycompat.py @ 30578:c6ce11f2ee50
py3: make a bytes version of getopt.getopt()
getopt.getopt() deals with unicodes on Python 3 internally and if bytes
arguments are passed, then it will return TypeError. So we have now
pycompat.getoptb() which takes bytes arguments, convert them to unicode, call
getopt.getopt() and then convert the returned value back to bytes and then
return those value.
All the instances of getopt.getopt() are replaced with pycompat.getoptb().
author | Pulkit Goyal <7895pulkit@gmail.com> |
---|---|
date | Tue, 06 Dec 2016 06:36:36 +0530 |
parents | fc0cfe6c87d7 |
children | fbc3f73dc802 |
line wrap: on
line diff
--- a/mercurial/pycompat.py Tue Dec 06 11:44:49 2016 +0000 +++ b/mercurial/pycompat.py Tue Dec 06 06:36:36 2016 +0530 @@ -10,6 +10,7 @@ from __future__ import absolute_import +import getopt import os import sys @@ -87,6 +88,19 @@ setattr = _wrapattrfunc(builtins.setattr) xrange = builtins.range + # getopt.getopt() on Python 3 deals with unicodes internally so we cannot + # pass bytes there. Passing unicodes will result in unicodes as return + # values which we need to convert again to bytes. + def getoptb(args, shortlist, namelist): + args = [a.decode('latin-1') for a in args] + shortlist = shortlist.decode('latin-1') + namelist = [a.decode('latin-1') for a in namelist] + opts, args = getopt.getopt(args, shortlist, namelist) + opts = [(a[0].encode('latin-1'), a[1].encode('latin-1')) + for a in opts] + args = [a.encode('latin-1') for a in args] + return opts, args + else: def sysstr(s): return s @@ -106,6 +120,9 @@ def fsdecode(filename): return filename + def getoptb(args, shortlist, namelist): + return getopt.getopt(args, shortlist, namelist) + osname = os.name ospathsep = os.pathsep ossep = os.sep