From: Mark Hatle <mark.hatle@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 7/7] allarch: Drop various problematic allarch usages
Date: Mon, 15 Apr 2013 10:16:22 -0500 [thread overview]
Message-ID: <516C19C6.9080807@windriver.com> (raw)
In-Reply-To: <1366024028-24911-8-git-send-email-richard.purdie@linuxfoundation.org>
On 4/15/13 6:07 AM, Richard Purdie wrote:
> In each of these cases allarch is used where the package in question has a
> dependency on things which are not allach and change when MACHINE is changed.
>
> This leads to a rebuild of the package each time MACHINE is switched and
> the sstate checksum changes. The dependencies in question are not suited
> be being marked as ABISAFE.
In each of these cases, does the contents of the package change when the MACHINE
(or something in that machine) are modified? If so, I agree they are definitely
not allarch.
However, if the dependency really doesn't matter, then why isn't ABISAFE correct
in each case?
--Mark
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> meta/recipes-core/udev/udev-extraconf_1.0.bb | 2 --
> meta/recipes-devtools/devel-config/anjuta-remote-run.bb | 2 --
> meta/recipes-devtools/devel-config/distcc-config.bb | 2 --
> meta/recipes-devtools/devel-config/nfs-export-root.bb | 2 --
> meta/recipes-graphics/ttf-fonts/liberation-fonts_1.04.bb | 2 +-
> meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb | 2 +-
> meta/recipes-graphics/x11-common/x11-common_0.1.bb | 2 --
> 7 files changed, 2 insertions(+), 12 deletions(-)
>
> diff --git a/meta/recipes-core/udev/udev-extraconf_1.0.bb b/meta/recipes-core/udev/udev-extraconf_1.0.bb
> index ea95a60..fd9a333 100644
> --- a/meta/recipes-core/udev/udev-extraconf_1.0.bb
> +++ b/meta/recipes-core/udev/udev-extraconf_1.0.bb
> @@ -4,8 +4,6 @@ LICENSE = "MIT"
> LIC_FILES_CHKSUM = "file://${COREBASE}/LICENSE;md5=3f40d7994397109285ec7b81fdeb3b58 \
> file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
>
> -inherit allarch
> -
> PR = "r8"
>
> SRC_URI = " \
> diff --git a/meta/recipes-devtools/devel-config/anjuta-remote-run.bb b/meta/recipes-devtools/devel-config/anjuta-remote-run.bb
> index 51fdf9e..b9cded7 100644
> --- a/meta/recipes-devtools/devel-config/anjuta-remote-run.bb
> +++ b/meta/recipes-devtools/devel-config/anjuta-remote-run.bb
> @@ -15,5 +15,3 @@ do_install() {
> PACKAGES = "${PN}"
> RDEPENDS_${PN} = "dbus rsync"
>
> -inherit allarch
> -
> diff --git a/meta/recipes-devtools/devel-config/distcc-config.bb b/meta/recipes-devtools/devel-config/distcc-config.bb
> index 8deaa66..c9c024d 100644
> --- a/meta/recipes-devtools/devel-config/distcc-config.bb
> +++ b/meta/recipes-devtools/devel-config/distcc-config.bb
> @@ -21,5 +21,3 @@ do_install() {
> }
>
> RDEPENDS_${PN} = "distcc"
> -
> -inherit allarch
> diff --git a/meta/recipes-devtools/devel-config/nfs-export-root.bb b/meta/recipes-devtools/devel-config/nfs-export-root.bb
> index 4d9f6dd..5fa8cb3 100644
> --- a/meta/recipes-devtools/devel-config/nfs-export-root.bb
> +++ b/meta/recipes-devtools/devel-config/nfs-export-root.bb
> @@ -15,5 +15,3 @@ do_install() {
> }
>
> RDEPENDS_${PN} = "packagegroup-core-nfs-server"
> -
> -inherit allarch
> diff --git a/meta/recipes-graphics/ttf-fonts/liberation-fonts_1.04.bb b/meta/recipes-graphics/ttf-fonts/liberation-fonts_1.04.bb
> index a34825a..c99521f 100644
> --- a/meta/recipes-graphics/ttf-fonts/liberation-fonts_1.04.bb
> +++ b/meta/recipes-graphics/ttf-fonts/liberation-fonts_1.04.bb
> @@ -12,7 +12,7 @@ RDEPENDS_${PN} = "fontconfig-utils"
> PR = "r4"
> PE = "1"
>
> -inherit allarch fontcache
> +inherit fontcache
>
> FONT_PACKAGES = "${PN}"
>
> diff --git a/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb b/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb
> index ccd865c..75f78a2 100644
> --- a/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb
> +++ b/meta/recipes-graphics/ttf-fonts/ttf-bitstream-vera_1.10.bb
> @@ -10,7 +10,7 @@ LIC_FILES_CHKSUM = "file://COPYRIGHT.TXT;md5=27d7484b1e18d0ee4ce538644a3f04be"
> PR = "r7"
> RDEPENDS_${PN} = "fontconfig-utils"
>
> -inherit allarch fontcache
> +inherit fontcache
>
> FONT_PACKAGES = "${PN}"
>
> diff --git a/meta/recipes-graphics/x11-common/x11-common_0.1.bb b/meta/recipes-graphics/x11-common/x11-common_0.1.bb
> index 49de5a3..2df1347 100644
> --- a/meta/recipes-graphics/x11-common/x11-common_0.1.bb
> +++ b/meta/recipes-graphics/x11-common/x11-common_0.1.bb
> @@ -9,8 +9,6 @@ SRC_URI = "file://etc \
>
> S = "${WORKDIR}"
>
> -inherit allarch
> -
> do_install() {
> cp -R ${S}/etc ${D}${sysconfdir}
> chmod -R 755 ${D}${sysconfdir}
>
next prev parent reply other threads:[~2013-04-15 15:33 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-15 11:07 [PATCH 1/7] kernel.bbclass: Ensure we have correct version information in deploy data Richard Purdie
2013-04-15 11:07 ` [PATCH] recipes: Fix ALLOW_EMPTY with no package specified Richard Purdie
2013-04-15 11:43 ` Martin Jansa
2013-04-15 12:06 ` Richard Purdie
2013-04-15 11:07 ` [PATCH 2/7] update-alternatives: Ensure DEPENDS is correct in multilib case Richard Purdie
2013-04-15 11:07 ` [PATCH 3/7] ttf-bitstream-vera: Use fontcache class for postinstall Richard Purdie
2013-04-15 11:07 ` [PATCH 4/7] encodings: Set RDEPENDS correctly Richard Purdie
2013-04-15 11:07 ` [PATCH 5/7] qemuwrapper-cross: Inhibit default dependencies Richard Purdie
2013-04-15 11:07 ` [PATCH 6/7] nfs-export-root: Update to use packagegroup naming Richard Purdie
2013-04-15 11:07 ` [PATCH 7/7] allarch: Drop various problematic allarch usages Richard Purdie
2013-04-15 15:16 ` Mark Hatle [this message]
2013-04-15 15:40 ` Richard Purdie
2013-04-15 15:49 ` Mark Hatle
2013-04-15 16:15 ` Martin Jansa
2013-04-15 11:38 ` [PATCH 1/7] kernel.bbclass: Ensure we have correct version information in deploy data Martin Jansa
2013-04-15 14:46 ` Richard Purdie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=516C19C6.9080807@windriver.com \
--to=mark.hatle@windriver.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.