[PATCH 0/1] Remove copy_boot_files task for SDK

[PATCH 0/1] Remove copy_boot_files task for SDK

[Uladzimir Bely]

In the downstream we faced up a problem with populating SDK when
DTB_FILES is set for the target.

Error happens in `copy_boot_files` task that looks like something that
shouldn't even be executed in case of SDK. It tries to copy kernel,
initramfs and required dtb files to the deploy directory. But this is
not required for SDK rootfs that is just a developer chroot.

Uladzimir Bely (1):
  Remove copy_boot_files task for SDK

 meta/classes/sdk.bbclass | 1 +
 1 file changed, 1 insertion(+)

-- 
2.20.1

[PATCH 1/1] Remove copy_boot_files task for SDK

[Uladzimir Bely]

This task just copies kernel/initrd/dtb files from rootfs to deploy dir
that is useless for SDK (and already done with non-sdk rootfs).

Moreover, if DTB_FILES is set (like in phyboard-mira target, for
example), copy_boot_files produces error when "populate_sdk" executed.

Signed-off-by: Uladzimir Bely <ubely@ilbers.de>
---
 meta/classes/sdk.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/sdk.bbclass b/meta/classes/sdk.bbclass
index 32ee2de7..ec5eed87 100644
--- a/meta/classes/sdk.bbclass
+++ b/meta/classes/sdk.bbclass
@@ -16,6 +16,7 @@ python sdk_virtclass_handler() {
         e.data.appendVar('OVERRIDES', ':class-sdk')
         # sdkchroot deploy only for sdk image
         bb.build.addtask('deploy_sdkchroot', 'do_build', 'do_rootfs', d)
+        bb.build.deltask('copy_boot_files', d)
     else:
         # add do_populate_sdk only to the non-sdk variant
         # it only exists to preserve the interface...
-- 
2.20.1

Re: [PATCH 0/1] Remove copy_boot_files task for SDK

[Anton Mikanovich]

24.10.2022 08:40, Uladzimir Bely wrote:
> In the downstream we faced up a problem with populating SDK when
> DTB_FILES is set for the target.
>
> Error happens in `copy_boot_files` task that looks like something that
> shouldn't even be executed in case of SDK. It tries to copy kernel,
> initramfs and required dtb files to the deploy directory. But this is
> not required for SDK rootfs that is just a developer chroot.
>
> Uladzimir Bely (1):
>    Remove copy_boot_files task for SDK
>
>   meta/classes/sdk.bbclass | 1 +
>   1 file changed, 1 insertion(+)
>
Applied to next, thanks.