This is an archive of the discontinued Mercurial Phabricator instance.

Differential D3786

py3: use '%d' for os.stat_result.st_nlink instead of '%s'
ClosedPublic

Authored by pulkit on Jun 18 2018, 6:22 AM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHG06c85cbd6824: py3: use '%d' for os.stat_result.st_nlink instead of '%s'
Summary

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

pulkit created this revision.Jun 18 2018, 6:22 AM
Closed by commit rHG06c85cbd6824: py3: use '%d' for os.stat_result.st_nlink instead of '%s' (authored by pulkit). · Explain WhyJun 18 2018, 8:33 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/f (2 lines)

Diff 9169

tests/f

facts.append(b'type unknown') facts.append(b'type unknown')
if not isstdin: if not isstdin:
stat = os.lstat(f) stat = os.lstat(f)
if opts.size and not isdir: if opts.size and not isdir:
facts.append(b'size=%d' % stat.st_size) facts.append(b'size=%d' % stat.st_size)
if opts.mode and not islink: if opts.mode and not islink:
facts.append(b'mode=%o' % (stat.st_mode & 0o777)) facts.append(b'mode=%o' % (stat.st_mode & 0o777))
if opts.links: if opts.links:
facts.append(b'links=%s' % stat.st_nlink) facts.append(b'links=%d' % stat.st_nlink)
if opts.newer: if opts.newer:
# mtime might be in whole seconds so newer file might be same # mtime might be in whole seconds so newer file might be same
if stat.st_mtime >= os.stat(opts.newer).st_mtime: if stat.st_mtime >= os.stat(opts.newer).st_mtime:
facts.append(b'newer than %s' % opts.newer) facts.append(b'newer than %s' % opts.newer)
else: else:
facts.append(b'older than %s' % opts.newer) facts.append(b'older than %s' % opts.newer)
if opts.md5 and content is not None: if opts.md5 and content is not None:
h = hashlib.md5(content) h = hashlib.md5(content)