Bug 132051 - [GTK] run-webkit-tests script is failing to invoke run-launcher
Summary: [GTK] run-webkit-tests script is failing to invoke run-launcher
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Lorenzo Tilve
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-23 01:45 PDT by Lorenzo Tilve
Modified: 2014-04-23 02:40 PDT (History)
6 users (show)

See Also:


Attachments
Patch (1.69 KB, patch)
2014-04-23 02:00 PDT, Lorenzo Tilve
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lorenzo Tilve 2014-04-23 01:45:57 PDT
When manually running a test like:

Tools/Scripts/run-webkit-tests --gtk webaudio/codec-tests/mp3/128kbps-44khz.html

As the --no-show-results parameter is not used, if there is some failing test, run-launcher is invoked to show the diffs, but this call seems to be failing due to be receiving a -2 parameter that is not longer supported.

0 tests ran as expected, 1 didn't:

ScriptError raised: Failed to run "['Tools/Scripts/run-launcher', '--release', '--gtk', u'file:///home/ltilve/WebKit/WebKitBuild/Release/layout-test-results/results.html', '-2']" exit_code: 1 cwd: /home/ltilve/WebKit
Traceback (most recent call last):
  File "/home/ltilve/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py", line 80, in main
    run_details = run(port, options, args, stderr)
  File "/home/ltilve/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py", line 401, in run
    run_details = manager.run(args)
  File "/home/ltilve/WebKit/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py", line 242, in run
    self._port.show_results_html_file(results_path)
  File "/home/ltilve/WebKit/Tools/Scripts/webkitpy/port/gtk.py", line 191, in show_results_html_file
    self._run_script("run-launcher", run_launcher_args)
  File "/home/ltilve/WebKit/Tools/Scripts/webkitpy/port/base.py", line 1283, in _run_script
    output = self._executive.run_command(run_script_command, cwd=self.webkit_base(), decode_output=decode_output, env=env)
  File "/home/ltilve/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 452, in run_command
    (error_handler or self.default_error_handler)(script_error)
  File "/home/ltilve/WebKit/Tools/Scripts/webkitpy/common/system/executive.py", line 370, in default_error_handler
    raise error
ScriptError: Failed to run "['Tools/Scripts/run-launcher', '--release', '--gtk', u'file:///home/ltilve/WebKit/WebKitBuild/Release/layout-test-results/results.html', '-2']" exit_code: 1 cwd: /home/ltilve/WebKit
Comment 1 Lorenzo Tilve 2014-04-23 02:00:27 PDT
Created attachment 229966 [details]
Patch
Comment 2 WebKit Commit Bot 2014-04-23 02:40:18 PDT
Comment on attachment 229966 [details]
Patch

Clearing flags on attachment: 229966

Committed r167702: <http://trac.webkit.org/changeset/167702>
Comment 3 WebKit Commit Bot 2014-04-23 02:40:22 PDT
All reviewed patches have been landed.  Closing bug.