[Webkit-unassigned] [Bug 37563] new-run-webkit-tests prints out nothing when build-dumprendertree fails

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=37563





--- Comment #2 from Dirk Pranke <dpranke at chromium.org>  2010-09-13 18:57:39 PST ---
(From update of attachment 67507)
> diff --git a/WebKitTools/ChangeLog b/WebKitTools/ChangeLog
> index 2b72256b90d79bd73b6ff47ee2268c2632cb1084..f08aae58e77c64060b673b6459da2b00a45b21d5 100644
> --- a/WebKitTools/ChangeLog
> +++ b/WebKitTools/ChangeLog
> @@ -1,3 +1,14 @@
> +2010-09-13  Kenichi Ishibashi  <bashi at google.com>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        new-run-webkit-tests prints out nothing when build-dumprendertree fails
> +        https://bugs.webkit.org/show_bug.cgi?id=37563
> +
> +        Print error message when build-dumprendertree fails.
> +
> +        * Scripts/webkitpy/layout_tests/port/webkit.py:
> +
>  2010-09-12  Dirk Pranke  <dpranke at chromium.org>
>  
>          Unreviewed, build fix.
> diff --git a/WebKitTools/Scripts/webkitpy/layout_tests/port/webkit.py b/WebKitTools/Scripts/webkitpy/layout_tests/port/webkit.py
> index b085ceb7e68fc6bdfc25bea2aafae8ab8bc92db8..a81b0fd21696b1b311cac81b0cdf650dff2adabb 100644
> --- a/WebKitTools/Scripts/webkitpy/layout_tests/port/webkit.py
> +++ b/WebKitTools/Scripts/webkitpy/layout_tests/port/webkit.py
> @@ -84,10 +84,14 @@ class WebKitPort(base.Port):
>          return ''
>  
>      def _build_driver(self):
> -        return not self._executive.run_command([
> +        exit_code = self._executive.run_command([
>              self.script_path("build-dumprendertree"),
>              self.flag_from_configuration(self._options.configuration),
>          ], return_exit_code=True)
> +        if exit_code != 0:
> +            _log.error("Failed to build DumpRenderTree")
> +            return False
> +        return True

It's probably bad that the generic code in run_webkit_tests.py isn't printing anything in this case. Can you add a _log.error('Build check failed') message at run_webkit_test.py:1420 or thereabouts as well?

Otherwise the change looks good to me (although I'm not a reviewer).

-- Dirk

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.