Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Peter Bergin <peter@berginkonsult.se>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] busybox: move default config fragments to defconfig
Date: Wed, 29 Mar 2017 11:24:29 +0100	[thread overview]
Message-ID: <1490783069.13980.307.camel@linuxfoundation.org> (raw)
In-Reply-To: <2b16b764-38e0-4112-48b4-9e17cbe129c3@berginkonsult.se>

On Wed, 2017-03-29 at 10:16 +0200, Peter Bergin wrote:
> On 2017-03-29 01:02, Richard Purdie wrote:
> > 
> > On Tue, 2017-03-28 at 22:08 +0200, Peter Bergin wrote:
> > > 
> > > Move config fragments that are unconditionally and always applied
> > > to the configuration to the default defconfig file. This will
> > > make it easier to provide your own defconfig file for your
> > > target.
> > > 
> > > The problem this commit will solve is that if you provide your
> > > own
> > > defconfig file for your target it will be tampered with by the
> > > configuration fragments in SRC_URI. If you not want those
> > > configurations in your own setup you have to modify SRC_URI
> > > variable in your setup. As those confguration fragment was always
> > > applied it is a better solution to have them in the default
> > > defconfig file instead.
> > This basically removes any standard feature grouping from being
> > used in
> > the base busybox recipe. I don't think this is something we want to
> > encourage, I'd actually like to see more feature group usage.
> > 
> > So we need to find some other way of solving your issue.
> > 
> > Cheers,
> > 
> > Richard
> I agree that it is nice and encouraged to group configurations
> together and make them selectable as a group.

Also, in case its not apparent, you can work around this today by
providing empty fragment files in your own layer. These would override
the default ones in the main recipe if you set it up right.

Cheers,

Richard


  parent reply	other threads:[~2017-03-29 10:24 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-28 20:08 [PATCH] busybox: move default config fragments to defconfig Peter Bergin
2017-03-28 23:02 ` Richard Purdie
2017-03-29  8:16   ` Peter Bergin
2017-03-29 10:22     ` Richard Purdie
2017-03-29 17:19       ` Peter Bergin
2017-03-30 10:08         ` Richard Purdie
2017-03-29 10:24     ` Richard Purdie [this message]
2017-03-29 15:23       ` Andre McCurdy
2017-03-29 17:25         ` Ulrich Ölmann
2017-03-29 18:25           ` Peter Bergin

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=1490783069.13980.307.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=peter@berginkonsult.se \
    /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