clapper icon indicating copy to clipboard operation
clapper copied to clipboard

gst-plugin: Provide option to include plugin statically

Open sp1ritCS opened this issue 5 months ago • 2 comments

This changeset adds a new meson option to allow including the clapper gst plugin statically. Additionally, I've dropped the ability of loading importers dynamically altogether, as it was easier this way and I didn't see the point of it.

sp1ritCS avatar Jun 29 '25 18:06 sp1ritCS

Hi. Thanks for PR.

I do agree, these "importer modules" should go away. Even more so now, since it looks like there are even bigger compatibility issues with GStreamer 1.26 (issue: https://github.com/Rafostar/clapper/issues/560).

But due to the above, I am not sure ATM whether to get this change in or just work on the new/next sink sooner and disable this "legacy" code by default (or maybe even drop it).

Rafostar avatar Jul 03 '25 16:07 Rafostar

But due to the above, I am not sure ATM whether to get this change in or just work on the new/next sink sooner and disable this "legacy" code by default (or maybe even drop it).

I don't see why not. I'll assume that you'll have to keep the old sink around, as others (mainly hanabi) are depending on it, but those loader modules are basically internal to clapper in the first place and being able to dynamically load them doesn't really help anybody.

sp1ritCS avatar Jul 03 '25 16:07 sp1ritCS