Patchwork D11082: dummyssh: use subprocess instead of os.call

login
register
mail settings
Submitter phabricator
Date July 11, 2021, 9:38 p.m.
Message ID <differential-rev-PHID-DREV-fnc5zlojgprzu54dccld-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/49394/
State Superseded
Headers show

Comments

phabricator - July 11, 2021, 9:38 p.m.
marmoute created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  Welcome in 2020

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D11082

AFFECTED FILES
  tests/dummyssh

CHANGE DETAILS




To: marmoute, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

Patch

diff --git a/tests/dummyssh b/tests/dummyssh
--- a/tests/dummyssh
+++ b/tests/dummyssh
@@ -3,6 +3,7 @@ 
 from __future__ import absolute_import
 
 import os
+import subprocess
 import sys
 
 os.chdir(os.getenv('TESTTMP'))
@@ -22,5 +23,5 @@ 
 if os.name == 'nt':
     # hack to make simple unix single quote quoting work on windows
     hgcmd = hgcmd.replace("'", '"')
-r = os.system(hgcmd)
+r = subprocess.call(hgcmd, shell=True)
 sys.exit(bool(r))