core-image-tiny-initramfs: delete populate_sdk task

Message ID 20211216050914.90422-1-Qi.Chen@windriver.com
State New
Headers show
Series core-image-tiny-initramfs: delete populate_sdk task | expand

Commit Message

ChenQi Dec. 16, 2021, 5:09 a.m. UTC
Currently, if we do `bitbake core-image-tiny-initramfs' and then
install the generated SDK, the following error appears.

  tar: ./sysroots/core2-64-poky-linux/dev/console: Cannot mknod: Operation not permitted

The populate_sdk task does not make much sense for this recipe and
considering the fact the its SDK cannot be installed, we'd better just
delete the task.

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/recipes-core/images/core-image-tiny-initramfs.bb | 2 ++
 1 file changed, 2 insertions(+)

Comments

Ross Burton Dec. 28, 2021, 9:36 p.m. UTC | #1
On Thu, 16 Dec 2021 at 05:09, Chen Qi <Qi.Chen@windriver.com> wrote:
> Currently, if we do `bitbake core-image-tiny-initramfs' and then
> install the generated SDK, the following error appears.
>
>   tar: ./sysroots/core2-64-poky-linux/dev/console: Cannot mknod: Operation not permitted
>
> The populate_sdk task does not make much sense for this recipe and
> considering the fact the its SDK cannot be installed, we'd better just
> delete the task.

Feels to me that this is working around a problem instead of fixing the problem.

This happens for -tiny-initramfs because the root fs contains a
populated /dev tree, with device nodes in.  There is no hard
association between "has a static /dev" and "SDKs are meaningless", so
a better fix would be to remove /dev from the SDK entirely, instead of
removing SDKs from one image, when this is a general problem.

Ross
ChenQi Jan. 5, 2022, 6:02 a.m. UTC | #2
On 12/29/21 5:36 AM, Ross Burton wrote:
> On Thu, 16 Dec 2021 at 05:09, Chen Qi <Qi.Chen@windriver.com> wrote:
>> Currently, if we do `bitbake core-image-tiny-initramfs' and then
>> install the generated SDK, the following error appears.
>>
>>    tar: ./sysroots/core2-64-poky-linux/dev/console: Cannot mknod: Operation not permitted
>>
>> The populate_sdk task does not make much sense for this recipe and
>> considering the fact the its SDK cannot be installed, we'd better just
>> delete the task.
> Feels to me that this is working around a problem instead of fixing the problem.
>
> This happens for -tiny-initramfs because the root fs contains a
> populated /dev tree, with device nodes in.  There is no hard
> association between "has a static /dev" and "SDKs are meaningless", so
> a better fix would be to remove /dev from the SDK entirely, instead of
> removing SDKs from one image, when this is a general problem.
>
> Ross

Hi Ross,

I've sent out a new patch to remove '/dev' from SDK.

Regards,

Qi

Patch

diff --git a/meta/recipes-core/images/core-image-tiny-initramfs.bb b/meta/recipes-core/images/core-image-tiny-initramfs.bb
index 47470849ed..66c73e4261 100644
--- a/meta/recipes-core/images/core-image-tiny-initramfs.bb
+++ b/meta/recipes-core/images/core-image-tiny-initramfs.bb
@@ -23,6 +23,8 @@  IMAGE_FSTYPES = "${INITRAMFS_FSTYPES}"
 
 inherit core-image
 
+deltask do_populate_sdk
+
 IMAGE_ROOTFS_SIZE = "8192"
 IMAGE_ROOTFS_EXTRA_SPACE = "0"