Hello everyone, With v4 we have moved from pre-processor to compilation checking for the external plugins support. Namely, as we build without one the dead-code elimination will discard all the relevant code. Ultimately this means we compile check both paths in order to catch mistakes. Thanks to Luiz for the suggestion. Link to the previous revision can be found below. Thanks Emil - Link to v3: https://lore.kernel.org/r/20240125-rm-ext-plugins-v3-0-d141f7870bb6@xxxxxxxxx --- Emil Velikov (8): configure, README: introduce --enable-external-plugins obexd: factor out external plugin support bluetoothd: remove external-dummy plugin bluetoothd: convert external sixaxis plugin to builtin bluetoothd: factor out external plugin support bluetoothd: don't export internal API bluetoothd: change plugin loading alike obexd android: export only (android) entrypoint from the modules Makefile.am | 17 ++---- Makefile.obexd | 2 + Makefile.plugins | 8 ++- README | 13 +++++ android/Makefile.am | 3 ++ android/hal-audio.c | 1 + android/hal-bluetooth.c | 1 + android/hal-sco.c | 1 + configure.ac | 10 ++++ obexd/src/obexd.h | 2 +- obexd/src/plugin.c | 89 ++++++++++++++++++++----------- obexd/src/plugin.h | 4 ++ plugins/external-dummy.c | 28 ---------- src/btd.h | 2 +- src/plugin.c | 135 ++++++++++++++++++++++++++++------------------- src/plugin.h | 4 ++ 16 files changed, 188 insertions(+), 132 deletions(-) --- base-commit: 0de32f67f685b95c35a5c2f1206081af89bd88b6 change-id: 20240116-rm-ext-plugins-ba0b852a492b Best regards, -- Emil Velikov <emil.l.velikov@xxxxxxxxx>
