From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Bruce Ashfield <bruce.ashfield@gmail.com>,
"Robert P. J. Day" <rpjday@crashcourse.ca>
Cc: OE Core mailing list <openembedded-core@lists.openembedded.org>
Subject: Re: design question: should layer.conf contain "PREFERRED_VERSION" settings?
Date: Wed, 22 Mar 2017 14:59:58 +0000 [thread overview]
Message-ID: <1490194798.13980.172.camel@linuxfoundation.org> (raw)
In-Reply-To: <CADkTA4OV+n4cMpR-6GTbM+Qu+bJ8TNntOXyQ7UWYDLHtB_JYfg@mail.gmail.com>
On Wed, 2017-03-22 at 08:33 -0400, Bruce Ashfield wrote:
>
>
> On Wed, Mar 22, 2017 at 8:24 AM, Robert P. J. Day <rpjday@crashcourse
> .ca> wrote:
> >
> > proper attributions seem to have been totally lost here ...
> >
> > On Wed, 22 Mar 2017, Bruce Ashfield wrote:
> >
> > > On Wed, Mar 22, 2017 at 8:02 AM, Burton, Ross <ross.burton@intel.
> > com> wrote:
> > >
> > > On 22 March 2017 at 11:57, Bruce Ashfield <bruce.ashfield@g
> > mail.com> wrote:
> > > So where are they supposed to be specified ? Surely
> > not a separate distro .conf ? and most
> > > certainly not in local.conf ... I've always been unsure of where
> > the right place to specify
> > > versions like that.
> > >
> > > If it were my layer I'd have an .inc file that set all the
> > preferred
> > > versions where the defaults don't work, that the user would have
> > to
> > > include in their distro.
> > >
> > > Aha. Kind of like the old versions file that used to float
> > around.
> > >
> > > Seems reasonable (if a bit less automatic than someone like me
> > who
> > > wants .. to use the layer like a black box)
> > >
> > > Luckily I have push access to that layer, so I can make a change
> > > like that ;)
> > >
> > > One final question, is it considered a design option to set those
> > > versions triggered off an IMAGE or DISTRO feature ? i.e. in
> > > anonymous python .. or is that already too late in the
> > > parsing/resolution process ?
> > >
> > > Bruce
> >
> > sorry, didn't mean to start something this early in the morning
> > ...
> > ok, i did. :-)
> >
> > in any event, can we agree that:
> >
> > 1) it's bad layer design if the simple act of including a layer in
> > bblayers.conf suddenly causes weird things to happen like the
> > above?
> >
> > 2) regardless of how the developer eventually does it, picking up
> > those PREFERRED VERSIONS from meta-openstack should require *some*
> > kind of explicit selection?
>
> Honestly .. I disagree on #2.
>
> The packages within the openstack layer *will not work* with other
> versions of those
> packages.
>
> So if someone actually wants to use the functionality provided by the
> layer, they are
> not optional. And the errors/issues that you get are extremely obtuse
> and hard to
> debug. That's the design point of the layer .. it doesn't let that
> happen.
>
> Hence making it something they must do as an extra step .. really is
> a bad idea.
>
> It gets in the way of people picking packages out of that layer if
> they don't want to
> actually build openstack, that's the crux of the problem. But to
> answer that issue,
> that's why meta-cloud-services exists, that doesn't' force versions
> and holds the
> generally useful "cloud" packages.
Having layers which magically change the "policy" such as selected
versions goes against the spirit and wording of Yocto Project
Compatible (and will go against the new layer checking tools).
What should happen is that the openstack recipe should complain loudly
(error) if anyone tried to build it (or parse it?) with anything except
the versions its known to work with.
Cheers,
Richard
next prev parent reply other threads:[~2017-03-22 15:00 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-22 11:14 design question: should layer.conf contain "PREFERRED_VERSION" settings? Robert P. J. Day
2017-03-22 11:19 ` Burton, Ross
2017-03-22 11:57 ` Bruce Ashfield
2017-03-22 12:02 ` Burton, Ross
2017-03-22 12:14 ` Bruce Ashfield
2017-03-22 12:24 ` Robert P. J. Day
2017-03-22 12:33 ` Bruce Ashfield
2017-03-22 12:53 ` Robert P. J. Day
2017-03-22 14:59 ` Richard Purdie [this message]
2017-03-22 15:44 ` Burton, Ross
2017-03-22 16:32 ` Richard Purdie
2017-03-24 13:54 ` Burton, Ross
2017-03-22 17:02 ` Bruce Ashfield
2017-03-22 15:37 ` Daniel Dickinson
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=1490194798.13980.172.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=bruce.ashfield@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=rpjday@crashcourse.ca \
/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.