* [PATCH] u-boot: set SYSROOT_DIRS to /boot and ${sysconfdir}
@ 2020-12-17 11:23 Diego Sueiro
2020-12-18 9:51 ` [OE-core] " Max Krummenacher
0 siblings, 1 reply; 3+ messages in thread
From: Diego Sueiro @ 2020-12-17 11:23 UTC (permalink / raw)
To: openembedded-core; +Cc: nd, Diego Sueiro
Since do_install is shipping files to /boot and ${sysconfdir}, we can set
SYSROOT_DIRS to these dirs and allow recipes which depend on u-boot to use its
installed artifacts from RECIPE_SYSROOT instead of DEPLOY_DIR_IMAGE.
Signed-off-by: Diego Sueiro <diego.sueiro@arm.com>
---
meta/recipes-bsp/u-boot/u-boot.inc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc
index 251178db33..6af0766571 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -266,6 +266,8 @@ FILES_${PN}-env = " \
"
FILES_${PN} = "/boot ${datadir}"
+SYSROOT_DIRS = "/boot ${sysconfdir}"
+
RDEPENDS_${PN} += "${PN}-env"
do_deploy () {
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [OE-core] [PATCH] u-boot: set SYSROOT_DIRS to /boot and ${sysconfdir}
2020-12-17 11:23 [PATCH] u-boot: set SYSROOT_DIRS to /boot and ${sysconfdir} Diego Sueiro
@ 2020-12-18 9:51 ` Max Krummenacher
2021-01-04 8:29 ` Diego Sueiro
0 siblings, 1 reply; 3+ messages in thread
From: Max Krummenacher @ 2020-12-18 9:51 UTC (permalink / raw)
To: Diego Sueiro, openembedded-core; +Cc: nd
Hi
Am Donnerstag, den 17.12.2020, 11:23 +0000 schrieb Diego Sueiro:
> Since do_install is shipping files to /boot and ${sysconfdir}, we can set
> SYSROOT_DIRS to these dirs and allow recipes which depend on u-boot to use its
> installed artifacts from RECIPE_SYSROOT instead of DEPLOY_DIR_IMAGE.
>
> Signed-off-by: Diego Sueiro <diego.sueiro@arm.com>
> ---
> meta/recipes-bsp/u-boot/u-boot.inc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc
> index 251178db33..6af0766571 100644
> --- a/meta/recipes-bsp/u-boot/u-boot.inc
> +++ b/meta/recipes-bsp/u-boot/u-boot.inc
> @@ -266,6 +266,8 @@ FILES_${PN}-env = " \
> "
>
> FILES_${PN} = "/boot ${datadir}"
> +SYSROOT_DIRS = "/boot ${sysconfdir}"
Wouldn't a += be more future proof. I.e. not dropping the directories which are staged by default.
Max
> +
> RDEPENDS_${PN} += "${PN}-env"
>
> do_deploy () {
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [OE-core] [PATCH] u-boot: set SYSROOT_DIRS to /boot and ${sysconfdir}
2020-12-18 9:51 ` [OE-core] " Max Krummenacher
@ 2021-01-04 8:29 ` Diego Sueiro
0 siblings, 0 replies; 3+ messages in thread
From: Diego Sueiro @ 2021-01-04 8:29 UTC (permalink / raw)
To: max.oss.09@gmail.com, openembedded-core@lists.openembedded.org; +Cc: nd
Hi,
>-----Original Message-----
>From: openembedded-core@lists.openembedded.org <openembedded-
>core@lists.openembedded.org> On Behalf Of Max Krummenacher via
>lists.openembedded.org
>Sent: 18 December 2020 09:52
>To: Diego Sueiro <Diego.Sueiro@arm.com>; openembedded-
>core@lists.openembedded.org
>Cc: nd <nd@arm.com>
>Subject: Re: [OE-core] [PATCH] u-boot: set SYSROOT_DIRS to /boot and
>${sysconfdir}
>
>Hi
>
>Am Donnerstag, den 17.12.2020, 11:23 +0000 schrieb Diego Sueiro:
>> Since do_install is shipping files to /boot and ${sysconfdir}, we can
>> set SYSROOT_DIRS to these dirs and allow recipes which depend on
>> u-boot to use its installed artifacts from RECIPE_SYSROOT instead of
>DEPLOY_DIR_IMAGE.
>>
>> Signed-off-by: Diego Sueiro <diego.sueiro@arm.com>
>> ---
>> meta/recipes-bsp/u-boot/u-boot.inc | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/meta/recipes-bsp/u-boot/u-boot.inc
>> b/meta/recipes-bsp/u-boot/u-boot.inc
>> index 251178db33..6af0766571 100644
>> --- a/meta/recipes-bsp/u-boot/u-boot.inc
>> +++ b/meta/recipes-bsp/u-boot/u-boot.inc
>> @@ -266,6 +266,8 @@ FILES_${PN}-env = " \ "
>>
>> FILES_${PN} = "/boot ${datadir}"
>> +SYSROOT_DIRS = "/boot ${sysconfdir}"
>
>Wouldn't a += be more future proof. I.e. not dropping the directories which
>are staged by default.
>
>Max
Thanks for your review. A v2 is under way.
>
>> +
>> RDEPENDS_${PN} += "${PN}-env"
>>
>> do_deploy () {
>>
>>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-01-04 8:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-17 11:23 [PATCH] u-boot: set SYSROOT_DIRS to /boot and ${sysconfdir} Diego Sueiro
2020-12-18 9:51 ` [OE-core] " Max Krummenacher
2021-01-04 8:29 ` Diego Sueiro
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox