[PATCH 0/1] Add remaining types to IMAGE_TYPES

[PATCH 0/1] Add remaining types to IMAGE_TYPES

[Joshua Lock]

My original patch to add the IMAGE_TYPES variable for use in hob missed
several image types from the list. The patch here fixes that.

The following changes since commit f10a3457cdfbb4a94978da998d178d4254632fa7:

  Move architecture specific TARGET_OS mangling into tune files (2011-07-25 14:19:54 +0100)

are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib josh/image-types
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=josh/image-types

Joshua Lock (1):
  classes/image_types: IMAGE_TYPES was missing some entries

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

-- 
1.7.6

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

[Joshua Lock]

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"
-- 
1.7.6

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

[Tom Rini]

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.

-- 
Tom Rini
Mentor Graphics Corporation

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

[Richard Purdie]

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

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

[Richard Purdie]

On Mon, 2011-07-25 at 21:21 -0700, 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(-)

I've merged this since its obviously correct, we probably do need to
think about better ways to generate it though as Tom mentions...

Cheers,

Richard

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

[Joshua Lock]

On Tue, 2011-07-26 at 15:53 +0100, Richard Purdie wrote:
> On Mon, 2011-07-25 at 21:21 -0700, 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(-)
> 
> I've merged this since its obviously correct, we probably do need to
> think about better ways to generate it though as Tom mentions...

Agreed, I'll add this to my to do list.

Thanks,
Joshua
-- 
Joshua Lock
        Yocto Project "Johannes factotum"
        Intel Open Source Technology Centre

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

[Tom Rini]

On 07/26/2011 09:03 AM, Joshua Lock wrote:
> On Tue, 2011-07-26 at 15:53 +0100, Richard Purdie wrote:
>> On Mon, 2011-07-25 at 21:21 -0700, 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(-)
>>
>> I've merged this since its obviously correct, we probably do need to
>> think about better ways to generate it though as Tom mentions...
> 
> Agreed, I'll add this to my to do list.

Thanks!

-- 
Tom Rini
Mentor Graphics Corporation