[PATCH 0/1] a fix to 2067

[PATCH 0/1] a fix to 2067

[Dexuan Cui]

The following changes since commit 6bd23e0faf329f086bb4885991ce5f5d095dac56:

  adt-installer: Corrected wrong environment file names. (2012-03-08 12:18:52 -0800)

are available in the git repository at:
  git://git.pokylinux.org/poky-contrib dcui/fix_2067
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/fix_2067

Dexuan Cui (1):
  image_types: ensure .rootfs.ext3 is created before vmdk is created.

 meta/classes/image_types.bbclass |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

-- 
1.7.6

[PATCH 1/1] image_types: ensure .rootfs.ext3 is created before vmdk is created.

[Dexuan Cui]

In the case of self-hosted-image.bb, IMAGE_FSTYPES = "vmdk", so the
variables alltypes and subimages don't contain ext3, and  .rootfs.ext3
won't be created, and finally the generated .hddimg and .vmdk don't have
an actual rootfs -- the size of the .vmdk file is only about 9MB.

[YOCTO #2067]

Signed-off-by: Dexuan Cui <dexuan.cui@intel.com>
---
 meta/classes/image_types.bbclass |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
index 8b82018..e92f519 100644
--- a/meta/classes/image_types.bbclass
+++ b/meta/classes/image_types.bbclass
@@ -26,13 +26,19 @@ def get_imagecmds(d):
     # Live and VMDK images will be processed via inheriting
     # bbclass and does not get processed here.
     # vmdk depend on live images also depend on ext3 so ensure its present
+    # Note: we need to ensure ext3 is in alltypes, otherwise, subimages may
+    # not contain ext3 and the .rootfs.ext3 file won't be created.
     if "vmdk" in types:
         if "ext3" not in types:
             types.append("ext3")
+        if "ext3" not in alltypes:
+            alltypes.append("ext3")
         types.remove("vmdk")
     if "live" in types:
         if "ext3" not in types:
             types.append("ext3")
+        if "ext3" not in alltypes:
+            alltypes.append("ext3")
         types.remove("live")
 
     if d.getVar('IMAGE_LINK_NAME', True):
-- 
1.7.6

Re: [PATCH 1/1] image_types: ensure .rootfs.ext3 is created before vmdk is created.

[Saul Wold]

On 03/08/2012 11:19 PM, Dexuan Cui wrote:
> In the case of self-hosted-image.bb, IMAGE_FSTYPES = "vmdk", so the
> variables alltypes and subimages don't contain ext3, and  .rootfs.ext3
> won't be created, and finally the generated .hddimg and .vmdk don't have
> an actual rootfs -- the size of the .vmdk file is only about 9MB.
>
> [YOCTO #2067]
>
> Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>

Nice Catch!

Acked-by: Saul Wold <sgw@linux.intel.com>

Sau!
> ---
>   meta/classes/image_types.bbclass |    6 ++++++
>   1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
> index 8b82018..e92f519 100644
> --- a/meta/classes/image_types.bbclass
> +++ b/meta/classes/image_types.bbclass
> @@ -26,13 +26,19 @@ def get_imagecmds(d):
>       # Live and VMDK images will be processed via inheriting
>       # bbclass and does not get processed here.
>       # vmdk depend on live images also depend on ext3 so ensure its present
> +    # Note: we need to ensure ext3 is in alltypes, otherwise, subimages may
> +    # not contain ext3 and the .rootfs.ext3 file won't be created.
>       if "vmdk" in types:
>           if "ext3" not in types:
>               types.append("ext3")
> +        if "ext3" not in alltypes:
> +            alltypes.append("ext3")
>           types.remove("vmdk")
>       if "live" in types:
>           if "ext3" not in types:
>               types.append("ext3")
> +        if "ext3" not in alltypes:
> +            alltypes.append("ext3")
>           types.remove("live")
>
>       if d.getVar('IMAGE_LINK_NAME', True):

Re: [PATCH 1/1] image_types: ensure .rootfs.ext3 is created before vmdk is created.

[Cui, Dexuan]

Saul Wold wrote on 2012-03-09:
> On 03/08/2012 11:19 PM, Dexuan Cui wrote:
>> In the case of self-hosted-image.bb, IMAGE_FSTYPES = "vmdk", so the
>> variables alltypes and subimages don't contain ext3, and
>> .rootfs.ext3 won't be created, and finally the generated .hddimg and
>> .vmdk don't have an actual rootfs -- the size of the .vmdk file is only about 9MB.
>> 
>> [YOCTO #2067]
>> 
>> Signed-off-by: Dexuan Cui<dexuan.cui@intel.com>
> 
> Nice Catch!
> 
> Acked-by: Saul Wold <sgw@linux.intel.com>
BTW, I didn't find the bug in my previous test because I didn't build
From scratch, so actually I already had a file
tmp/deploy/image/self-hosted-image-qemux86.ext3, and according to 
bootimg.bbclass's populate(), we used it -- but we should use a new .ext3 file.

Thanks,
-- Dexuan

Re: [PATCH 0/1] a fix to 2067

[Saul Wold]

On 03/08/2012 11:19 PM, Dexuan Cui wrote:
> The following changes since commit 6bd23e0faf329f086bb4885991ce5f5d095dac56:
>
>    adt-installer: Corrected wrong environment file names. (2012-03-08 12:18:52 -0800)
>
> are available in the git repository at:
>    git://git.pokylinux.org/poky-contrib dcui/fix_2067
>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/fix_2067
>
> Dexuan Cui (1):
>    image_types: ensure .rootfs.ext3 is created before vmdk is created.
>
>   meta/classes/image_types.bbclass |    6 ++++++
>   1 files changed, 6 insertions(+), 0 deletions(-)
>

Merged into OE-Core

Thanks
	Sau!