This is an archive of the discontinued Mercurial Phabricator instance.

Differential D6777

py3: make statprof's chrome output work
ClosedPublic

Authored by martinvonz on Aug 30 2019, 3:03 PM.

Details

Reviewers
pulkit
Group Reviewers
hg-reviewers
Commits
rHGcde1c101ab8a: py3: make statprof's chrome output work
rHG4187ecc19ff5: py3: make statprof's chrome output work
Summary

With this patch, this command works:

python3 hg --profile --config profiling.statformat=chrome st

(and it works with s/python3/python2/ as well)

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

martinvonz created this revision.Aug 30 2019, 3:03 PM
martinvonz edited the summary of this revision. (Show Details)Aug 30 2019, 3:03 PM
martinvonz planned changes to this revision.Aug 30 2019, 6:22 PM

There's a lot more that's needed for python3 hg --config profiling.statformat=chrome --profile to work, so I'll probably put more changes in this patch soon.

martinvonz added a parent revision: D6781: py3: for statprof's Chrome output, write json to string, then encode to bytes.Aug 31 2019, 1:35 PM
martinvonz retitled this revision from py3: go back to using strings for paths in statprof.py to py3: make statprof's chrome output work.
martinvonz edited the summary of this revision. (Show Details)
martinvonz updated this revision to Diff 16347.
pulkit accepted this revision.Aug 31 2019, 2:00 PM
This revision is now accepted and ready to land.Aug 31 2019, 2:00 PM
martinvonz added a commit: rHG4187ecc19ff5: py3: make statprof's chrome output work.Aug 31 2019, 6:59 PM
Closed by commit rHG4187ecc19ff5: py3: make statprof's chrome output work (authored by martinvonz). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/statprof.py (2 lines)

Diff 16342

mercurial/statprof.py

_pathcache = {} _pathcache = {}
def simplifypath(path): def simplifypath(path):
'''Attempt to make the path to a Python module easier to read by '''Attempt to make the path to a Python module easier to read by
removing whatever part of the Python search path it was found removing whatever part of the Python search path it was found
on.''' on.'''
if path in _pathcache: if path in _pathcache:
return _pathcache[path] return _pathcache[path]
hgpath = pycompat.fsencode(encoding.__file__).rsplit(os.sep, 2)[0] hgpath = encoding.__file__.rsplit(os.sep, 2)[0]
for p in [hgpath] + sys.path: for p in [hgpath] + sys.path:
prefix = p + os.sep prefix = p + os.sep
if path.startswith(prefix): if path.startswith(prefix):
path = path[len(prefix):] path = path[len(prefix):]
break break
_pathcache[path] = path _pathcache[path] = path
return path return path