From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Bruce Ashfield <bruce.ashfield@gmail.com>
Cc: "poky@yoctoproject.org" <poky@yoctoproject.org>
Subject: Re: [PATCH 1/4] meta-emenlow: update to the new BSP layout
Date: Thu, 23 Dec 2010 17:00:06 +0000 [thread overview]
Message-ID: <1293123606.17519.83.camel@rex> (raw)
In-Reply-To: <AANLkTim+fk4y63aQWUivQOOnK2WxiijvwoMUd7K7=Eps@mail.gmail.com>
On Wed, 2010-12-22 at 06:06 -0800, Bruce Ashfield wrote:
> I was just thinking about this while reading your patch series, and I'm
> caught between two problems here.
>
> I like:
>
> - BSPs that are self contained.
>
> I'm ambivalent:
>
> - about having to update a lot of SRCREVs when I update the BSP branches.
>
> I have a problem:
>
> - If all the SRCREVs are not in a single location, I can't hunt them all down
> and update them effectively. That makes point #2 into "strongly" dislike.
>
> I've got 350+ patches that are about to land for 2.6.34 (from the
> 2.6.34 -longterm
> tree), and I'll be merging them out to the BSPs. So this SRCREV along with the
> rest will change.
>
> I'm looking for something that is effective in controlling the
> branches, but is also
> maintainable.
>
> Richard: can we have a central SRCREV in the default revisions (per
> machine still), and
> then override it in a BSP layer ? That way if for some reason you
> don't have the
> default revisions your BSP will be standalone, but I still have a way
> to update everything
> at once (and we'll eventually update the BSP layers as well).
>
> I can also set the SRCREVs in the recipes themselves via python (i.e.
> set a default
> again), that allows me a single point of control .. but perhaps
> doesn't match existing
> solutions.
We have all the options with the syntax we have. You can set things in
the core, the BSP can override, we can set things in the BSP that
override the core.
If you can clearly describe a consistent way you need the variables to
work, we can then implement it :)
Cheers,
Richard
next prev parent reply other threads:[~2010-12-23 17:00 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-22 7:39 [PATCH 0/4] Consolidated Pull and World Fixes Saul Wold
2010-12-22 7:40 ` [PATCH 1/4] meta-emenlow: update to the new BSP layout Tom Zanussi
2010-12-22 14:06 ` Bruce Ashfield
2010-12-23 17:00 ` Richard Purdie [this message]
2010-12-23 17:14 ` Bruce Ashfield
2010-12-27 5:24 ` Bruce Ashfield
2010-12-22 7:40 ` [PATCH 2/4] send-pull-request: allow users to select git-send-email or sendmail Darren Hart
2010-12-22 7:40 ` [PATCH 3/4] apt: update use-host patch to support x86_64 Saul Wold
2010-12-22 7:40 ` [PATCH 4/4] apr: update x86_64 site config file Saul Wold
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=1293123606.17519.83.camel@rex \
--to=richard.purdie@linuxfoundation.org \
--cc=bruce.ashfield@gmail.com \
--cc=poky@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.