Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Phil Blundell <philb@gnu.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Cc: Yocto Project <yocto@yoctoproject.org>
Subject: Re: [PATCH 2/3] busybox: Add setsid and cttyhack for tiny DISTRO_FEATURE
Date: Thu, 14 Jun 2012 15:26:08 +0100	[thread overview]
Message-ID: <1339683970.14857.86.camel@phil-desktop> (raw)
In-Reply-To: <4FD9ED69.50800@linux.intel.com>

On Thu, 2012-06-14 at 06:55 -0700, Darren Hart wrote:
> Another approach would be to just consider these two features and decide
> if they shouldn't just be part of the oe-core busybox defconfig anyway.
> I don't see why setsid shouldn't be. I can see arguments against
> cttyhack (as it is a hack), but I wouldn't think either should be a huge
> deal to just include. I thought the DISTRO_FEATURE was a reasonable
> compromise between that and having to maintain a bbappend outside of
> oe-core.

Well, you don't need to maintain a .bbappend as such: you can just
provide your own, completely custom config file for busybox.  That's
what micro does, what shr does (I think), and what I would suggest that
most DISTROs with non-trivial configuration requirements should probably
do.

I think most folks do actually use a .bbappend to fiddle FILESPATH in
order to get the configuration file picked up, but I'm fairly sure you
could do this with a bit of suitable magic in your distro config file
and avoid the .bbappend altogether.

> How would you feel about just including these two features in the
> defconfig then?

Not massively enthusiastic, simply because the logical conclusion of
that approach is that we end up with basically everything in busybox
turned on.  If we're going to go along that path then I think I would
prefer us to be up-front about it and just make the oe-core default
configuration for busybox be a "maximum features enabled" one. 

p.





      reply	other threads:[~2012-06-14 14:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <cover.1339650859.git.dvhart@linux.intel.com>
2012-06-14  5:19 ` [PATCH 2/3] busybox: Add setsid and cttyhack for tiny DISTRO_FEATURE Darren Hart
2012-06-14  5:21   ` Darren Hart
2012-06-14  7:05   ` Khem Raj
2012-06-15 22:22     ` Darren Hart
2012-06-15 22:57       ` Darren Hart
2012-06-14  9:41   ` Phil Blundell
2012-06-14 13:55     ` Darren Hart
2012-06-14 14:26       ` Phil Blundell [this message]

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=1339683970.14857.86.camel@phil-desktop \
    --to=philb@gnu.org \
    --cc=openembedded-core@lists.openembedded.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox