Re: [PATCH] bootstrap.inc install sources-list then wrongly remove it, bugfix

[Jan Kiszka <jan.kiszka@siemens.com>]

On 10.01.23 12:50, roberto.foglietta@linuxteam.org wrote:
> From: "Roberto A. Foglietta" <roberto.foglietta@gmail.com>
> 
> bootstrap.inc, bugfix: two lines were in reverse order between them
> 
> Signed-off-by: Roberto A. Foglietta <roberto.foglietta@gmail.com>
> ---
>  meta/recipes-core/isar-bootstrap/isar-bootstrap.inc | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> index faba73f..3aa11a4 100644
> --- a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> +++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> @@ -296,6 +296,7 @@ do_bootstrap() {
>          if [ "${BOOTSTRAP_FOR_HOST}" = "0" ]; then
>              arch_param="--arch=${DISTRO_ARCH}"
>          fi
> +
>          ${DEBOOTSTRAP} $debootstrap_args \
>                         $arch_param \
>                         ${@get_distro_components_argument(d)} \
> @@ -327,9 +328,9 @@ do_bootstrap() {
>              install -v -m644 "${APTSRCS}" \
>                               "${ROOTFSDIR}/etc/apt/sources.list.d/bootstrap.list"
>          fi
> -        install -v -m644 "${APTSRCS_INIT}" "${ROOTFSDIR}/etc/apt/sources-list"
>          rm -f "${ROOTFSDIR}/etc/apt/sources.list"

sources-list != sources.list, if that was your concern. So the order is
not critical here. If it were, things should have exploded already in
all colors.

Jan

> -        rm -rf "${ROOTFSDIR}/var/lib/apt/lists/"*
> +        install -v -m644 "${APTSRCS_INIT}" "${ROOTFSDIR}/etc/apt/sources-list"
> +
>          find ${APT_KEYS_DIR}/ -type f | while read keyfile
>          do
>              MY_GPGHOME="$(chroot "${ROOTFSDIR}" mktemp -d /tmp/gpghomeXXXXXXXXXX)"

-- 
Siemens AG, Technology
Competence Center Embedded Linux