All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: "Robert P. J. Day" <rpjday@crashcourse.ca>
Cc: yocto@yoctoproject.org
Subject: Re: Use of BBPATH in a layer.conf file.
Date: Mon, 03 Dec 2012 10:34:13 +0000	[thread overview]
Message-ID: <5274191.3YuTQ9B7sW@helios> (raw)
In-Reply-To: <alpine.DEB.2.02.1212030315220.6264@oneiric>

On Monday 03 December 2012 03:16:10 Robert P. J. Day wrote:
> On Sun, 2 Dec 2012, Paul Eggleton wrote:
> > On Sunday 02 December 2012 15:35:54 you wrote:
> > > On 12/02/2012 03:24 PM, Paul Eggleton wrote:
> > > > On Sunday 02 December 2012 14:51:28 Scott Garman wrote:
> > > >> Robert Day has brought up an inconsistency in the way we append to
> > > >> BBPATH within a couple of our layer.conf files.
> > > >> 
> > > >> In meta-hob, meta-yocto-bsp, and meta-intel, we do:
> > > >> 
> > > >> BBPATH := "${BBPATH}:${LAYERDIR}"
> > > >> 
> > > >> but in meta-yocto, we do:
> > > >> 
> > > >> BBPATH := "${LAYERDIR}:${BBPATH}"
> > > >> 
> > > >> Unless someone explains to me that it's necessary to use this
> > > >> different
> > > >> ordering in meta-yocto's layer.conf, I will submit a patch to make
> > > >> this
> > > >> more consistent.
> > > > 
> > > > I think it actually ought to be:
> > > > 
> > > > BBPATH .= ":${LAYERDIR}"
> > > 
> > > Oh? Would this apply just to meta-yocto or all layer.conf files?
> > 
> > All, really. Functionally it makes no real difference, but I think it's
> > preferred stylistically based on previous discussions.
> 
>   hang on ... i thought the ordering of BBPATH would affect the
> processing of "include" directives.  no?

Yes, it does. What I mean is, functionally _within layer.conf_ the following 
two are equivalent:

BBPATH := "${BBPATH}:${LAYERDIR}"

BBPATH .= ":${LAYERDIR}"

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


  reply	other threads:[~2012-12-03 10:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-02 22:51 Use of BBPATH in a layer.conf file Scott Garman
2012-12-02 23:24 ` Paul Eggleton
2012-12-02 23:35   ` Scott Garman
2012-12-02 23:37     ` Paul Eggleton
2012-12-03  8:16       ` Robert P. J. Day
2012-12-03 10:34         ` Paul Eggleton [this message]
2012-12-03 17:43 ` Tomas Frydrych
2012-12-03 18:59   ` Scott Garman

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=5274191.3YuTQ9B7sW@helios \
    --to=paul.eggleton@linux.intel.com \
    --cc=rpjday@crashcourse.ca \
    --cc=yocto@yoctoproject.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.