Bug 182288 - Update test262 to Jan 30 version
Summary: Update test262 to Jan 30 version
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Yusuke Suzuki
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-01-30 06:20 PST by Yusuke Suzuki
Modified: 2018-02-03 01:59 PST (History)
7 users (show)

See Also:


Attachments
Patch (9.65 MB, patch)
2018-01-30 06:37 PST, Yusuke Suzuki
no flags Details | Formatted Diff | Diff
Patch (9.65 MB, patch)
2018-01-30 08:46 PST, Yusuke Suzuki
no flags Details | Formatted Diff | Diff
Patch (9.65 MB, patch)
2018-02-01 05:00 PST, Yusuke Suzuki
no flags Details | Formatted Diff | Diff
Patch (9.64 MB, patch)
2018-02-02 10:07 PST, Yusuke Suzuki
no flags Details | Formatted Diff | Diff
Patch (9.64 MB, patch)
2018-02-02 11:31 PST, Yusuke Suzuki
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yusuke Suzuki 2018-01-30 06:20:48 PST
Update Test262
Comment 1 Yusuke Suzuki 2018-01-30 06:37:52 PST
Created attachment 332651 [details]
Patch
Comment 2 Yusuke Suzuki 2018-01-30 08:46:03 PST
Created attachment 332658 [details]
Patch
Comment 3 Yusuke Suzuki 2018-01-30 09:33:09 PST
(In reply to Yusuke Suzuki from comment #2)
> Created attachment 332658 [details]
> Patch

Even with the latest ToT, patching always fails. I guess it is due to the size of patch...?
Comment 4 Saam Barati 2018-01-30 09:38:13 PST
rs=me
Comment 5 Yusuke Suzuki 2018-02-01 05:00:17 PST
Created attachment 332868 [details]
Patch
Comment 6 Yusuke Suzuki 2018-02-02 00:07:10 PST
Super funny thing is that, we cannot import test262 because SVN rejects its diff.
I'm not sure what happens right now. I guess that newer test262 includes some very tricky characters, which makes diff crazy. (like, u2028 / u2029?).
I'm still investigating...
Comment 7 Yusuke Suzuki 2018-02-02 10:07:18 PST
Created attachment 332979 [details]
Patch
Comment 8 EWS Watchlist 2018-02-02 10:28:45 PST
Attachment 332979 [details] did not pass style-queue:

Traceback (most recent call last):
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 84, in <module>
    main()
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 79, in main
    WebKitPatch(os.path.abspath(__file__)).main()
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main
    result = command.check_arguments_and_execute(options, args, self)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute
    return self.execute(options, args, tool) or 0
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute
    self._sequence.run_and_handle_errors(tool, options, state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors
    self._run(tool, options, state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run
    step(tool, options).run(state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/steps/checkstyle.py", line 66, in run
    self._tool.executive.run_and_throw_if_fail(self._tool.deprecated_port().check_webkit_style_command() + args, cwd=self._tool.scm().checkout_root)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 135, in run_and_throw_if_fail
    exit_code = self._run_command_with_teed_output(args, child_stdout, **kwargs)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 107, in _run_command_with_teed_output
    **kwargs)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 479, in popen
    return subprocess.Popen(string_args, **kwargs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child
    raise child_exception
OSError: [Errno 7] Argument list too long


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 9 Yusuke Suzuki 2018-02-02 11:31:36 PST
Created attachment 332986 [details]
Patch
Comment 10 EWS Watchlist 2018-02-02 11:53:32 PST
Attachment 332986 [details] did not pass style-queue:

Traceback (most recent call last):
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 84, in <module>
    main()
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkit-patch", line 79, in main
    WebKitPatch(os.path.abspath(__file__)).main()
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 305, in main
    result = command.check_arguments_and_execute(options, args, self)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/multicommandtool.py", line 123, in check_arguments_and_execute
    return self.execute(options, args, tool) or 0
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/abstractsequencedcommand.py", line 55, in execute
    self._sequence.run_and_handle_errors(tool, options, state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 73, in run_and_handle_errors
    self._run(tool, options, state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/commands/stepsequence.py", line 67, in _run
    step(tool, options).run(state)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/tool/steps/checkstyle.py", line 66, in run
    self._tool.executive.run_and_throw_if_fail(self._tool.deprecated_port().check_webkit_style_command() + args, cwd=self._tool.scm().checkout_root)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 135, in run_and_throw_if_fail
    exit_code = self._run_command_with_teed_output(args, child_stdout, **kwargs)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 107, in _run_command_with_teed_output
    **kwargs)
  File "/Volumes/Data/StyleQueue/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 479, in popen
    return subprocess.Popen(string_args, **kwargs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child
    raise child_exception
OSError: [Errno 22] Invalid argument


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 11 WebKit Commit Bot 2018-02-02 13:57:57 PST
Comment on attachment 332986 [details]
Patch

Clearing flags on attachment: 332986

Committed r228028: <https://trac.webkit.org/changeset/228028>
Comment 12 WebKit Commit Bot 2018-02-02 13:57:59 PST
All reviewed patches have been landed.  Closing bug.
Comment 13 Radar WebKit Bug Importer 2018-02-02 14:00:40 PST
<rdar://problem/37169557>
Comment 14 Ryan Haddad 2018-02-02 17:22:12 PST
Tests are failing to run on the bots after this change:

https://build.webkit.org/builders/Apple%20Sierra%20Debug%20Test262%20%28Tests%29/builds/383

./Tools/Scripts/run-jsc-stress-tests:1405:in `realpath': No such file or directory - /Volumes/Data/slave/sierra-debug-tests-test262/build/JSTests/test262/test/language/expressions/assignment/white-space.js (Errno::ENOENT)
Comment 15 Ryan Haddad 2018-02-02 17:33:15 PST
(In reply to Ryan Haddad from comment #14)
> Tests are failing to run on the bots after this change:
> 
> https://build.webkit.org/builders/
> Apple%20Sierra%20Debug%20Test262%20%28Tests%29/builds/383
> 
> ./Tools/Scripts/run-jsc-stress-tests:1405:in `realpath': No such file or
> directory -
> /Volumes/Data/slave/sierra-debug-tests-test262/build/JSTests/test262/test/
> language/expressions/assignment/white-space.js (Errno::ENOENT)
Removed this test from test262.yaml in https://trac.webkit.org/r228039
Comment 16 Yusuke Suzuki 2018-02-03 01:48:39 PST
(In reply to Ryan Haddad from comment #15)
> (In reply to Ryan Haddad from comment #14)
> > Tests are failing to run on the bots after this change:
> > 
> > https://build.webkit.org/builders/
> > Apple%20Sierra%20Debug%20Test262%20%28Tests%29/builds/383
> > 
> > ./Tools/Scripts/run-jsc-stress-tests:1405:in `realpath': No such file or
> > directory -
> > /Volumes/Data/slave/sierra-debug-tests-test262/build/JSTests/test262/test/
> > language/expressions/assignment/white-space.js (Errno::ENOENT)
> Removed this test from test262.yaml in https://trac.webkit.org/r228039

Oops, thanks!
Comment 17 Yusuke Suzuki 2018-02-03 01:59:43 PST
Committed r228047: <https://trac.webkit.org/changeset/228047>