ladspa: Make RDF parsing truely optional

If the ladspa plugin is enabled explicitly or via auto-features, the
liblrdf dependency can not be disabled.

As the RDF parsing currently provides hardly any features, the possibility
to disable it fairly useful.

Fixes: #3168
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/5802>
This commit is contained in:
Stefan Brüns 2023-12-10 04:26:59 +01:00 committed by Tim-Philipp Müller
parent 51939b62b8
commit 2aefdac995
2 changed files with 6 additions and 6 deletions

View file

@ -12,18 +12,17 @@ if get_option('ladspa').disabled()
subdir_done()
endif
# This is an optional dep, but we make it optional only in auto mode
lrdf_dep = dependency('lrdf', required : get_option('ladspa'))
if lrdf_dep.found()
ladspa_cargs = ['-DHAVE_LRDF']
endif
have_ladspa_h = cc.has_header('ladspa.h')
if not have_ladspa_h and get_option('ladspa').enabled()
error('ladspa plugin is enabled but ladspa.h not found')
endif
if have_ladspa_h
lrdf_dep = dependency('lrdf', required : get_option('ladspa-rdf'))
if lrdf_dep.found()
ladspa_cargs = ['-DHAVE_LRDF']
endif
gstladspa = library('gstladspa',
ladspa_sources,
c_args : gst_plugins_bad_args + ladspa_cargs,

View file

@ -121,6 +121,7 @@ option('iqa', type : 'feature', value : 'auto', description : 'Image quality ass
option('kate', type : 'feature', value : 'auto', description : 'Kate subtitle parser, tagger, and codec plugin')
option('kms', type : 'feature', value : 'auto', description : 'KMS video sink plugin')
option('ladspa', type : 'feature', value : 'auto', description : 'LADSPA plugin bridge')
option('ladspa-rdf', type : 'feature', value : 'auto', description : 'LADSPA plugin bridge RDF support')
option('ldac', type : 'feature', value : 'auto', description : 'LDAC bluetooth audio codec plugin')
option('libde265', type : 'feature', value : 'auto', description : 'HEVC/H.265 video decoder plugin')
option('openaptx', type : 'feature', value : 'auto', description : 'Open Source implementation of Audio Processing Technology codec (aptX) plugin')