[Bug 795107] Meson: missing option to enable/disable plugins

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=795107

--- Comment #12 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 370762
  --> https://bugzilla.gnome.org/attachment.cgi?id=370762
Add option to enable/disable each plugin in good

Thanks for working on this!

Just two comments from a user-point-of-view:

>+if not get_option('audioparsers')
>+  subdir_done()
>+endif

I think this we could probably do nicer if we added something to subdir() ?

But what I think will be confusing is that users now have to know what type a
plugin is to enable it or disable it, if I understand correctly. Because the
dependencyless options are now booleans so you'd have to do

  meson -Daudioparsers=false

but for the ones with dependencies you'd have to do

  meson -Dpng=required (or whatever name we settle on later)

That seems unfortunate, but easy to fix by just using tristate options there
too I guess. Which would still make sense because it will allow for the
"disable everything auto by default, and let me opt-in by enabling things
explicitly", for example. Which won't be supported with boolean options.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.