package/kodi: fix selection of dependencies
authorYann E. MORIN <yann.morin.1998@free.fr>
Sat, 11 Sep 2021 08:20:47 +0000 (10:20 +0200)
committerYann E. MORIN <yann.morin.1998@free.fr>
Fri, 17 Sep 2021 21:27:47 +0000 (23:27 +0200)
commitb80c488d04940cf1280657cf294d58d0022f222f
tree4ba02b3c6b22f8c4781a87dd3f57df67cfff9e61
parent7f303516f69a143c2ecc317948ec1934a195235c
package/kodi: fix selection of dependencies

Commit 148e695e3756 (package/kodi: bump version to 19.0-Matrix) extended
the set of required libraries for various "platform" backends, by
selecting those libraries from the blind options. For example, we have:

    config BR2_PACKAGE_KODI_PLATFORM_SUPPORTS_GBM
        bool
        default y
        depends on [...]
        select BR2_PACKAGE_LIBINPUT
        [...]

However, that option is true as soon as the requirements are met (the
depends on), even when Kodi itself is not enabled.

This means that extra libraries are pulled in to the build, even when
not required.

We fix that by moving the actual selects to the main symbol, along with
the proper conditions. This means that we have two lines that select
libxbcommon, under two different conditions; we could make that a single
select, but the codition would need to be on two lines anyway, so meh...

This is not an ideal solution, because it is a bit ugly, but:
 1) adding three new blind options just for the select is kinda extreme
    and superfluous;
 2) our Kodi packaging is already a bit ugly anyway.

Fixes: #14206
Reported-by: Thomas Ruschival <t.ruschival@gmail.com>
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Cc: Bernd Kuhls <bernd.kuhls@t-online.de>
package/kodi/Config.in