From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 7312758318482587648 Date: Fri, 15 Dec 2023 03:01:11 -0800 (PST) From: Srinuvasan Arjunan To: isar-users Message-Id: In-Reply-To: <8b19681f-8237-4cef-9235-b6128a37f457@siemens.com> References: <20231215095639.566978-1-srinuvasan_a@mentor.com> <8b19681f-8237-4cef-9235-b6128a37f457@siemens.com> Subject: Re: [PATCH] recipes-devtools/sbuild-chroot: install self-built packages in sbuild flavor MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_36482_1761298502.1702638071033" X-TUID: prokbSM7PEh3 ------=_Part_36482_1761298502.1702638071033 Content-Type: multipart/alternative; boundary="----=_Part_36483_379965835.1702638071033" ------=_Part_36483_379965835.1702638071033 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Friday, December 15, 2023 at 4:04:49=E2=80=AFPM UTC+5:30 Jan Kiszka wrot= e: On 15.12.23 10:56, Srinuvasan Arjunan wrote:=20 > From: srinuvasan =20 >=20 > In some use-cases we need to install custom packages in sbuildchroot and= =20 > this can be referred later time during image generation part, hence we=20 > need the persistent sbuildchroot rather than the temporary sessions.=20 We already have a persistent sbuild flavor, but we may lack some=20 extension to make it usable not only from a package recipe but also an=20 image recipe. Please work towards that direction. yes my comment may be wrong, but the intention of the work towards=20 right directions, this will extend to install the custom packages in persistent sbuildchroot.=20 >=20 > Introduced persistent-sbuildchroot class to handle the above scenarios.= =20 >=20 > Introduced SBUILD_CHROOT_INSTALL_EXTRA variable, here you can define the= =20 > additional custom packages that you need to install in sbuild flavor.=20 I don't get the semantic of this variable. We so far have=20 SBUILD_CHROOT_PREINSTALL, why don't you introduce SBUILD_CHROOT_INSTALL=20 (SBUILD_CHROOT_PREINSTALL + DEPENDS), analogously to IMAGE_[PRE]INSTALL? This will address.=20 Jan=20 --=20 Siemens AG, Technology=20 Linux Expert Center=20 ------=_Part_36483_379965835.1702638071033 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: quoted-printable

On Friday, December 15, 2023 at 4:04:49= =E2=80=AFPM UTC+5:30 Jan Kiszka wrote:
On 15.12.23 10:56, Srinuvasan Arjunan wrote:
> From: srinuvasan <srinuv...@sie= mens.com>
>=20
> In some use-cases we need to install custom packages in sbuildch= root and
> this can be referred later time during image generation part, he= nce we
> need the persistent sbuildchroot rather than the temporary sessi= ons.

We already have a persistent sbuild flavor, but we may lack some
extension to make it usable not only from a package recipe but also a= n
image recipe. Please work towards that direction.
=C2=A0 =C2=A0 yes my comment may be wrong, but the intention= of the work towards right directions, this will extend to install the cust= om packages
=C2=A0 =C2=A0 in persistent sbuildchroot.=C2=A0
=


>=20
> Introduced persistent-sbuildchroot class to handle the above sce= narios.
>=20
> Introduced SBUILD_CHROOT_INSTALL_EXTRA variable, here you can de= fine the
> additional custom packages that you need to install in sbuild fl= avor.

I don't get the semantic of this variable. We so far have
SBUILD_CHROOT_PREINSTALL, why don't you introduce SBUILD_CHROOT_INSTA= LL
(SBUILD_CHROOT_PREINSTALL + DEPENDS), analogously to IMAGE_[PRE]INSTA= LL?

=C2=A0 =C2=A0This will address.=C2=A0=


Jan

--=20
Siemens AG, Technology
Linux Expert Center

------=_Part_36483_379965835.1702638071033-- ------=_Part_36482_1761298502.1702638071033--