D10939: run-tests: unroll a non-so-one-liner

Submitter	phabricator
Date	July 2, 2021, 10:37 p.m.
Message ID	<differential-rev-PHID-DREV-sjmufmjpxnewllz3hzpv-req@mercurial-scm.org>
Download	mbox \| patch
Permalink	/patch/49253/
State	Superseded
Headers	show Return-Path: <mercurial-devel-bounces@mercurial-scm.org> Date: Fri, 2 Jul 2021 22:37:51 +0000 To: Phabricator <phabricator@mercurial-scm.org> From: "marmoute (Pierre-Yves David)" <phabricator@mercurial-scm.org> Subject: D10939: run-tests: unroll a non-so-one-liner Message-ID: <differential-rev-PHID-DREV-sjmufmjpxnewllz3hzpv-req@mercurial-scm.org> Precedence: bulk Thread-Topic: PHID-DREV-sjmufmjpxnewllz3hzpv Thread-Index: ZDI0YTlmMDdmODBlOGIwYzU3YTI1YmQ4Zjcy MIME-Version: 1.0 Cc: mercurial-devel@mercurial-scm.org Reply-To: "marmoute \(Pierre-Yves David\)" <phabricator+D10939+public+48632a4d02559e4f@mercurial-scm.org> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: mercurial-devel-bounces@mercurial-scm.org Sender: "Mercurial-devel" <mercurial-devel-bounces@mercurial-scm.org>

Comments

phabricator - July 2, 2021, 10:37 p.m.

marmoute created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  This is shorter and clearer.

REPOSITORY
  rHG Mercurial

BRANCH
  default

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

AFFECTED FILES
  tests/run-tests.py

CHANGE DETAILS




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

Patch

diff --git a/tests/run-tests.py b/tests/run-tests.py
--- a/tests/run-tests.py
+++ b/tests/run-tests.py
@@ -3564,10 +3564,9 @@ 
                     f.write(b'py -3.%d "$@"\n' % sys.version_info[1])
 
             exedir, exename = os.path.split(sysexecutable)
-            vlog(
-                "# Modifying search path to find %s as %s in '%s'"
-                % (exename, pyexename, exedir)
-            )
+            msg = "# Modifying search path to find %s as %s in '%s'"
+            msg %= (exename, pyexename, exedir)
+            vlog(msg)
             path = os.environ['PATH'].split(os.pathsep)
             while exedir in path:
                 path.remove(exedir)

Patchwork D10939: run-tests: unroll a non-so-one-liner

Comments

Patch