Index: test/lib/TestGyp.py |
diff --git a/test/lib/TestGyp.py b/test/lib/TestGyp.py |
index f32099be98794f9a90db3cd378ad777314266ca7..f0c119e0e476bac3bb35969cfef3091c61469875 100644 |
--- a/test/lib/TestGyp.py |
+++ b/test/lib/TestGyp.py |
@@ -6,6 +6,8 @@ |
TestGyp.py: a testing framework for GYP integration tests. |
""" |
+from __future__ import print_function |
+ |
import collections |
from contextlib import contextmanager |
import itertools |
@@ -282,13 +284,13 @@ class TestGypBase(TestCommon.TestCommon): |
that expect exact output from the command (make) can |
just set stdout= when they call the run_build() method. |
""" |
- print "Build is not up-to-date:" |
- print self.banner('STDOUT ') |
- print self.stdout() |
+ print("Build is not up-to-date:") |
+ print(self.banner('STDOUT ')) |
+ print(self.stdout()) |
stderr = self.stderr() |
if stderr: |
- print self.banner('STDERR ') |
- print stderr |
+ print(self.banner('STDERR ')) |
+ print(stderr) |
def run_gyp(self, gyp_file, *args, **kw): |
""" |
@@ -326,7 +328,7 @@ class TestGypBase(TestCommon.TestCommon): |
the tool-specific subclasses or clutter the tests themselves |
with platform-specific code. |
""" |
- if kw.has_key('SYMROOT'): |
+ if 'SYMROOT' in kw: |
del kw['SYMROOT'] |
super(TestGypBase, self).run(*args, **kw) |
@@ -556,7 +558,7 @@ class TestGypMake(TestGypBase): |
# Makefile.gyp_filename), so use that if there is no Makefile. |
chdir = kw.get('chdir', '') |
if not os.path.exists(os.path.join(chdir, 'Makefile')): |
- print "NO Makefile in " + os.path.join(chdir, 'Makefile') |
+ print("NO Makefile in " + os.path.join(chdir, 'Makefile')) |
arguments.insert(0, '-f') |
arguments.insert(1, os.path.splitext(gyp_file)[0] + '.Makefile') |
kw['arguments'] = arguments |
@@ -663,7 +665,7 @@ def FindMSBuildInstallation(msvs_version = 'auto'): |
msbuild_basekey = r'HKLM\SOFTWARE\Microsoft\MSBuild\ToolsVersions' |
if not registry.KeyExists(msbuild_basekey): |
- print 'Error: could not find MSBuild base registry entry' |
+ print('Error: could not find MSBuild base registry entry') |
return None |
msbuild_version = None |
@@ -682,13 +684,13 @@ def FindMSBuildInstallation(msvs_version = 'auto'): |
msbuild_version = msbuild_test_version |
break |
if not msbuild_version: |
- print 'Error: could not find MSBuild registry entry' |
+ print('Error: could not find MSBuild registry entry') |
return None |
msbuild_path = registry.GetValue(msbuild_basekey + '\\' + msbuild_version, |
'MSBuildToolsPath') |
if not msbuild_path: |
- print 'Error: could not get MSBuild registry entry value' |
+ print('Error: could not get MSBuild registry entry value') |
return None |
return os.path.join(msbuild_path, 'MSBuild.exe') |
@@ -741,7 +743,7 @@ def FindVisualStudioInstallation(): |
uses_msbuild = msvs_version >= '2010' |
msbuild_path = FindMSBuildInstallation(msvs_version) |
return build_tool, uses_msbuild, msbuild_path |
- print 'Error: could not find devenv' |
+ print('Error: could not find devenv') |
sys.exit(1) |
class TestGypOnMSToolchain(TestGypBase): |
@@ -886,10 +888,10 @@ class TestGypMSVS(TestGypOnMSToolchain): |
Verifies that a build of the specified Visual Studio target is up to date. |
Beware that VS2010 will behave strangely if you build under |
- C:\USERS\yourname\AppData\Local. It will cause needless work. The ouptut |
+ C:/USERS/yourname/AppData/Local. It will cause needless work. The ouptut |
Nico
2016/07/29 22:22:06
out of interest, is this needed? with \ it's more
AWhetter
2016/11/05 23:59:50
I've worked around it by making the docstring a ra
|
will be "1 succeeded and 0 up to date". MSBuild tracing reveals that: |
- "Project 'C:\Users\...\AppData\Local\...vcxproj' not up to date because |
- 'C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO 10.0\VC\BIN\1033\CLUI.DLL' |
+ "Project 'C:/Users/.../AppData/Local/...vcxproj' not up to date because |
+ 'C:/PROGRAM FILES (X86)/MICROSOFT VISUAL STUDIO 10.0/VC/BIN/1033/CLUI.DLL' |
was modified at 02/21/2011 17:03:30, which is newer than '' which was |
modified at 01/01/0001 00:00:00. |
@@ -1187,4 +1189,4 @@ def TestGyp(*args, **kw): |
for format_class in format_class_list: |
if format == format_class.format: |
return format_class(*args, **kw) |
- raise Exception, "unknown format %r" % format |
+ raise Exception("unknown format %r" % format) |