Re: [PATCH] console-at91sam9-image: console-image with apps

[Philip Balister <philip@balister.org>]

On 03/12/2010 05:30 PM, Ulf Samuelsson wrote:
> From: Nicolas Ferre<nicolas.ferre@atmel.com>
>
> Create a derivative of console-image with more applications
>
> +       usbview
> +       mplayer
> +       thttpd
> +       madplay
> +       alsa-utils-aplay
> +       alsa-utils-amixer
> +       iperf
> +       dosfstools
> +       mtd-utils
>
> Signed-off-by: Ulf Samuelsson<ulf.samuelsson@atmel.com>
> ---
>   recipes/images/console-at91sam9-image.bb |   34 ++++++++++++++++++++++++++++++
>   1 files changed, 34 insertions(+), 0 deletions(-)
>   create mode 100644 recipes/images/console-at91sam9-image.bb
>
> diff --git a/recipes/images/console-at91sam9-image.bb b/recipes/images/console-at91sam9-image.bb
> new file mode 100644
> index 0000000..7aae0fb
> --- /dev/null
> +++ b/recipes/images/console-at91sam9-image.bb
> @@ -0,0 +1,34 @@
> +#Angstrom bootstrap at91sam9 image
> +
> +IMAGE_PREPROCESS_COMMAND = "create_etc_timestamp"
> +
> +ANGSTROM_EXTRA_INSTALL += " \
> +	usbview \
> +	mplayer \
> +	thttpd \
> +	madplay \
> +	alsa-utils-aplay \
> +	alsa-utils-amixer \
> +	iperf \
> +	dosfstools \
> +	mtd-utils \
> +#	nbench-byte \
> +	"

You should add these packages to IMAGE_INSTALL and (if you want the user 
to insert additional packages via an env var) add the var 
IMAGE_EXTRA_INSTALL to the IMAGE_INSTALL list. We do not want to keep 
adding the ANGSTROM_EXTRA_INSTALL variable to recipes that are not 
distro specific.

Philip

> +
> +DEPENDS = "task-base-extended \
> +           ${@base_contains("MACHINE_FEATURES", "screen", "psplash-zap", "",d)} \
> +	   "
> +
> +IMAGE_INSTALL = "task-base-extended \
> +	    ${ANGSTROM_EXTRA_INSTALL} \
> +	    ${@base_contains("MACHINE_FEATURES", "screen", "psplash-zap", "",d)} \
> +	   "
> +
> +export IMAGE_BASENAME = "console-at91sam9-image"
> +IMAGE_LINGUAS = ""
> +
> +#we dont need the kernel in the image
> +ROOTFS_POSTPROCESS_COMMAND += "rm -f ${IMAGE_ROOTFS}/boot/*Image*; "
> +
> +inherit image
> +