Bug 187187 - REGRESSION(r233362): check_flatpak() should not be verbose when flatpak environment has not been initialized
Summary: REGRESSION(r233362): check_flatpak() should not be verbose when flatpak envir...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 186771
  Show dependency treegraph
 
Reported: 2018-06-29 11:19 PDT by Carlos Alberto Lopez Perez
Modified: 2018-07-01 22:40 PDT (History)
5 users (show)

See Also:


Attachments
Patch. (2.75 KB, patch)
2018-06-30 04:48 PDT, Thibault Saunier
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2018-06-29 11:19:53 PDT
It happens that when the flatpak environment has not been initialized, the scripts still try to run the flatpak binary in the function check_flatpak() in flatpakutils.py


For example, If i just uninstall flatpak and I try to run I get

$ Tools/Scripts/run-webkit-tests --gtk --release

You need to install flatpak >= 0.10.0 to be able to use the 'Tools/Scripts/run-webkit-tests' script.

You can find some informations about how to install it for your distribution at:
    * http://flatpak.org/

20:15:13.186 21463 Using port 'gtk-wk2'
[...]

Or on the bots, just check the ouput generate-jsc-bundle from the release bot, the noise about flatpak there:

https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Build%29/builds/13303/steps/generate-jsc-bundle/logs/stdio


I think this should be avoided. 

The function check_flatpak() should avoid doing any check (or print any warning) if the flatpak environment has not been initialized
Comment 1 Thibault Saunier 2018-06-30 04:48:39 PDT
Created attachment 344011 [details]
Patch.
Comment 2 WebKit Commit Bot 2018-06-30 07:24:28 PDT
Comment on attachment 344011 [details]
Patch.

Clearing flags on attachment: 344011

Committed r233395: <https://trac.webkit.org/changeset/233395>
Comment 3 WebKit Commit Bot 2018-06-30 07:24:30 PDT
All reviewed patches have been landed.  Closing bug.