Commit Graph

165 Commits

Author SHA1 Message Date
Martin Robinson 744ffecdf7 [Flatpak SDK] Increase the startup performance of webkit-flatpak
https://bugs.webkit.org/show_bug.cgi?id=229185

Reviewed by Philippe Normand.

Improve flatpakutils.py by ensuring that we don't make redundant
calls to flatpak during initialization of the data structures. This
saves 5 seconds on every call to build-webkit and run-webkit-tests.

Before:
    $ time ./Tools/Scripts/webkit-flatpak -c true
    real    0m6,297s
    user    0m0,786s
    sys     0m0,513s

After:
    $ time ./Tools/Scripts/webkit-flatpak -c true
    real    0m1,243s
    user    0m0,375s
    sys     0m0,162s

* flatpak/flatpakutils.py:
(FlatpakPackages.__init__): Separate the update into another
method so that it can be called directly when a package is installed.
Add new packages directly to self.packages.
(FlatpakRepos.__init__): Ensure that we only update our repositories
and package list once we've initialized our list of repositories.
(FlatpakRepos.add): Return True if this method actually changed anything
and accept a parameter determining whether an update is done to
the repository and package list.
(FlatpakRepos.is_package_installed): Added this method which replaces
FlatpakRepo.is_app_installed.
(FlatpakRepo.__init__): Remove the app registry because it is duplicated
by FlatpakPackages.
(FlatpakPackage.is_installed): Now call FlatpakRepos.is_package_installed.
(WebkitFlatpak._reset_repository): Set up both repositories here and
pass then as part of the FlatpakRepos constructor.
(WebkitFlatpak.main): Separate out the check which deletes the Flatpak
repository from the one that checks if the packages are installed. This
makes sure we don't install packages and then immediately delete them.
(WebkitFlatpak.check_installed_packages): rename _get_packages to _get_dependency_packages
in order to make it clear that it doesn't return installed packages.
(WebkitFlatpak._get_dependency_packages): Ditto.
(WebkitFlatpak.install_all): Ditto.
(FlatpakRepo.is_app_installed): Deleted.
(WebkitFlatpak._get_packages): Deleted.


Canonical link: https://commits.webkit.org/240629@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@281184 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-08-18 09:13:47 +00:00
Philippe Normand 07b445fd44 [GLib] Expose API to access/modify capture devices states
https://bugs.webkit.org/show_bug.cgi?id=227902

Reviewed by Carlos Garcia Campos.

Source/WebKit:

Introduce new GLib API (and corresponding GObject properties):

webkit_web_view_get_camera_capture_state
webkit_web_view_get_microphone_capture_state
webkit_web_view_set_camera_capture_state
webkit_web_view_set_microphone_capture_state
webkit_web_view_get_display_capture_state
webkit_web_view_set_display_capture_state
webkit_user_media_permission_is_for_display_device

This can be useful in Web browsers willing to indicate the status of the capture devices
currently in use by the WebView.

Covered by API tests.

* UIProcess/API/glib/WebKitUIClient.cpp:
* UIProcess/API/glib/WebKitUserMediaPermissionRequest.cpp:
(webkit_user_media_permission_is_for_display_device):
* UIProcess/API/glib/WebKitWebView.cpp:
(webkitWebViewMediaCaptureStateDidChange):
(webkitWebViewSetProperty):
(webkitWebViewGetProperty):
(webkit_web_view_class_init):
(webkitWebViewConfigureMediaCapture):
(webkit_web_view_get_camera_capture_state):
(webkit_web_view_set_camera_capture_state):
(webkit_web_view_get_microphone_capture_state):
(webkit_web_view_set_microphone_capture_state):
(webkit_web_view_get_display_capture_state):
(webkit_web_view_set_display_capture_state):
* UIProcess/API/glib/WebKitWebViewPrivate.h:
* UIProcess/API/gtk/WebKitUserMediaPermissionRequest.h:
* UIProcess/API/gtk/WebKitWebView.h:
* UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt:
* UIProcess/API/wpe/WebKitUserMediaPermissionRequest.h:
* UIProcess/API/wpe/WebKitWebView.h:
* UIProcess/API/wpe/docs/wpe-1.0-sections.txt:

Tools:

The GTK MiniBrowser is now able to show media capture indicators, through the URI entry. A
new API test was added, checking support for getDisplayMedia().

* MiniBrowser/gtk/BrowserTab.c:
(decidePermissionRequest):
* MiniBrowser/gtk/BrowserWindow.c:
(webViewMediaCaptureStateChanged):
(webViewUriEntryIconPressed):
(browserWindowSwitchTab):
* TestWebKitAPI/Tests/WebKitGLib/TestUIClient.cpp:
(testWebViewUserMediaPermissionRequests):
(testWebViewAudioOnlyUserMediaPermissionRequests):
(testWebViewDisplayUserMediaPermissionRequests):
(beforeAll):
* TestWebKitAPI/glib/WebKitGLib/WebViewTest.cpp:
(displayCaptureChanged):
(WebViewTest::waitUntilDisplayCaptureStateChangedTo):
(microphoneCaptureChanged):
(WebViewTest::waitUntilMicrophoneCaptureStateChangedTo):
(cameraCaptureChanged):
(WebViewTest::waitUntilCameraCaptureStateChangedTo):
* TestWebKitAPI/glib/WebKitGLib/WebViewTest.h:
* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):

Canonical link: https://commits.webkit.org/239867@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@280172 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-07-22 10:54:07 +00:00
Philippe Normand 9b7e4812b5 [Flatpak SDK] Enable network access in the WPE build sandbox
https://bugs.webkit.org/show_bug.cgi?id=225914

Reviewed by Michael Catanzaro.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Allow network access for WPE build jobs, so that CMake is
able to git checkout Cog from github.

Canonical link: https://commits.webkit.org/238595@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@278606 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-06-08 14:16:21 +00:00
Philippe Normand 1c2a6d8af1 [Flatpak SDK] Allow custom build directories
https://bugs.webkit.org/show_bug.cgi?id=226142

Patch by Philippe Normand <pnormand@igalia.com> on 2021-06-08
Reviewed by Adrian Perez de Castro.

By default the build directory remains WebKitBuild/{GTK,WPE}/{Release,Debug}. Builds done
with the Flatpak SDK now support the standard WEBKIT_OUTPUTDIR environment variable, eg.
setting $WEBKIT_OUTPUTDIR to "/path/to/somewhere", the built files will end up in
/path/to/somewhere/{GTK,WPE}/Release.

* flatpak/flatpakutils.py:
(WebkitFlatpak.clean_args):

Canonical link: https://commits.webkit.org/238589@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@278600 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-06-08 10:31:16 +00:00
Philippe Normand c59f9c67d2 [Flatpak SDK] Enable LLVM extension
https://bugs.webkit.org/show_bug.cgi?id=223882

Patch by Philippe Normand <pnormand@igalia.com> on 2021-04-07
Reviewed by Adrian Perez de Castro.

The clang shipped by default in the FDO SDK is not usable for ASan, UBSan (and I guess TSan)
builds, but the Flatpak extension shipping LLVM11 allows this, even though UBSan builds
still fail (linking errors in JSC and WTF) and will require further investigation. Still,
having clang-11 in the SDK is an improvement.

This patch also adds support for cleaning up unused toolchain archives.

* flatpak/flatpakutils.py:
(run_sanitized):
(WebkitFlatpak.load_from_args):
(WebkitFlatpak.clean_args):
(WebkitFlatpak.run_in_sandbox):
(WebkitFlatpak.main):
(WebkitFlatpak.purge_unused_toolchains):
(WebkitFlatpak.pack_toolchain):
(WebkitFlatpak._get_packages):

Canonical link: https://commits.webkit.org/236240@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@275595 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-04-07 09:03:11 +00:00
Philippe Normand cc6c634217 [Flatpak SDK] PATH for rustc no longer set correctly
https://bugs.webkit.org/show_bug.cgi?id=222311

Patch by Philippe Normand <pnormand@igalia.com> on 2021-02-24
Reviewed by Žan Doberšek.

Pass PATH as flatpak --env option, setting is through bwrap no longer works.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):
* flatpak/webkit-bwrap:

Canonical link: https://commits.webkit.org/234504@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@273377 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-24 10:23:21 +00:00
Philippe Normand 5375e87038 [Flatpak SDK] gdb unusable in sandbox
https://bugs.webkit.org/show_bug.cgi?id=222247

Reviewed by Žan Doberšek.

This patch adds support for unix signals disabling in developer builds, for the Flatpak SDK
runtime. Currently SIGINT, SIGHUP and SIGTERM will be ignored by the flatpak and bwrap
processes when they're running gdb in the sandbox runtime. This allows for nicer interactive
debugging sessions where Ctrl-C is actually handled by gdb.

* PlatformGTK.cmake:
* PlatformWPE.cmake:
* flatpak/CMakeLists.txt: Added.
* flatpak/flatpakutils.py:
(nullcontext):
(disable_signals):
(WebkitFlatpak.execute_command):
(WebkitFlatpak.run_in_sandbox):
* flatpak/sigaction-disabler.c: Added.
(real_sigaction):
(sigaction):
* flatpak/webkit-bwrap:

Canonical link: https://commits.webkit.org/234426@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@273240 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-22 12:27:44 +00:00
Philippe Normand 778e6f957b [Flatpak SDK] Add Monado for XR runtime support
https://bugs.webkit.org/show_bug.cgi?id=220738

Patch by Philippe Normand <pnormand@igalia.com> on 2021-02-22
Reviewed by Žan Doberšek.

Tools:

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Allow runtime XR environment setup in the sandbox through
the XR_RUNTIME_JSON variable.

Tools/buildstream:

* elements/sdk-platform.bst:
* elements/sdk/eigen.bst: Added.
* elements/sdk/monado.bst: Added.

Canonical link: https://commits.webkit.org/234425@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@273239 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-22 12:24:38 +00:00
Philippe Normand b8a69600a0 [Flatpak SDK] X11 SSH forwarding fails
https://bugs.webkit.org/show_bug.cgi?id=222246

Reviewed by Žan Doberšek.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Forward the DISPLAY env var to the sandbox.
* flatpak/webkit-bwrap: Set the DISPLAY env var if it was set in the parent process.

Canonical link: https://commits.webkit.org/234424@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@273238 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-22 12:20:16 +00:00
Philippe Normand 31d6de83af [Flatpak SDK] Cog build fails when gst-build is enabled
https://bugs.webkit.org/show_bug.cgi?id=222102

Patch by Philippe Normand <pnormand@igalia.com> on 2021-02-22
Reviewed by Žan Doberšek.

Merge gst-build env vars in the sandbox environment, without overriding previously set PATH
values such as PKG_CONFIG_PATH.

* flatpak/flatpakutils.py:
(WebkitFlatpak.setup_gstbuild):
(WebkitFlatpak.run_in_sandbox):

Canonical link: https://commits.webkit.org/234420@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@273234 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-22 10:21:53 +00:00
Lauro Moura 2c78fa76dd [Flatpak SDK] Fix env forwarding for older flatpak versions
https://bugs.webkit.org/show_bug.cgi?id=221757

Rubber-stamped by Philippe Normand.

Older flatpak versions (pre 1.10) might not have been patched for the
envvar CVE that triggered the new behavior and would still be
receiving the environment through envvars instead of the --args file
descriptor.

* flatpak/webkit-bwrap:

Canonical link: https://commits.webkit.org/233973@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272733 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-11 17:09:48 +00:00
Lauro Moura b03574d8a9 [Flatpak SDK] Support multiple builds running at the same time
https://bugs.webkit.org/show_bug.cgi?id=221711

Reviewed by Philippe Normand.

Instead of writing the environment to a file, read the --setenv
arguments from the --args file descriptor passed to webkit-bwrap like
bwrap itself would do.

* flatpak/flatpakutils.py: Do not write the env file anymore
(WebkitFlatpak.run_in_sandbox):
* flatpak/webkit-bwrap: Read the environment from the args file
descriptor.


Canonical link: https://commits.webkit.org/233957@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272712 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-11 09:29:22 +00:00
Philippe Normand ba509165f1 [WPE] Optionally build Cog as external project and replacement for MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=221518

Reviewed by Adrian Perez de Castro.

.:

* Source/cmake/OptionsWPE.cmake: New variable for JSC forwarded headers path, useful
uninstalled builds of projects depending on JSC.

Source/WebKit:

* PlatformWPE.cmake: Produce uninstalled pkg-config files for wpe-webkit and web-extension
libraries. Also populate the JSC ForwardedHeaders path during the build.
* wpe/wpe-web-extension-uninstalled.pc.in: Added.
* wpe/wpe-webkit-uninstalled.pc.in: Added.

Tools:

By passing `-DENABLE_COG=ON` during the WPE build, the Cog browser is now built as an
external project. The repository is cloned in Tools/wpe/cog and the code can be edited
there.

The run-minibrowser --wpe script will now first try to execute cog and fallback to
MiniBrowser if it wasn't found. Cog is also a MiniBrowser, but with some additional
features, see https://github.com/Igalia/cog for more informations.

* PlatformWPE.cmake: Configure Cog as an external project in developer builds.
* Scripts/webkitpy/port/wpe.py:
(WPEPort.run_minibrowser): First check for cog, fallback to MiniBrowser if not found.
* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Set pkg-config path to lookup libraries in the WebKit's
build directory as well.

Canonical link: https://commits.webkit.org/233902@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272643 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-10 10:18:43 +00:00
Daniel Kolesa cd9cc53972 Allow icecream toolchain override in flatpakutils
https://bugs.webkit.org/show_bug.cgi?id=221605

Patch by Daniel Kolesa <dkolesa@igalia.com> on 2021-02-09
Reviewed by Philippe Normand.

This allows the user to specify a custom icecream toolchain.
Usually you will not want to do this, but it is useful e.g. when
you have a machine in your cluster that does not have a compiler
in the SDK. You can either force the whole version string by using
ICECC_VERSION_OVERRIDE, or append to the original string with
ICECC_VERSION_APPEND.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):

Canonical link: https://commits.webkit.org/233849@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272577 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-09 14:53:58 +00:00
Lauro Moura a68ee2589e [Flatpak SDK] Print stderr when coredumpctl fails
https://bugs.webkit.org/show_bug.cgi?id=221448

Reviewed by Philippe Normand.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_gdb):

Canonical link: https://commits.webkit.org/233739@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272415 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-02-05 16:19:08 +00:00
Lauro Moura d778028b6d [Flatpak SDK] Flatpak environment JSON is prematurely deleted with flatpak-spawn calls
https://bugs.webkit.org/show_bug.cgi?id=221070

Reviewed by Philippe Normand.

To avoid nested webkit-flatpak calls overwriting/deleting whe
environment file, only create if there is no such file and only delete
in the same process that created it.

This approach has the drawback of limiting nested calls changing the
variable values and a file undeleted after a crash might avoid the
next normal invocation creating it.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):


Canonical link: https://commits.webkit.org/233443@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272018 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-01-28 18:26:52 +00:00
Lauro Moura 0a89544226 [Flatpak SDK] Avoid "Invalid byte sequence in conversion input" errors and other encoding issues
https://bugs.webkit.org/show_bug.cgi?id=221068

Reviewed by Philippe Normand.

* flatpak/flatpakutils.py:
(WebkitFlatpak.execute_command): Decode the arguments before printing.
(WebkitFlatpak.run_in_sandbox): Avoid iterating mutating dict and
export LANG.
* flatpak/webkit-bwrap: Ignore LANG as it's exported by
flatpakutils.py.


Canonical link: https://commits.webkit.org/233436@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@272011 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-01-28 12:48:07 +00:00
Philippe Normand 3bb9767424 [Flatpak SDK] Flatpak 1.10 environment variable issues
https://bugs.webkit.org/show_bug.cgi?id=220781

Patch by Philippe Normand  <pnormand@igalia.com> and Lauro Moura <lmoura@igalia.com> on 2021-01-27
Reviewed by Adrian Perez de Castro.

Flatpak no longer passes its env variables to bwrap, so the workaround is now to serialize
it on-disk in a JSON file and load it from our webkit-bwrap script.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):
* flatpak/webkit-bwrap:

Canonical link: https://commits.webkit.org/233384@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@271938 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2021-01-27 09:05:59 +00:00
Lauro Moura c40835b579 [Flapak SDK] Allow setting custom SDK repository locations and other small fixes
https://bugs.webkit.org/show_bug.cgi?id=219884

Reviewed by Philippe Normand.

Tools:

- Update envvar to match the WEBKIT_FLATPAK prefix
- Give precedence to the verbose flag instead of envvar
- Enable logging earlier to be used in the parse step
- Error out if the user provides an user repo that is not accessible.

* flatpak/flatpakutils.py:
(WebkitFlatpak.load_from_args):
(WebkitFlatpak.clean_args):

Tools/buildstream:

- Allow inheriting the RSYNC info from envvar for local testing.
- Add a 'dry-push-repo' command to be able to perform push-repo dry-runs.

* Makefile:


Canonical link: https://commits.webkit.org/232647@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@271050 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-12-22 12:22:49 +00:00
Lauro Moura 79f5bf0b31 [Flapak SDK] Add envvar to enable verbose mode
https://bugs.webkit.org/show_bug.cgi?id=219744

Reviewed by Adrian Perez de Castro.

While webkit-flatpak has the --verbose option, some scripts like
run-minibrowser, run-gtk-tests etc might not expose it to the user.

This change checks for WEBKIT_SDK_VERBOSE in order to be able to
control it regardless of command line options.

* flatpak/flatpakutils.py:
(WebkitFlatpak.load_from_args):


Canonical link: https://commits.webkit.org/232300@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@270633 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-12-10 18:07:19 +00:00
Carlos Alberto Lopez Perez 0055641472 [FlatPak] update-webkitgtk-libs fails after a clean build
https://bugs.webkit.org/show_bug.cgi?id=218724

Reviewed by Philippe Normand.

The issue was caused because when adding a new flatpak repository
via the method FlatpakRepos.add() that repository is not added to
the internal list of available repositories inside the object FlatpakRepos.
So then the check on setup_builddir() added in r268542 failed because
the internal list of repositories on the object FlatpakRepos() was
empty on the first run (after a clean build).
To fix this we ensure to re-generate the internal list of flatpak
repositories any time that a new reporistory is added by calling
FlatpakRepos.update() after FlatpakRepos.add()

On top of that fix, we add another fix to make the code more robust.
Now it allows the generation of toolchains to fail without causing
a fatal error. Also a new check is added in order to retry to generate
the toolchains in the next run if is detected that they were not
correctly generated.

* flatpak/flatpakutils.py:
(FlatpakRepos.add):
(WebkitFlatpak.load_from_args):
(WebkitFlatpak.main):
(WebkitFlatpak.check_toolchains_generated):
(WebkitFlatpak.pack_toolchain):


Canonical link: https://commits.webkit.org/231699@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@269949 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-11-18 10:44:41 +00:00
Jonathan Bedard a07c5274bd [webkitpy] Use allowlist and blocklist
https://bugs.webkit.org/show_bug.cgi?id=217985
<rdar://problem/70499499>

Reviewed by Aakash Jain.

* Scripts/webkitpy/pylintrc:
* Scripts/webkitpy/style/checkers/cpp.py:
(check_namespace_indentation):
* Scripts/webkitpy/w3c/common.py:
(is_basename_skipped):
* flatpak/flatpakutils.py:
(WebkitFlatpak.setup_gstbuild):


Canonical link: https://commits.webkit.org/231067@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@269205 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-10-30 20:32:08 +00:00
Truitt Savell 98be7bf063 Unreviewed, reverting r268761.
May have casued webrtc crashes on Catalina Debug.

Reverted changeset:

"[webkitpy] Use allowlist and blocklist"
https://bugs.webkit.org/show_bug.cgi?id=217985
https://trac.webkit.org/changeset/268761

Canonical link: https://commits.webkit.org/230844@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@268938 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-10-23 20:13:34 +00:00
Carlos Alberto Lopez Perez 18a4aba4ea update-webkitgtk-libs and run-webkit-tests fail silently if flatpak has problems updating
https://bugs.webkit.org/show_bug.cgi?id=218047

Reviewed by Philippe Normand.

This scripts were returning zero status when flatpak failed to
update in some cases. Fix them to return a non-zero status.

* Scripts/update-webkit-flatpak: Add missing sys.exit call with the return code.
* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): In python false is zero when casted to int.

Canonical link: https://commits.webkit.org/230766@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@268827 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-10-21 21:37:25 +00:00
Jonathan Bedard 11bd95f540 [webkitpy] Use allowlist and blocklist
https://bugs.webkit.org/show_bug.cgi?id=217985
<rdar://problem/70499499>

Reviewed by Aakash Jain.

* Scripts/webkitpy/pylintrc:
* Scripts/webkitpy/style/checkers/cpp.py:
(check_namespace_indentation):
* Scripts/webkitpy/w3c/common.py:
(is_basename_skipped):
* flatpak/flatpakutils.py:
(WebkitFlatpak.setup_gstbuild):


Canonical link: https://commits.webkit.org/230704@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@268761 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-10-20 21:40:40 +00:00
Philippe Normand 9545bc8383 [Flatpak SDK] flatpakutils.py reports "KeyError: 'gcc'" error if using with icecc and toolchains aren't generated yet
https://bugs.webkit.org/show_bug.cgi?id=217799

Patch by Philippe Normand <pnormand@igalia.com> on 2020-10-17
Reviewed by Fujii Hironori.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Gracefully exit with an error message when the icecc
configuration failed to complete.

Canonical link: https://commits.webkit.org/230597@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@268639 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-10-17 08:16:12 +00:00
Lauro Moura f8d3f1d111 [Flatpak SDK] Check for installed packages in setup_buildir
https://bugs.webkit.org/show_bug.cgi?id=217768

Reviewed by Philippe Normand.

During a cleanup of one of the bots, setup_builddir failed with a null
self.sdk member, which is set in get_packages.

While I could not reproduce it again, this commit prevents this by
checking whether the packages are installed before continuing the
original setup_builddir steps.

* flatpak/flatpakutils.py:
(WebkitFlatpak.setup_builddir): Check for installed packages.
(WebkitFlatpak.run_in_sandbox): Check for setup_builddir return code.
(WebkitFlatpak.check_installed_packages): Extracted from
setup_dev_env.
(WebkitFlatpak.setup_dev_env): Call extracted method.


Canonical link: https://commits.webkit.org/230505@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@268542 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-10-15 19:09:01 +00:00
Victor Jaquez f675000064 [Flatpak SDK] bwrap: add pipewire socket as bind mount
https://bugs.webkit.org/show_bug.cgi?id=216606

Patch by Víctor Manuel Jáquez Leal <vjaquez@igalia.com> on 2020-09-17
Reviewed by Philippe Normand.

Add pipewire socket to bwrap as a try binding mount.

* flatpak/webkit-bwrap:

Canonical link: https://commits.webkit.org/229474@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@267200 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-09-17 18:07:45 +00:00
Philippe Normand e0c2827f62 [Flatpak] webkit-bwrap drive-by cleanups
Rubber-stamped by Adrian Perez de Castro.

* flatpak/webkit-bwrap: Extend lists with tuples, use itertools and map to build the final
command line.

Canonical link: https://commits.webkit.org/229425@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@267149 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-09-16 16:39:23 +00:00
Philippe Normand 49b00aa41d [Flatpak SDK] Migration to SDK 0.3 branch
https://bugs.webkit.org/show_bug.cgi?id=216073

Reviewed by Adrian Perez de Castro.

Tools:

As this new SDK branch is based on the FDO 20.08 SDK which breaks ABI, existing WebKit
builds done with the 0.2 branch will no longer be usable. Developers are advised to remove
their WebKitBuild/{GTK,WPE} local directories after updating to 0.3.

* Scripts/webkitpy/port/gtk.py:
(GtkPort.setup_environ_for_server): Since we now ship Mesa directly in the SDK we can easily
configure the prefix where DRI modules are installed. This is now the same as in the JHBuild
sysroot.
* flatpak/flatpakutils.py:
(WebkitFlatpak.__init__): Bump to 0.3 branch. Also un-hardcode host platform architecture so
that we can more easily add aarch64 bots later.
(WebkitFlatpak._get_packages): The GL extension is no longer needed because Mesa is shipped
directly within our SDK now. Also install the rust extension, which is useful eg when doing
development with gst-build.
* flatpak/webkit-bwrap: Add rustc path in PATH.

LayoutTests:

* fast/xsl/resources/xslt-enc-cyr.xsl: Add name attribute in xsl templates, as mandated by the spec.
* fast/xsl/resources/xslt-enc.xsl: Ditto.
* fast/xsl/resources/xslt-enc16.xsl: Ditto.
* fast/xsl/resources/xslt-text.xsl: Ditto.
* platform/glib/fast/xsl/xslt-mismatched-tags-in-xslt-expected.txt: Added.
* platform/glib/fast/xsl/xslt-processor-expected.txt: Added.

Canonical link: https://commits.webkit.org/229300@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@267010 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-09-14 09:51:24 +00:00
Philippe Normand 4e0c11cc20 [Flatpak SDK] Update to FDO 20.08
https://bugs.webkit.org/show_bug.cgi?id=215763

Reviewed by Žan Doberšek.

.:

* .gitignore: Ignore buildstream local Flatpak repository. This is created only when
manually building the SDK.

Tools:

Highlights:

- ICU 67 (see bug 215327)
- gcc update from 9.3.0 to 10.0
- clang update from 8.0 to 10.0
- perf is now included
- usable pipewire (host needs a working daemon and recent enough xdg-desktop-portal though)
- Mesa is no longer loaded from the flathub GL extension, but included in our downstream SDK, with debug symbols

This patch only includes the build definitions updates, bumping the SDK branch to 0.3. The
actual migration will be landed in a follow-up patch.

* buildstream/elements/flatpak-platform-extensions.bst:
* buildstream/elements/flatpak-runtimes.bst:
* buildstream/elements/flatpak/platform.bst:
* buildstream/elements/flatpak/sdk.bst:
* buildstream/elements/freedesktop-sdk.bst:
* buildstream/elements/sdk-build-depends/swig.bst:
* buildstream/elements/sdk-platform.bst:
* buildstream/elements/sdk.bst:
* buildstream/elements/sdk/at-spi2-atk.bst: Removed.
* buildstream/elements/sdk/at-spi2-core.bst: Removed.
* buildstream/elements/sdk/atk.bst: Removed.
* buildstream/elements/sdk/brotli.bst: Removed.
* buildstream/elements/sdk/glib.bst:
* buildstream/elements/sdk/gobject-introspection.bst: Removed.
* buildstream/elements/sdk/graphene.bst:
* buildstream/elements/sdk/gst-plugins-bad.bst:
* buildstream/elements/sdk/gst-plugins-good.bst:
* buildstream/elements/sdk/gstreamer.bst:
* buildstream/elements/sdk/gtk+-3.bst:
* buildstream/elements/sdk/gtk.bst:
* buildstream/elements/sdk/libdrm.bst: Added.
* buildstream/elements/sdk/libgcrypt.bst:
* buildstream/elements/sdk/libsecret.bst:
* buildstream/elements/sdk/libsoup.bst:
* buildstream/elements/sdk/libsrtp2.bst: Removed.
* buildstream/elements/sdk/mesa.bst: Added.
* buildstream/elements/sdk/pygobject.bst:
* buildstream/elements/sdk/python2-pygobject.bst:
* buildstream/elements/sdk/rr.bst:
* buildstream/elements/sdk/shared-mime-info.bst:
* buildstream/elements/sdk/v4lutils.bst: Removed.
* buildstream/elements/sdk/woff2.bst:
* buildstream/elements/sdk/xorg-server.bst:
* buildstream/elements/test-infra/httpd.bst:
* buildstream/patches/fdo/0001-Pango-Bump-to-1.45-master.patch: Removed.
* buildstream/patches/fdo/0001-dav1d-Bump-to-0.7.0.patch: Removed.
* buildstream/patches/gtk-meson-Bump-to-latest-version-of-the-3.24-branch.patch: Removed.
* buildstream/patches/libsecret-b53e9876f4611525b7a8f69a3110fbec51dcede0.patch: Removed.
* buildstream/patches/mesa/mesa_libdrm_deps.patch: Added.
* buildstream/patches/xserver-search-for-DRI-drivers-at-LIBGL_DRIVERS_PATH-environ.patch:
Removed because a variant was landed upstream 2 years ago:
319af6f471
* buildstream/project.conf:
* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): White-list a few Mesa environment variables.

Canonical link: https://commits.webkit.org/228860@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266455 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-09-02 13:29:55 +00:00
Philippe Normand cc47da5693 [Flatpak] SDK branch upgrade path is broken
https://bugs.webkit.org/show_bug.cgi?id=215927

Patch by Philippe Normand <pnormand@igalia.com> on 2020-08-31
Reviewed by Adrian Perez de Castro.

The development environment (flatpak install) needs to be performed before attempting to
rebuild toolchain archives. In most cases this wasn't an issue, but when the SDK needs to be
upgraded to a new ABI/branch, the previous version needs to be removed, that triggers a new
`flatpak install` and toolchain archives update, which need then to be done in the right
order.

* flatpak/flatpakutils.py:
(WebkitFlatpak.main):

Canonical link: https://commits.webkit.org/228785@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@266349 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-08-31 09:45:02 +00:00
Philippe Normand 7441cdce0e [Flatpak SK] Enable GNOME keyring support in subversion
https://bugs.webkit.org/show_bug.cgi?id=215498

Patch by Philippe Normand <pnormand@igalia.com> on 2020-08-14
Reviewed by Sergio Villar Senin.

* buildstream/elements/sdk/subversion.bst: Enable GNOME Keyring support, which pulls into libsecret.
* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Allow session dbus access to the host secret service.

Canonical link: https://commits.webkit.org/228262@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@265666 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-08-14 10:08:04 +00:00
Jonathan Bedard 4e6bb0f846 [webkitpy] Use webkitcorepy's auto installer for coverage
https://bugs.webkit.org/show_bug.cgi?id=215419
<rdar://problem/66922400>

Reviewed by Stephanie Lewis.

* Scripts/webkitpy/__init__.py: Add coverage and it's dependencies.
* Scripts/webkitpy/test/main.py:
(Tester._run_tests): Stop relying on webkitpy's autoinstaller.
* Scripts/webkitpy/thirdparty/__init__.py:
(AutoinstallImportHook.find_module): Remove _install_coverage, _install_toml.
(AutoinstallImportHook._install_coverage): Deleted.
(AutoinstallImportHook._install_toml): Deleted.
* Scripts/webkitpy/thirdparty/__init___unittest.py:
(ThirdpartyTest.test_imports): Remove coverage.
* flatpak/flatpakutils.py: Stop relying on webkitpy's autoinstaller.


Canonical link: https://commits.webkit.org/228225@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@265607 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-08-13 15:42:38 +00:00
Philippe Normand 24fa973b41 [GTK][WPE] Add MiniBrowser wrapper env vars and white-list MESA env vars in flatpak
https://bugs.webkit.org/show_bug.cgi?id=215361

Patch by Philippe Normand <pnormand@igalia.com> on 2020-08-11
Reviewed by Žan Doberšek.

* Scripts/webkitpy/port/gtk.py:
(GtkPort.run_minibrowser): Add support for the $WEBKIT_MINI_BROWSER_PREFIX env var which can
be useful when debugging MiniBrowser with gdb or rr.
* Scripts/webkitpy/port/wpe.py:
(WPEPort.run_minibrowser): Ditto.
* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): White-list MESA environment variables in the sandbox.

Canonical link: https://commits.webkit.org/228147@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@265496 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-08-11 13:27:23 +00:00
Philippe Normand 93a1fbb8b8 REGRESSION(r264096): [flatpak] Clean gst-build within build-webkit fails
https://bugs.webkit.org/show_bug.cgi?id=214080

Patch by Philippe Normand <pnormand@igalia.com> on 2020-08-03
Reviewed by Adrian Perez de Castro.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Don't attempt to configure the gst-env when starting sccache-dist.

Canonical link: https://commits.webkit.org/227880@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@265194 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-08-03 12:26:11 +00:00
Philippe Normand 01faeeb2c0 [Flatpak SDK] Unable to build without Internet connection
https://bugs.webkit.org/show_bug.cgi?id=213920

Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-09
Reviewed by Sergio Villar Senin.

Set the flathub ostree repo url explicitely in order to avoid retrieving it from the
flatpakrepo remote file. This is unlikely to change.

* flatpak/flatpakutils.py:
(WebkitFlatpak._get_packages):

Canonical link: https://commits.webkit.org/226937@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@264165 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-07-09 11:43:35 +00:00
Philippe Normand 94d3dae257 [Flatpak SDK] Pre-start sccache server
https://bugs.webkit.org/show_bug.cgi?id=213973

Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-08
Reviewed by Žan Doberšek.

Pre-start the sccache-dist server when the flatpak sandbox is being invoked for a build.
Usually the first sccache client process would do it but it seems to be racy when a large
number of clients is being started simultaneously.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):

Canonical link: https://commits.webkit.org/226880@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@264096 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-07-08 11:22:43 +00:00
Diego Pino Garcia 9bf6957b34 [Flatpak] Avoid error message 'error: org.webkit.Sdk/*unspecified*/*unspecified*' not installed
https://bugs.webkit.org/show_bug.cgi?id=213871

Reviewed by Philippe Normand.

The error message happens when requesting a Flatpak SDK version but there's no Flatpak SDK installed yet.
In that case, ignore stderr to filter out the message.

* flatpak/flatpakutils.py:
(run_sanitized):
(FlatpakObject.flatpak):
(FlatpakObject.version):


Canonical link: https://commits.webkit.org/226874@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@264064 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-07-08 08:39:59 +00:00
Philippe Normand 92e94cb76f [Flatpak SDK] Initial installation with IceCC enabled fails
https://bugs.webkit.org/show_bug.cgi?id=213833

Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-08
Reviewed by Adrian Perez de Castro.

Avoid attempting to use the IceCC environment during the toolchain
archive update. The initial SDK installation triggers this when
IceCC is enabled on the command-line or via CCACHE_PREFIX, after
resetting the internal toolchain mapping. So the solution is to
opt-out of this runtime configuration setup, which is useless
anyway when the toolchain archive is being generated.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):
(WebkitFlatpak.pack_toolchain):

Canonical link: https://commits.webkit.org/226873@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@264063 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-07-08 08:20:05 +00:00
Philippe Normand 572af84bf4 [Flatpak SDK] Misc updates
https://bugs.webkit.org/show_bug.cgi?id=213968

Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-06
Reviewed by Sergio Villar Senin.

* buildstream/README.rst: ReST format fixes and doc additions.
* buildstream/elements/sdk/perl-term-readkey.bst: Remove perllocal.pod as it can clash with other recipes.
* flatpak/flatpakutils.py:
(run_in_sandbox_if_available): Add --flatpak- option pass-through support. This was lost
after the migration of run-minibrowser to python.

Canonical link: https://commits.webkit.org/226785@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263963 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-07-06 08:35:31 +00:00
Alicia Boya Garcia 83c4f319a4 [flatpak] Ensure en_US.UTF-8 locale
https://bugs.webkit.org/show_bug.cgi?id=212459

Reviewed by Philippe Normand.

When developing WebKit we want as few system differences as possible
to make tests reproducible.

en_US.UTF-8 is indeed the only locale installed in our development
flatpak, but our scripts were not correctly clearing most locale
environment variables, which resulted in warnings when running some
tools (notably perl) when the user OS has a different locale.

This patch ensures all locale environment variables are cleared and
LANG is set to en_US.UTF-8. It also removes now redundant code from
flatpakutils.py.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox):
* flatpak/webkit-bwrap:


Canonical link: https://commits.webkit.org/226303@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263397 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-23 15:48:20 +00:00
Philippe Normand 9ab0d3f0b8 [Flatpak SDK] relax restriction on build-webkit script name
https://bugs.webkit.org/show_bug.cgi?id=213432

Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-23
Reviewed by Žan Doberšek.

* flatpak/flatpakutils.py:
(WebkitFlatpak.run_in_sandbox): Match the build-webkit string in the whole command-line
argument and thus allow alternative implementations of build-webkit. Also let the
MAX_CPU_LOAD env var in the sandbox.

Canonical link: https://commits.webkit.org/226296@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263390 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-23 09:15:41 +00:00
Philippe Normand 6c14d0731e [Flatpak SDK] Persist sccache auth token across flatpak updates
https://bugs.webkit.org/show_bug.cgi?id=213427

Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-22
Reviewed by Žan Doberšek.

After a successful flatpak update check if an sccache auth token exists and reuse it when
updating the sccache config file.

* flatpak/flatpakutils.py:
(WebkitFlatpak.save_config):

Canonical link: https://commits.webkit.org/226250@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263344 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-22 09:23:19 +00:00
Philippe Normand a46f308758 REGRESSION(r263029): [Flatpak SDK] Broke initial SDK installation
Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-16
Rubber-stamped by Žan Doberšek.

* flatpak/flatpakutils.py:
(FlatpakObject.version): Gracefully return an empty version when the flatpak info command
fails. This usually means the app has not been installed yet.

Canonical link: https://commits.webkit.org/226020@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263083 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-16 08:56:44 +00:00
Philippe Normand 18ddc481fb [Flatpak SDK] flatpak update still not displayed
https://bugs.webkit.org/show_bug.cgi?id=213180

Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-15
Reviewed by Žan Doberšek.

Added a new --version option to the script. We no longer gather the flatpak update command
output to know if something was updated or not. Instead we get the version of the SDK prior
to the update operation and compare that with the version after the update.

* flatpak/flatpakutils.py:
(FlatpakObject.version):
(WebkitFlatpak.load_from_args):
(WebkitFlatpak.main):

Canonical link: https://commits.webkit.org/225967@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263029 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-15 09:31:24 +00:00
Philippe Normand 142553213c [Flatpak SDK] Add libavif
https://bugs.webkit.org/show_bug.cgi?id=212964
<rdar://problem/64291035>

Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-15
Reviewed by Žan Doberšek.

Because of the ABI bump of dav1d, which ffmpeg depends on, ffmpeg needs to be rebuilt. So
this is a good opportunity to include it in the SDK and not rely on the ffmpeg-full
extension anymore, because it doesn't provide debug symbols.

* buildstream/elements/flatpak/platform.bst:
* buildstream/elements/flatpak/sdk.bst:
* buildstream/elements/freedesktop-sdk.bst:
* buildstream/elements/sdk-platform.bst:
* buildstream/elements/sdk/ffmpeg.bst: Added.
* buildstream/elements/sdk/gst-libav.bst:
* buildstream/elements/sdk/libavif.bst: Added.
* buildstream/elements/sdk/ruby-highline.bst: Remove the spec source, un-needed.
* buildstream/patches/fdo/0001-dav1d-Bump-to-0.7.0.patch: Added.
* buildstream/patches/fdo/0001-ffmpeg-Add-MS-MPEG-DivX-variants.patch: Removed.
* buildstream/patches/fdo/0002-ffmpeg-Add-VP6-and-Sorenson-Spark-video-codecs.patch: Removed.
* buildstream/patches/fdo/0003-ffmpeg-Add-Intel-Indeo-and-Cinepak-video-support.patch: Removed.
* buildstream/patches/fdo/0004-ffmpeg-Add-MPEG-2-video-decoder.patch: Removed.
* buildstream/patches/fdo/0005-ffmpeg-Add-msmpeg4v3-support.patch: Removed.
* buildstream/project.conf:
* flatpak/flatpakutils.py:
(WebkitFlatpak._get_packages):

Canonical link: https://commits.webkit.org/225965@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263027 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-15 09:20:01 +00:00
Philippe Normand a6402b81f2 [Flatpak SDK] Remove webkit-build-directory dependency from flatpakutils
https://bugs.webkit.org/show_bug.cgi?id=213179

Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-15
Reviewed by Žan Doberšek.

This makes webkit-flatpak usable without installing additional perl modules (version) on the host OS.

* flatpak/flatpakutils.py:
(WebkitFlatpak.load_from_args): Make --release and --debug mutually exclusive, and default to Release builds.
(WebkitFlatpak.__init__):
(WebkitFlatpak.clean_args): No need to pull in webkitpy which pulls webkitdirs.pm only to
determine the default configuration. Assume we use Release by default.

Canonical link: https://commits.webkit.org/225962@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263024 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-15 09:09:04 +00:00
Victor Jaquez 32ae062ab3 [Flatpak][GStreamer] enhance GST_ environment variable passing
https://bugs.webkit.org/show_bug.cgi?id=212905

Patch by Víctor Manuel Jáquez Leal <vjaquez@igalia.com> on 2020-06-09
Reviewed by Philippe Normand.

Handle the passing of several GST environment variables,
particularly GST_DEBUG, which, when using * symbol, python adds
quotes, but they break the parsing in GStreamer.

Also the variables are handled by gst-env.py when using
GST_BUILD_PATH, so they are filtered out when it's exported,
avoiding duplication. But it GST_BUILD_PATH is not defined, the
variables are passed to flatpak.

* flatpak/flatpakutils.py:
(WebkitFlatpak.setup_gstbuild):
(WebkitFlatpak.run_in_sandbox):

Canonical link: https://commits.webkit.org/225760@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@262782 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-09 08:33:56 +00:00
Philippe Normand 9d67161c21 [Flatpak SDK] Install and update don't provide any terminal output
https://bugs.webkit.org/show_bug.cgi?id=212579

Patch by Philippe Normand <pnormand@igalia.com> on 2020-06-01
Reviewed by Žan Doberšek.

The `show_output` option was removed from the process execution routines. We now assume the
output will be displayed, unless the call-site asks `gather_output`. This change makes
webkit-flatpak more chatty, which is nice during install/update operations.

* flatpak/flatpakutils.py:
(run_sanitized):
(FlatpakObject.flatpak):
(FlatpakPackage.install):
(FlatpakPackage.update):
(WebkitFlatpak.setup_builddir):
(WebkitFlatpak.run_in_sandbox):

Canonical link: https://commits.webkit.org/225432@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@262384 268f45cc-cd09-0410-ab3c-d52691b4dbfc
2020-06-01 18:10:06 +00:00