From mboxrd@z Thu Jan  1 00:00:00 1970
X-GM-THRID: 6588037979633090560
X-Received: by 2002:a19:9643:: with SMTP id y64-v6mr775588lfd.42.1534149255865;
        Mon, 13 Aug 2018 01:34:15 -0700 (PDT)
X-BeenThere: isar-users@googlegroups.com
Received: by 2002:a19:1f07:: with SMTP id f7-v6ls915310lff.8.gmail; Mon, 13
 Aug 2018 01:34:15 -0700 (PDT)
X-Google-Smtp-Source: AA+uWPyHdvhHvcmoLgdtyH1W5AA7dzzYfIDVzyAgia0K02tC7bqbqZXENYxFCI8/t9GXbKgcgB+/
X-Received: by 2002:a19:6313:: with SMTP id x19-v6mr789300lfb.25.1534149255397;
        Mon, 13 Aug 2018 01:34:15 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; t=1534149255; cv=none;
        d=google.com; s=arc-20160816;
        b=wqN8tyRqUQnp2wReh1XZvIOaaT6XSOzaQqiuNexrxFeBDzrKB/EziSFNi4VidFi46q
         yJCfh0rIja4jw1YJNlm3DSr5gREhd2maxwk6wsjAm3g6Lg60MUF7fRZ98I/tuPl2hf/A
         cyXQsRBA3hvMvZGdqFc5oaaLaIoOT434P/Ng7wmA2TmPU9GsFjvpERnhmzeoN4R3rwEC
         gETLF3f98bJb8q64DrNPxrQAqtG1t5OGKPC1R+lIi4wlucA4t9TNQtuxTTCxBgiyroXB
         UVwVOLbQ3sopvekhky7/0K0jgkYP8OdAWWKPfE3VRf0EtBW3M4fWshLZMJceQa46sK0g
         CTXw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816;
        h=content-transfer-encoding:mime-version:references:in-reply-to
         :message-id:subject:cc:to:from:date:arc-authentication-results;
        bh=4rE7r+KdSIO0NJGqCiJE6foZ5MQkDvlGy3o+Xo2iF88=;
        b=yR1PpFbjiUbbGbziDCblNK1wmBfr2/3F1EKEM30TajEH2glpEe3PSryXQratoJDZOX
         58jmwR3/KuYAjQ69e+SkDrq2XXvyciInj3e9m1Obzr/4hI6PWQPIyIAG0xSM7BwwkFvA
         6659/rLLuY5mN1dR00F7r1IB9DgSbbAEO6R25wkxMP9OgAEuwmP9Od7RThjqw00LK51x
         WxgftKnUHj4Op1ZaoO3f/ja/nt4KdmBXlXXx5q8ruIRTxxa+vNT+f2adQ0LPLdL+eOkm
         4k4Ck/r7IQ2Ko/WXPh0WH+d5S9ACMbFmy59OQ3YpROhTtzh9yqXMVDOeY2fcqLP+UaFj
         sTXw==
ARC-Authentication-Results: i=1; gmr-mx.google.com;
       spf=pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.28 as permitted sender) smtp.mailfrom=henning.schild@siemens.com
Return-Path: <henning.schild@siemens.com>
Received: from goliath.siemens.de (goliath.siemens.de. [192.35.17.28])
        by gmr-mx.google.com with ESMTPS id v16-v6si522285ljg.3.2018.08.13.01.34.15
        for <isar-users@googlegroups.com>
        (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);
        Mon, 13 Aug 2018 01:34:15 -0700 (PDT)
Received-SPF: pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.28 as permitted sender) client-ip=192.35.17.28;
Authentication-Results: gmr-mx.google.com;
       spf=pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.28 as permitted sender) smtp.mailfrom=henning.schild@siemens.com
Received: from mail2.sbs.de (mail2.sbs.de [192.129.41.66])
	by goliath.siemens.de (8.15.2/8.15.2) with ESMTPS id w7D8YEO0028736
	(version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK);
	Mon, 13 Aug 2018 10:34:14 +0200
Received: from md1pvb1c.ad001.siemens.net (md1pvb1c.ad001.siemens.net [139.25.68.40])
	by mail2.sbs.de (8.15.2/8.15.2) with ESMTP id w7D8YEX5007877;
	Mon, 13 Aug 2018 10:34:14 +0200
Date: Mon, 13 Aug 2018 10:34:13 +0200
From: Henning Schild <henning.schild@siemens.com>
To: "Maxim Yu. Osipov" <mosipov@ilbers.de>
Cc: Jan Kiszka <jan.kiszka@siemens.com>, <isar-users@googlegroups.com>
Subject: Re: [PATCH 2/2] isar-bootstrap: exit if command fails under flock
Message-ID: <20180813103413.7cb514f5@md1pvb1c.ad001.siemens.net>
In-Reply-To: <239acee3-b103-3cdf-8edc-f7810ec5b8a0@ilbers.de>
References: <20180810103150.28342-1-mosipov@ilbers.de>
	<20180810103150.28342-3-mosipov@ilbers.de>
	<ba2a806c-525d-9ead-bb38-88243adfd5f4@siemens.com>
	<239acee3-b103-3cdf-8edc-f7810ec5b8a0@ilbers.de>
X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.32; x86_64-pc-linux-gnu)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-TUID: R6i+aGXAlZRU

Am Sun, 12 Aug 2018 11:46:30 +0300
schrieb "Maxim Yu. Osipov" <mosipov@ilbers.de>:

> Hi Jan,
>=20
> On 08/11/2018 06:52 PM, Jan Kiszka wrote:
> > On 2018-08-10 12:31, Maxim Yu. Osipov wrote: =20
> >> If command under 'flock' fails, isar_bootstrap continues to
> >> execute. This is wrong behaviour, which leads that the
> >> do_bootstrap task is considered successfully done.
> >>
> >> Signed-off-by: Maxim Yu. Osipov <mosipov@ilbers.de>
> >> ---
> >> =C2=A0 .../recipes-core/isar-bootstrap/isar-bootstrap.inc | 28=20
> >> +++++++++++-----------
> >> =C2=A0 1 file changed, 14 insertions(+), 14 deletions(-)
> >>
> >> diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc=20
> >> b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> >> index 5c90713..60e5d10 100644
> >> --- a/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> >> +++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap.inc
> >> @@ -187,7 +187,7 @@ isar_bootstrap() {
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ${DEBOOTSTRAP_KEYRING} \
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 "${@get_distro_suite(d,=20
> >> True)}" \
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 "${ROOTFSDIR}" \
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0 "${@get_distro_source(d,=20
> >> True)}"
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0 "${@get_distro_source(d,=20
> >> True)}" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 else
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 sudo -E "${DEBOOTSTRAP}" --verbose \
> >> @@ -198,45 +198,45 @@ isar_bootstrap() {
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 ${DEBOOTSTRAP_KEYRING}
> >> \ "${@get_distro_suite(d,=20
> >> False)}" \
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 "${ROOTFSDIR}" \
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0 "${@get_distro_source(d,=20
> >> False)}"
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0 "${@get_distro_source(d,=20
> >> False)}" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 fi
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 # Install apt config
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo mkdir -p "${ROOTFSDIR}/etc/apt/preferences.d"
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo install -v -m644 "${APTPREFS}" \
> >> -                                 =20
> >> "${ROOTFSDIR}/etc/apt/preferences.d/bootstrap"
> >> +                                 =20
> >> "${ROOTFSDIR}/etc/apt/preferences.d/bootstrap" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo mkdir -p "${ROOTFSDIR}/etc/apt/sources.list.d"
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo install -v -m644 "${APTSRCS}" \
> >> -                                 =20
> >> "${ROOTFSDIR}/etc/apt/sources.list.d/bootstrap.list"
> >> +                                 =20
> >> "${ROOTFSDIR}/etc/apt/sources.list.d/bootstrap.list" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo rm -f "${ROOTFSDIR}/etc/apt/sources.list"
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo mkdir -p "${ROOTFSDIR}/etc/apt/apt.conf.d"
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo install -v -m644 "${WORKDIR}/isar-apt.conf" \
> >> -                                 =20
> >> "${ROOTFSDIR}/etc/apt/apt.conf.d/50isar.conf"
> >> +                                 =20
> >> "${ROOTFSDIR}/etc/apt/apt.conf.d/50isar.conf" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 if [ "${@get_distro_suite(d, True)}" =3D "stretch" ]
> >> && [ "${@get_host_release().split('.')[0]}" -lt "4" ]; then
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 sudo install -v -m644=20
> >> "${WORKDIR}/isar-apt-fallback.conf" \
> >> -                                     =20
> >> "${ROOTFSDIR}/etc/apt/apt.conf.d/55isar-fallback.conf"
> >> +                                     =20
> >> "${ROOTFSDIR}/etc/apt/apt.conf.d/55isar-fallback.conf" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 fi
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 # Set locale
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do install -v -m644 "${WORKDIR}/locale"=20
> >> "${ROOTFSDIR}/etc/locale"
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do install -v -m644 "${WORKDIR}/locale"=20
> >> "${ROOTFSDIR}/etc/locale" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo sed -i '/en_US.UTF-8 UTF-8/s/^#//g'=20
> >> "${ROOTFSDIR}/etc/locale.gen"
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do -E chroot "${ROOTFSDIR}" /usr/sbin/locale-gen
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do -E chroot "${ROOTFSDIR}" /usr/sbin/locale-gen ||
> >> exit 1 # setup chroot
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do install -v -m755 "${WORKDIR}/chroot-setup.sh"=20
> >> "${ROOTFSDIR}/chroot-setup.sh"
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do "${ROOTFSDIR}/chroot-setup.sh" "setup"
> >> "${ROOTFSDIR}"
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do install -v -m755 "${WORKDIR}/chroot-setup.sh"=20
> >> "${ROOTFSDIR}/chroot-setup.sh" || exit 1
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do "${ROOTFSDIR}/chroot-setup.sh" "setup"=20
> >> "${ROOTFSDIR}" || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 # update APT
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do mount -t devtmpfs -o mode=3D0755,nosuid devtmpfs=20
> >> ${ROOTFSDIR}/dev
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do mount -t proc none ${ROOTFSDIR}/proc
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do mount -t devtmpfs -o mode=3D0755,nosuid devtmpfs=20
> >> ${ROOTFSDIR}/dev || exit 1
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do mount -t proc none ${ROOTFSDIR}/proc || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 E=3D"${@bb.utils.export_proxies(d)}"
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 export DEBIAN_FRONTEND=3Dnoninteractive
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do -E chroot "${ROOTFSDIR}" /usr/bin/apt-get update
> >> -y
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 su=
do -E chroot "${ROOTFSDIR}" /usr/bin/apt-get update
> >> -y || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 sudo -E chroot "${ROOTFSDIR}" /usr/bin/apt-get=20
> >> dist-upgrade -y \
> >> -=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0 -o
> >> Debug::pkgProblemResolver=3Dyes
> >> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0 -o=20
> >> Debug::pkgProblemResolver=3Dyes || exit 1
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 # Finalize debootstrap by setting the link in deploy
> >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0 ln -Tfsr "${ROOTFSDIR}" "${DEPLOY_ISAR_BOOTSTRAP}"
> >> =20
> >=20
> > This is a bit ugly and leaves out a number of commands. Why can't
> > we do "set -e" inside that section? =20
>=20
> Actually, 'set -e' is put automatically in corresponding=20
> run.do_bootstrap script but it doesn't work for flock case.

The problem is that all these steps are "the command" for flock. That
is pretty ugly to begin with. If one needs to do that, all those
commands should be chained with &&.
What would be nicer would be a bitbake lock for isar_bootstrap or the
two "do_bootstrap" calls and get rid of the flock.

Your patch fails to deal with all the errors the other commands could
trigger. And i wonder why we need sudo inside "sudo flock". (commit
ab0a1c8c)

Henning

> Maxim.
>=20