Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows can run WPT #17419

Merged
merged 2 commits into from Jun 22, 2017
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Next

Minimal changes to get WPT tests running on Windows.

  • Loading branch information
camlorn committed Jun 20, 2017
commit f78dc9f613ec8788c894343955bfd376e260bb41
@@ -7,6 +7,7 @@
import os
import sys
import subprocess
import platform

DEFAULT_MOVE_UP_CODE = u"\x1b[A"
DEFAULT_CLEAR_EOL_CODE = u"\x1b[K"
@@ -29,8 +30,12 @@ def __init__(self):

# TODO(mrobinson, 8313): We need to add support for Windows terminals here.
if self.interactive:
self.line_width = int(subprocess.check_output(['stty', 'size']).split()[1])
self.move_up, self.clear_eol = self.get_move_up_and_clear_eol_codes()
if platform.system() != "Windows":
self.line_width = int(subprocess.check_output(['stty', 'size']).split()[1])
else:
# Until we figure out proper Windows support, this makes things work well enough to run.
self.line_width = 80

self.expected = {
'OK': 0,
@@ -256,8 +256,10 @@ def exe_path(name):
return

path = find_executable(name)
if os.access(path, os.X_OK):
if path and os.access(path, os.X_OK):
return path
else:
return None


def check_args(kwargs):
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.