Bug 190247 - [WPE][GTK] API test runner fails to respect default build type
Summary: [WPE][GTK] API test runner fails to respect default build type
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-03 06:59 PDT by Michael Catanzaro
Modified: 2018-11-05 08:45 PST (History)
5 users (show)

See Also:


Attachments
Patch (2.00 KB, patch)
2018-10-03 07:01 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2018-10-03 06:59:48 PDT
$ run-gtk-tests
No handlers could be found for logger "webkitpy.thirdparty.autoinstalled.keyring.backend"
Could not determine build directory.


Problem is it's looking for a release build, even though I've run set-webkit-configuration --debug. It should look for a debug build unless I explicitly pass --release.
Comment 1 Michael Catanzaro 2018-10-03 07:01:08 PDT
Created attachment 351517 [details]
Patch
Comment 2 Adrian Perez 2018-10-03 08:17:40 PDT
Comment on attachment 351517 [details]
Patch

LGTM, informally.
Comment 3 Carlos Garcia Campos 2018-11-05 02:24:30 PST
Comment on attachment 351517 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=351517&action=review

> Tools/glib/api_test_runner.py:51
> +        if self._options.debug:
> +            self._build_type = "Debug"
> +        elif self._options.release:
> +            self._build_type = "Release"
> +        else:
> +            self._build_type = self._port.default_configuration()
> +        common.set_build_types((self._build_type,))

Why did you move this after the port and driver initialization?
Comment 4 Michael Catanzaro 2018-11-05 08:20:17 PST
Comment on attachment 351517 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=351517&action=review

>> Tools/glib/api_test_runner.py:51
>> +        common.set_build_types((self._build_type,))
> 
> Why did you move this after the port and driver initialization?

To be able to use self._port on the line above!
Comment 5 WebKit Commit Bot 2018-11-05 08:45:27 PST
Comment on attachment 351517 [details]
Patch

Clearing flags on attachment: 351517

Committed r237805: <https://trac.webkit.org/changeset/237805>
Comment 6 WebKit Commit Bot 2018-11-05 08:45:29 PST
All reviewed patches have been landed.  Closing bug.