[meta-freescale] [meta-fsl-arm][PATCH] New packagegroups to organize tools.

[Daiane Angolini]

On Mon, Jul 6, 2015 at 5:32 PM, Ann Thornton <ra43240 at freescale.com> wrote:
> The tools are grouped into core and extended with i.MX specific
> tools in a .bbappend file specific to _imx.
>
> Signed-off-by: Ann Thornton <ra43240 at freescale.com>
> ---
>  .../packagegroups/packagegroup-fsl-tools-core.bb   |   39 +++++++++++++++++
>  .../packagegroup-fsl-tools-extended.bb             |   45 ++++++++++++++++++++
>  .../packagegroup-fsl-tools-extended.bbappend       |   22 ++++++++++
>  3 files changed, 106 insertions(+)
>  create mode 100644 recipes-fsl/packagegroups/packagegroup-fsl-tools-core.bb
>  create mode 100644 recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bb
>  create mode 100644 recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bbappend
>
> diff --git a/recipes-fsl/packagegroups/packagegroup-fsl-tools-core.bb b/recipes-fsl/packagegroups/packagegroup-fsl-tools-core.bb
> new file mode 100644
> index 0000000..7b7fdae
> --- /dev/null
> +++ b/recipes-fsl/packagegroups/packagegroup-fsl-tools-core.bb
> @@ -0,0 +1,39 @@
> +# Copyright (C) 2015 Freescale Semiconductor
> +# Released under the MIT license (see COPYING.MIT for the terms)
> +
> +SUMMARY = "Freescale Package group for core tools"
> +DESCRIPTION = "Tools that are considered to be core tools.  This can be used \
> +    to build a core image."
> +
> +LICENSE = "MIT"
> +LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58 \
> +                    file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
> +
> +inherit packagegroup
> +
> +PACKAGES = "${PN}"
> +
> +RDEPENDS_${PN} = " \
> +    e2fsprogs \
> +    e2fsprogs-badblocks \
> +    e2fsprogs-e2fsck \
> +    e2fsprogs-tune2fs  \
> +    i2c-tools \
> +    kmod \
> +    libhugetlbfs \
> +    lmsensors-sensors \
> +    memtester \
> +    pkgconfig \
> +    python-subprocess \
> +    python-datetime \
> +    python-json \
> +    procps \
> +    minicom \
> +    coreutils \
> +    elfutils \
> +    file \
> +    psmisc \
> +    sysfsutils \
> +    sysklogd \
> +    sysstat \
> +"
> diff --git a/recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bb b/recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bb
> new file mode 100644
> index 0000000..54c9b78
> --- /dev/null
> +++ b/recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bb
> @@ -0,0 +1,45 @@
> +# Copyright (C) 2015 Freescale Semiconductor
> +# Released under the MIT license (see COPYING.MIT for the terms)
> +
> +SUMMARY = "Freescale Package group for extended tools"
> +DESCRIPTION = "These tools are considered useful but not necessary in a minimal \
> +    or core image."
> +LICENSE = "MIT"
> +LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58 \
> +                    file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
> +
> +inherit packagegroup
> +
> +PACKAGES = "${PN}"
> +
> +X11_TOOLS = "${@base_contains('DISTRO_FEATURES', 'x11', \
> +    ' lsb \
> +    lsbinitscripts \
> +    lsbtest ', '', d)} \
> +"
> +
> +RDEPENDS_${PN} = "\
> +    bc \
> +    chkconfig \
> +    cronie \
> +    db \
> +    debianutils \
> +    lsof \
> +    man \
> +    man-pages \
> +    oprofile \
> +    parted \
> +    perf \
> +    rng-tools \
> +    rt-tests \
> +    sqlite3 \
> +    texinfo \
> +    unzip \
> +    usbutils \
> +    usbutils-ids \
> +    watchdog \
> +    which \
> +    xz  \
> +    zip \
> +    ${X11_TOOLS} \
> +"
> diff --git a/recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bbappend b/recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bbappend
> new file mode 100644
> index 0000000..a6b1776
> --- /dev/null
> +++ b/recipes-fsl/packagegroups/packagegroup-fsl-tools-extended.bbappend
> @@ -0,0 +1,22 @@
> +# Copyright 2015 Freescale Semiconductor, Inc.
> +# Released under the MIT license (see COPYING.MIT for the terms)
> +
> +DESCRIPTION_imx = "Tools specific to i.MX."
> +LICENSE = "MIT"
> +LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58 \
> +                    file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
> +
> +inherit packagegroup
> +
> +RDEPENDS_${PN}_append_imx = " \
> +    imx-kobs \
> +    ${SOC_TOOLS_TEST} \
> +"
> +
> +SOC_TOOLS_TEST = ""
> +SOC_TOOLS_TEST_vf  = "imx-test"
> +SOC_TOOLS_TEST_mxs = "imx-test"
> +SOC_TOOLS_TEST_mx3 = "imx-test"
> +SOC_TOOLS_TEST_mx5 = "imx-test"
> +SOC_TOOLS_TEST_mx6 = "imx-test"
> +SOC_TOOLS_TEST_mx7 = "imx-test"

We still don't have mx7 support.

I don't quite undertand why it is important to set SOC_TOOLS_TEST for
the machines with the dummy imx-test package if you start using an
empty variable. I would replace it with

 +SOC_TOOLS_TEST = ""
 +SOC_TOOLS_TEST_mx5 = "imx-test"
 +SOC_TOOLS_TEST_mx6 = "imx-test"

But I may be missing something.


Daiane

> --
> 1.7.9.5
>
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale