All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <tom_rini@mentor.com>
To: openembedded-devel@lists.openembedded.org
Cc: Chris Larson <chris_larson@mentor.com>
Subject: Re: [PATCH] Reverse the order of OVERRIDES
Date: Fri, 15 Oct 2010 07:18:38 -0700	[thread overview]
Message-ID: <4CB862BE.9070908@mentor.com> (raw)
In-Reply-To: <AANLkTimjtmWvRCypjwRfyFc7VnaJUL4R_0miq3aC3MRW@mail.gmail.com>

Khem Raj wrote:
> On Thu, Oct 14, 2010 at 8:36 PM, Chris Larson <kergoth@gmail.com> wrote:
>> From: Chris Larson <chris_larson@mentor.com>
>>
>> Given the current implementation of OVERRIDES in bitbake, the variable is
>> expected to contain elements in the order least specific to most specific,
>> however, our current usage of it does not match that.  As one example, "local"
>> is supposed to always be the most specific override, yet currently it's the
>> least specific.  As another example, currently the target architecture is seen
>> as more specific than the machine, which is also clearly wrong.
>>
>> It becomes clear that a reversal of the current value will bring us to a more
>> sane behavior, and avoids the need for the dual overrides hack mentioned in
>> the comments, so clean those up as well.
>>
>> This also introduces a MACHINE_OVERRIDES variable as a generic mechanism to
>> inject overrides elements which are more specific than the distro but less
>> specific than the machine, which is where things like MACHINE_CLASS or
>> SOC_FAMILY or the like would go.  This variable is *space* separated, to make
>> it easier and more convenient to assemble the variable incrementally, and it's
>> then translated to : separated when used in OVERRIDES.
>>
>> Signed-off-by: Chris Larson <chris_larson@mentor.com>
> 
> both things look nice to me. We might need to document MACHINE_OVERRIDES though.

Yes, no new variables without updating the docs.

-- 
Tom Rini
Mentor Graphics Corporation



  parent reply	other threads:[~2010-10-15 14:19 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-15  3:36 [PATCH] Reverse the order of OVERRIDES Chris Larson
2010-10-15  4:07 ` Khem Raj
2010-10-15  6:56   ` Frans Meulenbroeks
2010-10-15 14:18   ` Tom Rini [this message]
2010-10-15 10:42 ` Koen Kooi
2010-10-15 14:12 ` Maupin, Chase
2010-10-15 15:17   ` Chris Larson
2010-10-15 15:29     ` Maupin, Chase
2010-10-15 15:41 ` [PATCH (v2)] " Chris Larson
2010-10-15 16:02   ` Maupin, Chase
2010-10-15 17:20   ` Khem Raj
2010-10-15 19:37   ` Koen Kooi
2010-10-15 19:44     ` Chris Larson
2010-11-10 14:26       ` Richard Purdie
2010-11-10 16:22         ` Otavio Salvador
2010-11-10 16:27           ` Tom Rini
2010-11-10 16:29         ` Tom Rini
2010-12-01 20:26           ` Maupin, Chase
2010-12-01 21:29             ` Chris Larson
2010-10-16 18:04 ` [PATCH] " Martin Jansa
2010-10-16 18:24   ` Chris Larson
2010-10-16 18:50     ` Martin Jansa
2010-10-16 19:35       ` Martin Jansa
2010-10-16 19:54         ` Martin Jansa
2010-10-16 19:24     ` [PATCH] bitbake.conf: don't append MACHINE_OVERRIDES to OVERRIDES when it's empty Martin Jansa
2010-10-19 19:05       ` Maupin, Chase
2010-10-19 19:09         ` Chris Larson
2010-10-19 19:18           ` Maupin, Chase
2010-10-19 19:24             ` Denys Dmytriyenko
2010-10-20 23:29               ` Denys Dmytriyenko
2010-10-20 23:34                 ` Chris Larson
2010-10-21  2:45                   ` Denys Dmytriyenko
2010-10-21  6:27                 ` Martin Jansa
2010-10-21 14:04                   ` Chris Larson
2010-10-19 19:24           ` Maupin, Chase
2010-10-16 19:32     ` [PATCH] bitbake.conf: reverse OVERRIDES order in FILESPATH definition Martin Jansa

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=4CB862BE.9070908@mentor.com \
    --to=tom_rini@mentor.com \
    --cc=chris_larson@mentor.com \
    --cc=openembedded-devel@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.