[totem/wip/hadess/bug-fixes] build: Fix build with missing plugins support disabled



commit fffc1fb0247f548a61c63f4374141dce10af628e
Author: Bastien Nocera <hadess hadess net>
Date:   Thu Feb 7 22:03:21 2019 +0100

    build: Fix build with missing plugins support disabled
    
    Some code was using gstreamer-pbutils-1.0 provided functions without
    declaring them as a dep in code that wasn't guarded by
    ENABLE_MISSING_PLUGIN_INSTALLATION.
    
    Also fix have_easy_codec declaration in the same circumstances.

 meson.build             | 9 ++++-----
 src/backend/meson.build | 1 +
 2 files changed, 5 insertions(+), 5 deletions(-)
---
diff --git a/meson.build b/meson.build
index 9db961cac..98f22831b 100644
--- a/meson.build
+++ b/meson.build
@@ -128,6 +128,7 @@ gtk_dep = dependency('gtk+-3.0', version: gtk_req_version)
 gst_dep = dependency('gstreamer-1.0', version: gst_req_version)
 gst_tag_dep = dependency('gstreamer-tag-1.0', version: '>= 0.11.93')
 gst_video_dep = dependency('gstreamer-video-1.0')
+gst_pbutils_dep = dependency('gstreamer-pbutils-1.0')
 peas_dep = dependency('libpeas-1.0', version: peas_req_version)
 peas_gtk_dep = dependency('libpeas-gtk-1.0', version: peas_req_version)
 totem_plparser_dep = dependency('totem-plparser', version: totem_plparser_req_version)
@@ -162,19 +163,17 @@ endif
 missing_plugins_deps = []
 
 easy_codec_option = get_option('enable-easy-codec-installation')
+have_easy_codec = false
 if easy_codec_option != 'no'
-  have_easy_codec = false
-
-  gst_pbutils_dep = dependency('gstreamer-pbutils-1.0', required: (easy_codec_option == 'yes'))
   if gst_pbutils_dep.found()
     have_easy_codec = true
     gio_unix_dep = dependency('gio-unix-2.0')
   endif
 
   missing_plugins_deps += [ gst_pbutils_dep, gio_unix_dep ]
-  config_h.set('ENABLE_MISSING_PLUGIN_INSTALLATION', have_easy_codec,
-               description: 'Whether we can and want to do installation of missing plugins')
 endif
+config_h.set('ENABLE_MISSING_PLUGIN_INSTALLATION', have_easy_codec,
+             description: 'Whether we can and want to do installation of missing plugins')
 
 # python support
 have_python = false
diff --git a/src/backend/meson.build b/src/backend/meson.build
index ef8da1a7a..7a63bab1b 100644
--- a/src/backend/meson.build
+++ b/src/backend/meson.build
@@ -61,6 +61,7 @@ libbacon_video_widget_deps = missing_plugins_deps + [
   gst_dep,
   dependency('gstreamer-base-1.0', version: gst_req_version),
   dependency('gstreamer-plugins-base-1.0', version: gst_req_version),
+  gst_pbutils_dep,
   gst_tag_dep,
   dependency('gstreamer-audio-1.0'),
   gst_video_dep,


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]