Re: [PATCH] isar-bootstrap: Clear local apt state information

[Henning Schild <henning.schild@siemens.com>]

Hi,

i am not sure i fully get the problem. With the first stage you
probably mean "debootstrap" and the second stage the isar bootstrap on
top.

The first stage should be using the primary entry from the
sources.list, so as far as i remember we are just adding more entries.
Meaning the state information should stay valid. Maybe in the offline
"base-apt" case we have a flow that actually switches the upstream
mirror.

Maybe you can go into detail how to reproduce the issue at hand,
possibly add a test-case for it.

If the files should be deleted, my guess is that the deletion is
related to the deletion of sources.list in line 282. So it might be a
good idea to add your deletion right after or before that.

Henning

Am Wed, 20 Nov 2019 16:23:40 +0530
schrieb <vijaikumar.kanagarajan@gmail.com>:

> From: Vijai Kumar K <Vijaikumar_Kanagarajan@mentor.com>
> 
> Debootstrap's second stage switches to using the default debian
> mirror. But the apt state information is renamed and used from the
> first stage. This might cause apt-get update to fail with the below
> error when the sources list contain both custom and upstream repos.
> 
> | Reading package lists...
> | E: Failed to fetch
> http://deb.debian.org/debian/dists/stretch/InRelease \ |
> Unable to find expected entry 'contrib/source/Sources' in Release
> file \ |		(Wrong sources.list entry or malformed file)
> 
> Clear the apt state information before apt-get update.
> 
> Signed-off-by: Vijai Kumar K <Vijaikumar_Kanagarajan@mentor.com>
> ---
>  meta/recipes-core/isar-bootstrap/isar-bootstrap.inc | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc index
> d614d9a..5a379f7 100644 ---
> a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc +++
> b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc @@ -320,6
> +320,7 @@ isar_bootstrap() { chroot "${ROOTFSDIR}" /usr/bin/dpkg
> --add-architecture ${DISTRO_ARCH} fi
>  
> +            rm -rf "${ROOTFSDIR}/var/lib/apt/lists/"*
>              chroot "${ROOTFSDIR}" /usr/bin/apt-get update -y
>              chroot "${ROOTFSDIR}" /usr/bin/apt-get install -y -f
>              chroot "${ROOTFSDIR}" /usr/bin/apt-get dist-upgrade -y \