From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6771003123014041600 Date: Tue, 28 Jan 2020 11:45:18 -0800 (PST) From: vijai kumar To: isar-users Message-Id: <4f458566-c16e-4af0-9b08-174188b21ef8@googlegroups.com> In-Reply-To: <20200128175235.5645b10d@md1za8fc.ad001.siemens.net> References: <20200128154312.337e61f6@md1za8fc.ad001.siemens.net> <20200128154213.22353-1-vijaikumar.kanagarajan@gmail.com> <20200128154213.22353-27-vijaikumar.kanagarajan@gmail.com> <20200128175235.5645b10d@md1za8fc.ad001.siemens.net> Subject: Re: [PATCH v3 26/26] meta/recipes-kernel: Export KBUILD_DEPENDS to cache MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_509_356252382.1580240718809" X-Google-Token: EM6ewvEFRlOxWqfkWFU0 X-Google-IP: 192.94.34.34 X-TUID: bsq4ZA2O7Q7v ------=_Part_509_356252382.1580240718809 Content-Type: multipart/alternative; boundary="----=_Part_510_1341970694.1580240718810" ------=_Part_510_1341970694.1580240718810 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On Tuesday, January 28, 2020 at 10:22:37 PM UTC+5:30, Henning Schild wrote: > > Thanks. > > On Tue, 28 Jan 2020 21:12:13 +0530 > Vijai Kumar K > wrote: > > > From: Vijai Kumar K > > > > > linux-custom overrides do_dpkg_builddeps with its own variant. > > Since deb-dl-dir* is not called KBUILD_DEPENDS is not cached. > > > > Call deb-dl-dir-import/deb-dl-dir-export to cache it. > > > > Signed-off-by: Vijai Kumar K > > > --- > > meta/recipes-kernel/linux/linux-custom.inc | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/meta/recipes-kernel/linux/linux-custom.inc > > b/meta/recipes-kernel/linux/linux-custom.inc index 17f8464..37eb3c2 > > 100644 --- a/meta/recipes-kernel/linux/linux-custom.inc > > +++ b/meta/recipes-kernel/linux/linux-custom.inc > > @@ -41,6 +41,7 @@ KERNEL_LIBC_DEV_DEPLOY ?= "0" > > do_install_builddeps() { > > dpkg_do_mounts > > E="${@ isar_export_proxies(d)}" > > + deb_dl_dir_import "${BUILDCHROOT_DIR}" > > sudo -E chroot ${BUILDCHROOT_DIR} \ > > apt-get update \ > > -o > > Dir::Etc::SourceList="sources.list.d/isar-apt.list" \ @@ -50,6 +51,7 > > @@ do_install_builddeps() { apt-get install \ > > -y -o Debug::pkgProblemResolver=yes \ > > --no-install-recommends ${KBUILD_DEPENDS} > > In between here you will have to do the split from p8. Otherwise if you > have post-install hooks to clear the caches the next statement will > copy nothing. > > Ah, I see. Will change that. > I guess it is ok to send just one patch and note that it is supposed to > sit on top of my v2. Sure. Thanks, Vijai Kumar K > > > Henning > > > + deb_dl_dir_export "${BUILDCHROOT_DIR}" > > dpkg_undo_mounts > > } > > > > ------=_Part_510_1341970694.1580240718810 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: quoted-printable


On Tuesday, January 28, 2020 at 10:22:37 PM UTC+5:= 30, Henning Schild wrote:
Thank= s.

On Tue, 28 Jan 2020 21:12:13 +0530
Vijai Kumar K <vijaikumar...@gmail.com> wrote:

> From: Vijai Kumar K <Vijaikumar_...@mentor.com>
>=20
> linux-custom overrides do_dpkg_builddeps with its own variant.
> Since deb-dl-dir* is not called KBUILD_DEPENDS is not cached.
>=20
> Call deb-dl-dir-import/deb-dl-dir-export to cache it.
>=20
> Signed-off-by: Vijai Kumar K <Vijaikumar_...@mentor.com&g= t;
> ---
> =C2=A0meta/recipes-kernel/linux/linux-custom.inc | 2 ++
> =C2=A01 file changed, 2 insertions(+)
>=20
> diff --git a/meta/recipes-kernel/linux/linux-custom.inc
> b/meta/recipes-kernel/linux/linux-custom.inc index 17f8464..3= 7eb3c2
> 100644 --- a/meta/recipes-kernel/linux/linux-custom.inc
> +++ b/meta/recipes-kernel/linux/linux-custom.inc
> @@ -41,6 +41,7 @@ KERNEL_LIBC_DEV_DEPLOY ?=3D "0"
> =C2=A0do_install_builddeps() {
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0dpkg_do_moun= ts
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0E=3D"${= @ isar_export_proxies(d)}"
> + =C2=A0 =C2=A0 =C2=A0 =C2=A0deb_dl_dir_import "${BUILDCHROOT= _DIR}"
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0sudo -E chro= ot ${BUILDCHROOT_DIR} \
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0apt-get update \
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=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
> Dir::Etc::SourceList=3D"sources.list.d/isar-apt.list&quo= t; \ @@ -50,6 +51,7
> @@ do_install_builddeps() { apt-get install \
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0-y -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--no-install-recommends ${KBUILD_DEPENDS}

In between here you will have to do the split from p8. Otherwise if you
have post-install hooks to clear the caches the next statement will
copy nothing.


Ah, I see. Will change that.
=C2=A0
I guess it is ok = to send just one patch and note that it is supposed to
sit on top of my v2.

Sure.
Thanks,
Vijai Kumar K
=C2=A0


Henning

> + =C2=A0 =C2=A0 =C2=A0 =C2=A0deb_dl_dir_export "${BUILDCHROOT= _DIR}"
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0dpkg_undo_mo= unts
> =C2=A0}
> =C2=A0

------=_Part_510_1341970694.1580240718810-- ------=_Part_509_356252382.1580240718809--