From: Martin Jansa <martin.jansa@gmail.com>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 5/5] busybox.inc: Install hwclock init script only if rtc is present in MACHINE_FEATURES
Date: Wed, 11 Jul 2012 11:11:37 +0200 [thread overview]
Message-ID: <20120711091137.GA4190@jama.jama.net> (raw)
In-Reply-To: <1341960238.2786.103.camel@x121e.pbcl.net>
[-- Attachment #1: Type: text/plain, Size: 1008 bytes --]
On Tue, Jul 10, 2012 at 11:43:58PM +0100, Phil Blundell wrote:
> On Wed, 2012-07-11 at 01:15 +0300, Andrei Gherzan wrote:
> > And to move a little further, busybox should be configured with
> > CONFIG_HWCLOCK=y only if it makes sense for that MACHINE. In my
> > opinion this is machine specific.
>
> That's a DISTRO decision. I suspect most that have binary feeds would
> prefer to have a single busybox binary per architecture, and accept the
> few wasted bytes on machines without RTC, than to make it MACHINE
> specific and end up building it multiple times.
Agreed.
It would be better to package initscript + hwclock to separate packages
and then pull it to image only for machines with RTC.. but not making
whole busybox machine specific.
Another advantage of this would be option to use busybox-hwclock as utility
and hwclock-systemd instead of busybox-hwclock-init for images which are
using systemd.
Cheers,
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
next prev parent reply other threads:[~2012-07-11 9:22 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1341932186.git.andrei@gherzan.ro>
2012-07-10 14:59 ` [PATCH 1/5] utils.py: Modify and rename distro_features_backfill python function Andrei Gherzan
2012-07-10 14:59 ` [PATCH 2/5] bitbake.conf: Use the new form of features_backfill function Andrei Gherzan
2012-07-10 14:59 ` [PATCH 3/5] bitbake.conf: Add MACHINE_FEATURES_BACKFILL and append to MACHINE_FEATURES Andrei Gherzan
2012-07-10 14:59 ` [PATCH 4/5] bitbake.conf: Add rtc to MACHINE_FEATURES_BACKFILL Andrei Gherzan
2012-07-10 14:59 ` [PATCH 5/5] busybox.inc: Install hwclock init script only if rtc is present in MACHINE_FEATURES Andrei Gherzan
2012-07-10 16:02 ` Richard Purdie
2012-07-10 20:53 ` Andrei Gherzan
2012-07-10 22:15 ` Andrei Gherzan
2012-07-10 22:43 ` Phil Blundell
2012-07-11 9:11 ` Martin Jansa [this message]
2012-07-25 18:13 ` Andrei Gherzan
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=20120711091137.GA4190@jama.jama.net \
--to=martin.jansa@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 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.