meson: fix ladspa dependencies

There are two issues, both related to dependency checking with the meson
support for the ladspa plugin.

With autotools, lrdf is handled like an optional dependency. But with
meson it is required. This makes the meson support less flexible and
inconsistent with autotools.

When autotools is used it properly checks if ladspa.h is available.
But with meson it does not, instead it treats lrdf as the main
dependency. This could cause a build failure if lrdf is installed, but
the ladspa sdk is not.

https://bugzilla.gnome.org/show_bug.cgi?id=794350
This commit is contained in:
Patrik Nilsson 2018-03-05 17:43:26 +01:00 committed by Sebastian Dröge
parent 3fdcdf2705
commit e61e840d09

View file

@ -6,14 +6,20 @@ ladspa_sources = [
'gstladspautils.c',
]
ladspa_dep = dependency('lrdf', required : false)
if ladspa_dep.found()
gstrsvg = library('gstladspa',
ladspa_cargs = []
lrdf_dep = dependency('lrdf', required : false)
if lrdf_dep.found()
ladspa_cargs = ['-DHAVE_LRDF']
endif
if cc.has_header('ladspa.h')
gstladspa = library('gstladspa',
ladspa_sources,
c_args : gst_plugins_bad_args,
c_args : gst_plugins_bad_args + ladspa_cargs,
link_args : noseh_link_args,
include_directories : [configinc, libsinc],
dependencies : [gstaudio_dep, gstbase_dep, gmodule_dep, ladspa_dep, mathlib],
dependencies : [gstaudio_dep, gstbase_dep, gmodule_dep, mathlib],
install : true,
install_dir : plugins_install_dir,
)