Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Ross Burton <ross.burton@intel.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 0/2] Consolidate _FOR_BUILD variables
Date: Fri,  9 Nov 2012 10:09:27 +0000	[thread overview]
Message-ID: <cover.1352455397.git.ross.burton@intel.com> (raw)

Hi,

It's autotools best practise[1] to use the variables CC_FOR_BUILD etc when
building tools that are required to build the rest of the package.  A lot of
packages do this, and they all repeat the same variable assignements, so
consolidate these assignments in autotools.bbclass.

Not every instance of _FOR_BUILD has been removed from the metadata, as there
are several cases where the assignment isn't the same (added flags, mainly).

Tested with a world build of oe-core, zero buildhistory differences.

Ross

[1] http://www.sourceware.org/autobook/autobook/autobook_270.html#SEC270

The following changes since commit 78983e939ab17f02f8911c8b0d0e326b419856b9:

  lib/oe/classextend: Ensure we don't extend expressions more than once (2012-11-06 09:34:47 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ross/cross

for you to fetch changes up to 154a30032b03a9ce269752adbf5bbec85205720e:

  meta: remove redundant _FOR_BUILD variables (2012-11-09 10:00:05 +0000)

----------------------------------------------------------------
Ross Burton (2):
      autotools: set _FOR_BUILD variables here
      meta: remove redundant _FOR_BUILD variables

 meta/classes/autotools.bbclass                 |   15 +++++++++++++++
 meta/recipes-devtools/binutils/binutils.inc    |    4 ++--
 meta/recipes-devtools/rpm/rpm_5.4.9.bb         |    1 -
 meta/recipes-extended/bash/bash.inc            |    1 -
 meta/recipes-gnome/gnome/metacity_2.30.3.bb    |    5 ++---
 meta/recipes-graphics/mutter/mutter.inc        |    5 ++---
 meta/recipes-graphics/xorg-lib/libx11.inc      |    8 ++------
 meta/recipes-kernel/systemtap/systemtap_git.bb |    4 ----
 meta/recipes-support/gmp/gmp_4.2.1.bb          |    2 --
 meta/recipes-support/gmp/gmp_5.0.5.bb          |    2 --
 10 files changed, 23 insertions(+), 24 deletions(-)

Ross Burton (2):
  autotools: set _FOR_BUILD variables here
  meta: remove redundant _FOR_BUILD variables

 meta/classes/autotools.bbclass                 |   15 +++++++++++++++
 meta/recipes-devtools/binutils/binutils.inc    |    4 ++--
 meta/recipes-devtools/rpm/rpm_5.4.9.bb         |    1 -
 meta/recipes-extended/bash/bash.inc            |    1 -
 meta/recipes-gnome/gnome/metacity_2.30.3.bb    |    5 ++---
 meta/recipes-graphics/mutter/mutter.inc        |    5 ++---
 meta/recipes-graphics/xorg-lib/libx11.inc      |    8 ++------
 meta/recipes-kernel/systemtap/systemtap_git.bb |    4 ----
 meta/recipes-support/gmp/gmp_4.2.1.bb          |    2 --
 meta/recipes-support/gmp/gmp_5.0.5.bb          |    2 --
 10 files changed, 23 insertions(+), 24 deletions(-)

-- 
1.7.10




             reply	other threads:[~2012-11-09 10:24 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-09 10:09 Ross Burton [this message]
2012-11-09 10:09 ` [PATCH 1/2] autotools: set _FOR_BUILD variables here Ross Burton
2012-11-09 10:09 ` [PATCH 2/2] meta: remove redundant _FOR_BUILD variables Ross Burton
2012-11-09 22:32   ` Otavio Salvador
2012-11-09 22:57     ` Richard Purdie
2012-11-09 23:23       ` Otavio Salvador
2012-11-10 10:05         ` Richard Purdie
2012-11-10 11:11         ` Phil Blundell
2012-11-10 11:20           ` Tomas Frydrych

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=cover.1352455397.git.ross.burton@intel.com \
    --to=ross.burton@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox