* [PATCH] image_types_wic.bbclass: Ensure '-c image_wic' works
@ 2017-12-19 17:17 Otavio Salvador
2017-12-19 17:22 ` Richard Purdie
0 siblings, 1 reply; 5+ messages in thread
From: Otavio Salvador @ 2017-12-19 17:17 UTC (permalink / raw)
To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador
The 'wic' image may use files which has been deployed (e.g in
do_deploy) for some partitions that may need to be copied in a
specific partition or filesystem (e.g a bootloader).
When using 'bitbake <image> -c image_wic', from a clean build, the
contents of do_deploy must also be available so we need to ensure all
do_build of the image dependencies has been complete.
Reported-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
meta/classes/image_types_wic.bbclass | 3 +++
1 file changed, 3 insertions(+)
diff --git a/meta/classes/image_types_wic.bbclass b/meta/classes/image_types_wic.bbclass
index 222ae00433..4d7305a3a3 100644
--- a/meta/classes/image_types_wic.bbclass
+++ b/meta/classes/image_types_wic.bbclass
@@ -41,6 +41,9 @@ WKS_FILE_CHECKSUM = "${@'${WKS_FULL_PATH}:%s' % os.path.exists('${WKS_FULL_PATH}
do_image_wic[file-checksums] += "${WKS_FILE_CHECKSUM}"
do_image_wic[depends] += "${@' '.join('%s-native:do_populate_sysroot' % r for r in ('parted', 'gptfdisk', 'dosfstools', 'mtools'))}"
+# We ensure all artfacts are deployed (e.g virtual/bootloader)
+do_image_wic[recrdeptask] += "do_build"
+
WKS_FILE_DEPENDS_DEFAULT = "syslinux-native bmap-tools-native cdrtools-native btrfs-tools-native squashfs-tools-native e2fsprogs-native"
WKS_FILE_DEPENDS_BOOTLOADERS = ""
WKS_FILE_DEPENDS_BOOTLOADERS_x86 = "syslinux grub-efi systemd-boot"
--
2.15.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] image_types_wic.bbclass: Ensure '-c image_wic' works
2017-12-19 17:17 [PATCH] image_types_wic.bbclass: Ensure '-c image_wic' works Otavio Salvador
@ 2017-12-19 17:22 ` Richard Purdie
2017-12-19 17:38 ` Otavio Salvador
0 siblings, 1 reply; 5+ messages in thread
From: Richard Purdie @ 2017-12-19 17:22 UTC (permalink / raw)
To: Otavio Salvador, OpenEmbedded Core Mailing List
On Tue, 2017-12-19 at 15:17 -0200, Otavio Salvador wrote:
> The 'wic' image may use files which has been deployed (e.g in
> do_deploy) for some partitions that may need to be copied in a
> specific partition or filesystem (e.g a bootloader).
>
> When using 'bitbake <image> -c image_wic', from a clean build, the
> contents of do_deploy must also be available so we need to ensure all
> do_build of the image dependencies has been complete.
>
> Reported-by: Fabio Berton <fabio.berton@ossystems.com.br>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>
> meta/classes/image_types_wic.bbclass | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/meta/classes/image_types_wic.bbclass
> b/meta/classes/image_types_wic.bbclass
> index 222ae00433..4d7305a3a3 100644
> --- a/meta/classes/image_types_wic.bbclass
> +++ b/meta/classes/image_types_wic.bbclass
> @@ -41,6 +41,9 @@ WKS_FILE_CHECKSUM = "${@'${WKS_FULL_PATH}:%s' %
> os.path.exists('${WKS_FULL_PATH}
> do_image_wic[file-checksums] += "${WKS_FILE_CHECKSUM}"
> do_image_wic[depends] += "${@' '.join('%s-
> native:do_populate_sysroot' % r for r in ('parted', 'gptfdisk',
> 'dosfstools', 'mtools'))}"
>
> +# We ensure all artfacts are deployed (e.g virtual/bootloader)
> +do_image_wic[recrdeptask] += "do_build"
Shouldn't that be do_deploy instead of do_build?
Cheers,
Richard
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] image_types_wic.bbclass: Ensure '-c image_wic' works
2017-12-19 17:22 ` Richard Purdie
@ 2017-12-19 17:38 ` Otavio Salvador
2017-12-19 18:07 ` Richard Purdie
0 siblings, 1 reply; 5+ messages in thread
From: Otavio Salvador @ 2017-12-19 17:38 UTC (permalink / raw)
To: Richard Purdie; +Cc: Otavio Salvador, OpenEmbedded Core Mailing List
On Tue, Dec 19, 2017 at 3:22 PM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Tue, 2017-12-19 at 15:17 -0200, Otavio Salvador wrote:
>> The 'wic' image may use files which has been deployed (e.g in
>> do_deploy) for some partitions that may need to be copied in a
>> specific partition or filesystem (e.g a bootloader).
>>
>> When using 'bitbake <image> -c image_wic', from a clean build, the
>> contents of do_deploy must also be available so we need to ensure all
>> do_build of the image dependencies has been complete.
>>
>> Reported-by: Fabio Berton <fabio.berton@ossystems.com.br>
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> ---
>>
>> meta/classes/image_types_wic.bbclass | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/meta/classes/image_types_wic.bbclass
>> b/meta/classes/image_types_wic.bbclass
>> index 222ae00433..4d7305a3a3 100644
>> --- a/meta/classes/image_types_wic.bbclass
>> +++ b/meta/classes/image_types_wic.bbclass
>> @@ -41,6 +41,9 @@ WKS_FILE_CHECKSUM = "${@'${WKS_FULL_PATH}:%s' %
>> os.path.exists('${WKS_FULL_PATH}
>> do_image_wic[file-checksums] += "${WKS_FILE_CHECKSUM}"
>> do_image_wic[depends] += "${@' '.join('%s-
>> native:do_populate_sysroot' % r for r in ('parted', 'gptfdisk',
>> 'dosfstools', 'mtools'))}"
>>
>> +# We ensure all artfacts are deployed (e.g virtual/bootloader)
>> +do_image_wic[recrdeptask] += "do_build"
>
> Shouldn't that be do_deploy instead of do_build?
Not all recipes has do_deploy. Wouldn't it cause problems?
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] image_types_wic.bbclass: Ensure '-c image_wic' works
2017-12-19 17:38 ` Otavio Salvador
@ 2017-12-19 18:07 ` Richard Purdie
2017-12-19 18:17 ` Otavio Salvador
0 siblings, 1 reply; 5+ messages in thread
From: Richard Purdie @ 2017-12-19 18:07 UTC (permalink / raw)
To: Otavio Salvador; +Cc: Otavio Salvador, OpenEmbedded Core Mailing List
On Tue, 2017-12-19 at 15:38 -0200, Otavio Salvador wrote:
> On Tue, Dec 19, 2017 at 3:22 PM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > On Tue, 2017-12-19 at 15:17 -0200, Otavio Salvador wrote:
> > >
> > > The 'wic' image may use files which has been deployed (e.g in
> > > do_deploy) for some partitions that may need to be copied in a
> > > specific partition or filesystem (e.g a bootloader).
> > >
> > > When using 'bitbake <image> -c image_wic', from a clean build,
> > > the
> > > contents of do_deploy must also be available so we need to ensure
> > > all
> > > do_build of the image dependencies has been complete.
> > >
> > > Reported-by: Fabio Berton <fabio.berton@ossystems.com.br>
> > > Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> > > ---
> > >
> > > meta/classes/image_types_wic.bbclass | 3 +++
> > > 1 file changed, 3 insertions(+)
> > >
> > > diff --git a/meta/classes/image_types_wic.bbclass
> > > b/meta/classes/image_types_wic.bbclass
> > > index 222ae00433..4d7305a3a3 100644
> > > --- a/meta/classes/image_types_wic.bbclass
> > > +++ b/meta/classes/image_types_wic.bbclass
> > > @@ -41,6 +41,9 @@ WKS_FILE_CHECKSUM = "${@'${WKS_FULL_PATH}:%s' %
> > > os.path.exists('${WKS_FULL_PATH}
> > > do_image_wic[file-checksums] += "${WKS_FILE_CHECKSUM}"
> > > do_image_wic[depends] += "${@' '.join('%s-
> > > native:do_populate_sysroot' % r for r in ('parted', 'gptfdisk',
> > > 'dosfstools', 'mtools'))}"
> > >
> > > +# We ensure all artfacts are deployed (e.g virtual/bootloader)
> > > +do_image_wic[recrdeptask] += "do_build"
> > Shouldn't that be do_deploy instead of do_build?
> Not all recipes has do_deploy. Wouldn't it cause problems?
No, its designed to handle that:
$ grep recrdep * -R | grep deploy
meta/classes/populate_sdk_ext.bbclass:do_sdk_depends[recrdeptask] += "do_populate_lic do_package_qa do_populate_sysroot do_deploy ${SDK_RECRDEP_TASKS}"
meta/classes/archiver.bbclass:do_deploy_all_archives[recrdeptask] = "do_deploy_archives"
meta/classes/populate_sdk_ext.bbclass.orig:do_sdk_depends[recrdeptask] += "do_populate_lic do_package_qa do_populate_sysroot do_deploy ${SDK_RECRDEP_TASKS}"
meta/classes/base.bbclass:do_build[recrdeptask] += "do_deploy"
Cheers,
Richard
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] image_types_wic.bbclass: Ensure '-c image_wic' works
2017-12-19 18:07 ` Richard Purdie
@ 2017-12-19 18:17 ` Otavio Salvador
0 siblings, 0 replies; 5+ messages in thread
From: Otavio Salvador @ 2017-12-19 18:17 UTC (permalink / raw)
To: Richard Purdie; +Cc: Otavio Salvador, OpenEmbedded Core Mailing List
On Tue, Dec 19, 2017 at 4:07 PM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Tue, 2017-12-19 at 15:38 -0200, Otavio Salvador wrote:
>> On Tue, Dec 19, 2017 at 3:22 PM, Richard Purdie
>> <richard.purdie@linuxfoundation.org> wrote:
>> >
>> > On Tue, 2017-12-19 at 15:17 -0200, Otavio Salvador wrote:
>> > >
>> > > The 'wic' image may use files which has been deployed (e.g in
>> > > do_deploy) for some partitions that may need to be copied in a
>> > > specific partition or filesystem (e.g a bootloader).
>> > >
>> > > When using 'bitbake <image> -c image_wic', from a clean build,
>> > > the
>> > > contents of do_deploy must also be available so we need to ensure
>> > > all
>> > > do_build of the image dependencies has been complete.
>> > >
>> > > Reported-by: Fabio Berton <fabio.berton@ossystems.com.br>
>> > > Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> > > ---
>> > >
>> > > meta/classes/image_types_wic.bbclass | 3 +++
>> > > 1 file changed, 3 insertions(+)
>> > >
>> > > diff --git a/meta/classes/image_types_wic.bbclass
>> > > b/meta/classes/image_types_wic.bbclass
>> > > index 222ae00433..4d7305a3a3 100644
>> > > --- a/meta/classes/image_types_wic.bbclass
>> > > +++ b/meta/classes/image_types_wic.bbclass
>> > > @@ -41,6 +41,9 @@ WKS_FILE_CHECKSUM = "${@'${WKS_FULL_PATH}:%s' %
>> > > os.path.exists('${WKS_FULL_PATH}
>> > > do_image_wic[file-checksums] += "${WKS_FILE_CHECKSUM}"
>> > > do_image_wic[depends] += "${@' '.join('%s-
>> > > native:do_populate_sysroot' % r for r in ('parted', 'gptfdisk',
>> > > 'dosfstools', 'mtools'))}"
>> > >
>> > > +# We ensure all artfacts are deployed (e.g virtual/bootloader)
>> > > +do_image_wic[recrdeptask] += "do_build"
>> > Shouldn't that be do_deploy instead of do_build?
>> Not all recipes has do_deploy. Wouldn't it cause problems?
>
> No, its designed to handle that:
>
> $ grep recrdep * -R | grep deploy
> meta/classes/populate_sdk_ext.bbclass:do_sdk_depends[recrdeptask] += "do_populate_lic do_package_qa do_populate_sysroot do_deploy ${SDK_RECRDEP_TASKS}"
> meta/classes/archiver.bbclass:do_deploy_all_archives[recrdeptask] = "do_deploy_archives"
> meta/classes/populate_sdk_ext.bbclass.orig:do_sdk_depends[recrdeptask] += "do_populate_lic do_package_qa do_populate_sysroot do_deploy ${SDK_RECRDEP_TASKS}"
> meta/classes/base.bbclass:do_build[recrdeptask] += "do_deploy"
Thanks for clarifying it. I sent a v2 adjusting that.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-12-19 18:17 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-19 17:17 [PATCH] image_types_wic.bbclass: Ensure '-c image_wic' works Otavio Salvador
2017-12-19 17:22 ` Richard Purdie
2017-12-19 17:38 ` Otavio Salvador
2017-12-19 18:07 ` Richard Purdie
2017-12-19 18:17 ` Otavio Salvador
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.