| Summary: | [GTK][WPE] install-dependencies should install the gstreamer plugins | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> | ||||
| Component: | Tools / Tests | Assignee: | Carlos Alberto Lopez Perez <clopez> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | agomez, aperez, calvaris, clopez, ews-watchlist, ltilve, mcatanzaro, pnormand, webkit-bug-importer, zan | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Carlos Alberto Lopez Perez
2020-07-07 08:25:41 PDT
Created attachment 403691 [details]
Patch
I don't know how recommended this is since we have sometimes dependencies on certain patches or certain versions. Maybe these scripts should be changed to only install whatever dependencies are needed to build the flatpak runtime...? The idea of the install-dependencies script is to install every package you will need for developing webkit. The official way of developing developing webkit is with flatpak, but sometimes you might be interested in not using flatpak (I do). So I think its handy that this script makes easier to achieve that. Currently this scripts install every package you will need for - Building webkit with flatpak - Building webkit without flatpak (against your system libraries) - Running tests (with and without flatpak) For me that is fine, but if you are concerned with that maybe we can split this scripts? Perhaps something like this may work: - Tools/flatpak/install-dependencies: install dependencies for building/testing with flatpak - Tools/gtk/install-dependencies: install dependencies for building/testing without flatpak for gtk - Tools/wpe/install-dependencies: install dependencies for building/testing without flatpak for wpe But this looks it can be more confusing than what we have now. Personally I lean towards keeping this as is. I don't see this extra packages installed by the script as a concern. BTW, note this scripts are already installing libgstreamer-plugins-bad1.0-dev and libgstreamer-plugins-base1.0-dev from the system. This patch only changes that the plugins are also installed Committed r264094: <https://trac.webkit.org/changeset/264094> All reviewed patches have been landed. Closing bug and clearing flags on attachment 403691 [details]. |