[meta-virtualization] [PATCH] docker: make aufs dependency depend on distro feature

Bruce Ashfield bruce.ashfield at gmail.com
Wed Mar 14 05:40:36 PDT 2018


The problem is .. that this is now a magic distro feature that no one
knows about.

At a minimum, this new distro feature use needs to be documented in the layers
README.

Bruce

On Wed, Mar 14, 2018 at 4:39 AM, Pascal Bach <pascal.bach at siemens.com> wrote:
> Only include aufs-utils if the corresponding distro feature is enabled.
> Without that the aufs kernel driver is not included too.
>
> Signed-off-by: Pascal Bach <pascal.bach at siemens.com>
> ---
>  recipes-containers/docker/docker_git.bb | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-containers/docker/docker_git.bb b/recipes-containers/docker/docker_git.bb
> index 5c21138..206c884 100644
> --- a/recipes-containers/docker/docker_git.bb
> +++ b/recipes-containers/docker/docker_git.bb
> @@ -64,8 +64,9 @@ DEPENDS = " \
>  PACKAGES =+ "${PN}-contrib"
>
>  DEPENDS_append_class-target = " lvm2"
> -RDEPENDS_${PN} = "curl aufs-util git util-linux iptables \
> -                  ${@bb.utils.contains('DISTRO_FEATURES','systemd','','cgroup-lite',d)} \
> +RDEPENDS_${PN} = "util-linux iptables \
> +                  ${@bb.utils.contains('DISTRO_FEATURES', 'aufs', 'aufs-util', '', d)} \
> +                  ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', '', 'cgroup-lite', d)} \
>                   "
>  RDEPENDS_${PN} += "virtual/containerd virtual/runc"
>
> --
> 2.11.0
>
> --
> _______________________________________________
> meta-virtualization mailing list
> meta-virtualization at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-virtualization



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end"


More information about the meta-virtualization mailing list