[PATCH 0/1] Fix PACKAGE_EXTRA_ARCHS

[PATCH 0/1] Fix PACKAGE_EXTRA_ARCHS

[Paul Eggleton]

Here's a fix as discussed for the PACKAGE_EXTRA_ARCHS issue since the
tune file changes. It may not be the proper fix but it does at least it
makes the value of this variable correct again.

The following changes since commit 1fe892ab6876c405599c79657221a8b4675b6ecf:

  Update TERMCMD message to align with previous change (2011-07-27 17:43:27 +0100)

are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib paule/core-pkgarch-fix
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/core-pkgarch-fix

Paul Eggleton (1):
  bitbake.conf: remove PACKAGE_EXTRA_ARCHS_tune-XXX default

 meta/conf/bitbake.conf |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

-- 
1.7.4.1

[PATCH 1/1] bitbake.conf: remove PACKAGE_EXTRA_ARCHS_tune-XXX default

[Paul Eggleton]

Because of the way BitBake handles ??= under certain circumstances, this
default setting ends up stepping all over the real setting from the arch
include file. Since virtually all arch include files or tune files define
a real value for this we shouldn't need to have a default (or it needs to
be done in a different way).

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta/conf/bitbake.conf |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 6e109ec..9c80f11 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -84,7 +84,6 @@ HOST_EXEEXT = ""
 TUNE_ARCH ??= "INVALID"
 TUNE_CCARGS ??= ""
 TUNE_FEATURES ??= "${TUNE_FEATURES_tune-${DEFAULTTUNE}}"
-PACKAGE_EXTRA_ARCHS_tune-${DEFAULTTUNE} ??= "${TARGET_ARCH}"
 LIBCEXTENSION ??= ""
 ABIEXTENSION ??= ""
 
-- 
1.7.4.1

Re: [PATCH 1/1] bitbake.conf: remove PACKAGE_EXTRA_ARCHS_tune-XXX default

[Kumar Gala]

On Jul 28, 2011, at 9:18 AM, Paul Eggleton wrote:

> Because of the way BitBake handles ??= under certain circumstances, this
> default setting ends up stepping all over the real setting from the arch
> include file. Since virtually all arch include files or tune files define
> a real value for this we shouldn't need to have a default (or it needs to
> be done in a different way).
> 
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> ---
> meta/conf/bitbake.conf |    1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)

Acked-by: Kumar Gala <galak@kernel.crashing.org>

- k

Re: [PATCH 0/1] Fix PACKAGE_EXTRA_ARCHS

[Saul Wold]

On 07/28/2011 07:18 AM, Paul Eggleton wrote:
> Here's a fix as discussed for the PACKAGE_EXTRA_ARCHS issue since the
> tune file changes. It may not be the proper fix but it does at least it
> makes the value of this variable correct again.
>
> The following changes since commit 1fe892ab6876c405599c79657221a8b4675b6ecf:
>
>    Update TERMCMD message to align with previous change (2011-07-27 17:43:27 +0100)
>
> are available in the git repository at:
>    git://git.openembedded.org/openembedded-core-contrib paule/core-pkgarch-fix
>    http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/core-pkgarch-fix
>
> Paul Eggleton (1):
>    bitbake.conf: remove PACKAGE_EXTRA_ARCHS_tune-XXX default
>
>   meta/conf/bitbake.conf |    1 -
>   1 files changed, 0 insertions(+), 1 deletions(-)
>
Pulled into OE-Core
Thanks
	Sau!