Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Andre McCurdy <armccurdy@gmail.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/8] busybox: merge login-utilities.cfg into defconfig
Date: Wed, 29 Jul 2015 23:08:17 +0100	[thread overview]
Message-ID: <1438207697.11208.78.camel@linuxfoundation.org> (raw)
In-Reply-To: <1438125665-14966-2-git-send-email-armccurdy@gmail.com>

On Tue, 2015-07-28 at 16:20 -0700, Andre McCurdy wrote:
> The login-utilities.cfg busybox config fragment was added during the
> transition of the default login manager from tinylogin to busybox [1].
> The tinylogin recipe was removed from oe-core prior to the 1.5 (dora)
> release [2].
> 
> Merging the config fragment into defconfig simplifies the process of
> keeping the oe-core busybox defconfig aligned with new releases of
> busybox.
> 
>   [1] https://bugzilla.yoctoproject.org/show_bug.cgi?id=4207
>   [2] http://git.openembedded.org/openembedded-core/commit/?id=2762ff976a3473be4259889029e048ab8b3be5ab
> 
> Signed-off-by: Andre McCurdy <armccurdy@gmail.com>

I'm kind of torn on this. The original intent was to split up the
busybox config into more granular fragments so that rather that having
to provide a defconfig for it, you could just select the components you
wanted and your custom config on top of that.

If we do go down that route, it does imply a different kind of
maintenance of the config fragments is needed as we move forward. In
some ways I'd hope the fragment tools could help with that.

Obviously any more split up hasn't happened as yet and I know that
personally, given my workload I'm unlikely to find time soon :(. I'd be
interested to see what others think?

Cheers,

Richard






  reply	other threads:[~2015-07-29 22:08 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-28 23:20 [PATCH 0/8 v2] refresh busybox defconfig Andre McCurdy
2015-07-28 23:20 ` [PATCH 1/8] busybox: merge login-utilities.cfg into defconfig Andre McCurdy
2015-07-29 22:08   ` Richard Purdie [this message]
2015-07-30 11:45     ` Otavio Salvador
2015-07-28 23:20 ` [PATCH 2/8] busybox: re-order defconfig to align with busybox 1.23.2 Andre McCurdy
2015-07-28 23:21 ` [PATCH 3/8] busybox: remove CONFIG_FEATURE_VI_OPTIMIZE_CURSOR from defconfig Andre McCurdy
2015-07-28 23:21 ` [PATCH 4/8] busybox: refresh defconfig for busybox v1.23.2 Andre McCurdy
2015-07-28 23:21 ` [PATCH 5/8] busybox: disable sha3sum in defconfig Andre McCurdy
2015-07-28 23:21 ` [PATCH 6/8] busybox: disable fatattr " Andre McCurdy
2015-07-28 23:21 ` [PATCH 7/8] busybox: disable fstrim " Andre McCurdy
2015-07-29 14:22   ` Burton, Ross
2015-07-29 16:28     ` Otavio Salvador
2015-07-29 18:39       ` Andre McCurdy
2015-07-28 23:21 ` [PATCH 8/8] busybox: disable volumeid support for exFAT, F2FS and NILFS Andre McCurdy
2015-07-29 16:33   ` Otavio Salvador
2015-07-29 18:49     ` Andre McCurdy
2015-07-29 18:50       ` Otavio Salvador

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=1438207697.11208.78.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=armccurdy@gmail.com \
    --cc=openembedded-core@lists.openembedded.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