Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 1 | From fd8f49dba8c09d47425da80f5faab3bfa4a7c962 Mon Sep 17 00:00:00 2001 |
| 2 | From: Jose Quaresma <quaresma.jose@gmail.com> |
| 3 | Date: Sat, 10 Oct 2020 19:09:03 +0000 |
| 4 | Subject: [PATCH 1/3] gstpluginloader: when env var is set do not fall through |
| 5 | to system plugin scanner |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 6 | |
| 7 | If we set a custom GST_PLUGIN_SCANNER env var, then we probably want to use that and only that. |
| 8 | |
| 9 | Falling through to the one installed on the system is problamatic in cross-compilation |
| 10 | environemnts, regardless of whether one pointed to by the env var succeeded or failed. |
| 11 | |
Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 12 | taken from: |
| 13 | http://cgit.openembedded.org/openembedded-core/commit/meta/recipes-multimedia/gstreamer/gstreamer1.0/0001-gst-gstpluginloader.c-when-env-var-is-set-do-not-fal.patch?id=0db7ba34ca41b107042306d13a6f0162885c123b |
Andrew Geissler | 6ce62a2 | 2020-11-30 19:58:47 -0600 | [diff] [blame] | 14 | |
Andrew Geissler | c926e17 | 2021-05-07 16:11:35 -0500 | [diff] [blame] | 15 | Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/669> |
| 16 | |
| 17 | Upstream-Status: Backport [https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/9f958058697e6fbf5bde325228034572331d1a3a] |
| 18 | |
Andrew Geissler | 6ce62a2 | 2020-11-30 19:58:47 -0600 | [diff] [blame] | 19 | Signed-off-by: Jose Quaresma <quaresma.jose@gmail.com> |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 20 | --- |
Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 21 | gst/gstpluginloader.c | 15 +++++++-------- |
| 22 | 1 file changed, 7 insertions(+), 8 deletions(-) |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 23 | |
| 24 | diff --git a/gst/gstpluginloader.c b/gst/gstpluginloader.c |
Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 25 | index d1e404d98..c626bf263 100644 |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 26 | --- a/gst/gstpluginloader.c |
| 27 | +++ b/gst/gstpluginloader.c |
Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 28 | @@ -464,20 +464,19 @@ gst_plugin_loader_spawn (GstPluginLoader * loader) |
| 29 | if (loader->child_running) |
| 30 | return TRUE; |
| 31 | |
| 32 | - /* Find the gst-plugin-scanner: first try the env-var if it is set, |
| 33 | - * otherwise use the installed version */ |
| 34 | + /* Find the gst-plugin-scanner */ |
| 35 | env = g_getenv ("GST_PLUGIN_SCANNER_1_0"); |
| 36 | if (env == NULL) |
| 37 | env = g_getenv ("GST_PLUGIN_SCANNER"); |
| 38 | |
| 39 | if (env != NULL && *env != '\0') { |
| 40 | + /* use the env-var if it is set */ |
| 41 | GST_LOG ("Trying GST_PLUGIN_SCANNER env var: %s", env); |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 42 | helper_bin = g_strdup (env); |
| 43 | res = gst_plugin_loader_try_helper (loader, helper_bin); |
| 44 | g_free (helper_bin); |
| 45 | - } |
| 46 | - |
| 47 | - if (!res) { |
| 48 | + } else { |
Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 49 | + /* use the installed version */ |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 50 | GST_LOG ("Trying installed plugin scanner"); |
| 51 | |
| 52 | #ifdef G_OS_WIN32 |
Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 53 | @@ -497,10 +496,10 @@ gst_plugin_loader_spawn (GstPluginLoader * loader) |
| 54 | #endif |
| 55 | res = gst_plugin_loader_try_helper (loader, helper_bin); |
| 56 | g_free (helper_bin); |
| 57 | + } |
| 58 | |
| 59 | - if (!res) { |
| 60 | - GST_INFO ("No gst-plugin-scanner available, or not working"); |
| 61 | - } |
| 62 | + if (!res) { |
| 63 | + GST_INFO ("No gst-plugin-scanner available, or not working"); |
| 64 | } |
| 65 | |
| 66 | return loader->child_running; |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 67 | -- |
Andrew Geissler | d1e8949 | 2021-02-12 15:35:20 -0600 | [diff] [blame] | 68 | 2.29.2 |
Brad Bishop | 316dfdd | 2018-06-25 12:45:53 -0400 | [diff] [blame] | 69 | |