Looks good to me. I would just change the subject to 'isar-image-base: 
remove "apt" from IMAGE_PREINSTALL'
(I found "listing" to be confusing/misleading until I looked at the patch)

Cedric

On Wednesday, August 22, 2018 at 9:55:45 PM UTC+2, Jan Kiszka wrote:
>
> debootstrap does that for us anyway. 
>
> Signed-off-by: Jan Kiszka <jan.k...@siemens.com <javascript:>> 
> --- 
>  meta-isar/recipes-core/images/isar-image-base.bb | 3 +-- 
>  1 file changed, 1 insertion(+), 2 deletions(-) 
>
> diff --git a/meta-isar/recipes-core/images/isar-image-base.bb 
> b/meta-isar/recipes-core/images/isar-image-base.bb 
> index a24aa3e..84001fd 100644 
> --- a/meta-isar/recipes-core/images/isar-image-base.bb 
> +++ b/meta-isar/recipes-core/images/isar-image-base.bb 
> @@ -18,8 +18,7 @@ inherit isar-bootstrap-helper 
>   
>  DEPENDS += "${IMAGE_INSTALL} ${IMAGE_TRANSIENT_PACKAGES}" 
>   
> -IMAGE_PREINSTALL += "apt \ 
> -                     dbus" 
> +IMAGE_PREINSTALL += "dbus" 
>  IMAGE_TRANSIENT_PACKAGES += "isar-cfg-localepurge" 
>   
>  WORKDIR = "${TMPDIR}/work/${DISTRO}-${DISTRO_ARCH}/${PN}" 
> -- 
> 2.16.4 
>