Re: [PATCH 1/1] classes/image_types: IMAGE_TYPES was missing some entries

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Tue, 2011-07-26 at 07:06 -0700, Tom Rini wrote:
> On 07/25/2011 09:21 PM, Joshua Lock wrote:
> > Some of these entries are required by the GUI to enable the expected workflow
> > of the ADT, namely a compressed rootfs image.
> > 
> > Signed-off-by: Joshua Lock <josh@linux.intel.com>
> > ---
> >  meta/classes/image_types.bbclass |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/meta/classes/image_types.bbclass b/meta/classes/image_types.bbclass
> > index 1108802..f1aef86 100644
> > --- a/meta/classes/image_types.bbclass
> > +++ b/meta/classes/image_types.bbclass
> > @@ -103,4 +103,4 @@ IMAGE_DEPENDS_ubi = "mtd-utils-native"
> >  IMAGE_DEPENDS_ubifs = "mtd-utils-native"
> >  
> >  # This variable is available to request which values are suitable for IMAGE_FSTYPES
> > -IMAGE_TYPES = "jffs2 cramfs ext2 ext2.gz ext3 ext3.gz squashfs squashfs-lzma ubi"
> > +IMAGE_TYPES = "jffs2 cramfs ext2 ext2.gz ext3 ext3.gz squashfs squashfs-lzma ubi tar tar.gz tar.bz2 tar.xz cpio cpio.gz cpio.xz cpio.lzma"
> 
> So, is there some way we can do this dynamically?  Either oe-core is
> missing ext4, or we'll have to add that and all it's permutations.  And
> then there's u-boot signed copies of all of the ext* compressed versions.

The configs for the image types could do something like:

IMAGE_TYPE += "jffs2"
IMAGE_CMD_jffs2 = "xxx"

or we could switch to the syntax:

IMAGE_CMD[jffs2] = "xxx"

?

Cheers,

Richard