This still needs to be enabled by default in the packageconfig, unless you can show that the existing builds were not detecting libselinux and using it.
Bruce On Fri, Feb 17, 2023 at 12:32 AM Chen Qi <[email protected]> wrote: > > For cri-o, libselinux is optional, this can be seen from > its Makefile. So let's make selinux optional by using PACKAGECONFIG. > In this way, meta-selinux dependency could be removed. > > Signed-off-by: Chen Qi <[email protected]> > --- > recipes-containers/cri-o/cri-o_git.bb | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/recipes-containers/cri-o/cri-o_git.bb > b/recipes-containers/cri-o/cri-o_git.bb > index 66d1116..7af698f 100644 > --- a/recipes-containers/cri-o/cri-o_git.bb > +++ b/recipes-containers/cri-o/cri-o_git.bb > @@ -39,14 +39,14 @@ DEPENDS = " \ > ostree \ > libdevmapper \ > libseccomp \ > - libselinux \ > " > RDEPENDS:${PN} = " \ > cni \ > libdevmapper \ > " > > -SKIP_RECIPE[cri-o] ?= "${@bb.utils.contains('BBFILE_COLLECTIONS', 'selinux', > '', 'Depends on libselinux from meta-selinux which is not included', d)}" > +PACKAGECONFIG ?= "" > +PACKAGECONFIG[selinux] = ",,libselinux" > > PACKAGES =+ "${PN}-config" > > -- > 2.37.1 > > > > -- - Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end - "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#7866): https://lists.yoctoproject.org/g/meta-virtualization/message/7866 Mute This Topic: https://lists.yoctoproject.org/mt/97023221/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/leave/6693005/21656/1014668956/xyzzy [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
