This is an archive of the discontinued Mercurial Phabricator instance.

Differential D8573

fsmonitor: coerce `clock` variable to byte-string (issue6321)
ClosedPublic

Authored by sheehan on May 20 2020, 11:28 AM.

Details

Reviewers
indygreg
Group Reviewers
hg-reviewers
Commits
rHG017cc5ee537f: fsmonitor: coerce `clock` variable to byte-string (issue6321)
Summary

Callers of fsmonitor.state.setlastclock pass their arguments
wrapped in pycompat.sysbytes to ensure the value is a bytes
on Python 3. However in fsmonitor.poststatus.__call__, if the
return value of getlastclock() is None, we use the value of
fsmonitor.poststatus._startclock instead, which is not converted
to a byte string in the same manner. This commit converts the
value of startclock to a byte string using pycompat.sysbytes
in the constructor for poststatus, to avoid the "str + bytes"
error from issue 6321.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

sheehan created this revision.May 20 2020, 11:28 AM
indygreg accepted this revision.May 20 2020, 8:05 PM
indygreg added a subscriber: indygreg.

I'm going to queue this for stable, as it is a Python 3 compatibility issue affecting a distinguished Australian end-user :p

This revision is now accepted and ready to land.May 20 2020, 8:05 PM

Revision Contents
Changeset List

PathPackages
Mhgext/fsmonitor/__init__.py (2 lines)

Diff 21454

hgext/fsmonitor/__init__.py

return scmutil.status( return scmutil.status(
modified, added, removed, deleted, unknown, ignored, clean modified, added, removed, deleted, unknown, ignored, clean
) )
class poststatus(object): class poststatus(object):
def __init__(self, startclock): def __init__(self, startclock):
self._startclock = startclock self._startclock = pycompat.sysbytes(startclock)
def __call__(self, wctx, status): def __call__(self, wctx, status):
clock = wctx.repo()._fsmonitorstate.getlastclock() or self._startclock clock = wctx.repo()._fsmonitorstate.getlastclock() or self._startclock
hashignore = _hashignore(wctx.repo().dirstate._ignore) hashignore = _hashignore(wctx.repo().dirstate._ignore)
notefiles = ( notefiles = (
status.modified status.modified
+ status.added + status.added
+ status.removed + status.removed