This is an archive of the discontinued Mercurial Phabricator instance.

Differential D11225

dummyssh: make sure we don't inherit files descriptor to the children
ClosedPublic

Authored by marmoute on Jul 28 2021, 10:57 AM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHGfb8389f227a0: dummyssh: make sure we don't inherit files descriptor to the children
Summary

This is one of the difference between python2 and python3 and could have been a
reason why test hang with python2 + chg. This does not seems to help the
hanging issue at all…

However, now that this is written lets reduce the difference between python2
and python3.

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

marmoute created this revision.Jul 28 2021, 10:57 AM
marmoute added a commit: rHGfb8389f227a0: dummyssh: make sure we don't inherit files descriptor to the children.Jul 30 2021, 11:11 AM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHGfb8389f227a0: dummyssh: make sure we don't inherit files descriptor to the children (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mtests/dummyssh (2 lines)

Diff 29752

tests/dummyssh

hgcmd = hgcmd.replace("'", '"') hgcmd = hgcmd.replace("'", '"')
cmds = shlex.split(hgcmd) cmds = shlex.split(hgcmd)
if cmds[0].endswith('.py'): if cmds[0].endswith('.py'):
python_exe = os.environ['PYTHON'] python_exe = os.environ['PYTHON']
cmds.insert(0, python_exe) cmds.insert(0, python_exe)
hgcmd = shlex.join(cmds) hgcmd = shlex.join(cmds)
# shlex generate windows incompatible string... # shlex generate windows incompatible string...
hgcmd = hgcmd.replace("'", '"') hgcmd = hgcmd.replace("'", '"')
r = subprocess.call(hgcmd, shell=True) r = subprocess.call(hgcmd, shell=True, close_fds=True)
sys.exit(bool(r)) sys.exit(bool(r))