All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Looijmans <mike.looijmans@topic.nl>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices
Date: Thu, 18 Dec 2014 14:54:57 +0100	[thread overview]
Message-ID: <5492DCB1.4000609@topic.nl> (raw)
In-Reply-To: <CAJTo0LYk9szY3r5FxMHLF+MF-G41rAqg1vimhWXzqdo3+kqa2w@mail.gmail.com>

On 12/18/2014 02:30 PM, Burton, Ross wrote:
>
> On 18 December 2014 at 06:23, Mike Looijmans <mike.looijmans@topic.nl
> <mailto:mike.looijmans@topic.nl>> wrote:
>
>     Putting the script into its own package is no problem, but how does one
>     arrange this with mdev.conf?
>
>
> That's a fairly good argument for keeping it in the same package, and we can
> expect that users of mdev will be providing their own configuration file anyway.

Indeed. I was just implementing the split up, but it looks weird. You end up 
with a package "busybox-mdev" that contains nothing but an initscript and a 
config file (because the actual "mdev" binary is supplied by busybox), which 
refers to scripts that don't exist, and then an extra busybox-mdev-extraconf 
that supplies these scripts, but doesn't actually contain any configuration 
files in spite of its name. And then I had to find a way to get this package 
installed by default, which also isn't trivial.

So can we keep it al in busybox-mdev? It's a tiny config package anyway, and 
when providing private configs (which should rarely be needed now), it's easy 
enough to replace these in a bbappend.

M.


Met vriendelijke groet / kind regards,

Mike Looijmans
System Expert


TOPIC Embedded Systems
Eindhovenseweg 32-C, NL-5683 KH Best
Postbus 440, NL-5680 AK Best
Telefoon: (+31) (0) 499 33 69 79
Telefax:  (+31) (0) 499 33 69 70
E-mail: mike.looijmans@topic.nl
Website: www.topic.nl

Please consider the environment before printing this e-mail

Topic zoekt gedreven (embedded) software specialisten!
http://topic.nl/vacatures/topic-zoekt-software-engineers/



  reply	other threads:[~2014-12-18 13:55 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-04 10:27 [PATCH 0/2] Busybox mdev improvements Mike Looijmans
2014-12-04 10:27 ` [PATCH 1/2] busybox-mdev: Install missing find-touchscreen.sh Mike Looijmans
2014-12-04 10:27 ` [PATCH 2/2] busybox/find-touchscreen.sh: Simplify script and recognize USB devices Mike Looijmans
2014-12-04 20:47 ` [PATCH 0/2] Busybox mdev improvements Otavio Salvador
2014-12-05  6:18   ` Mike Looijmans
2014-12-05 11:27     ` Otavio Salvador
2014-12-05 12:57       ` Mike Looijmans
2014-12-05 14:13         ` Otavio Salvador
2014-12-05  9:58 ` [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices Mike Looijmans
2014-12-05  9:58   ` [PATCH 2/2] busybox-mdev: Add hotplug kernel module support to mdev.conf Mike Looijmans
2014-12-17 22:01   ` [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices Burton, Ross
2014-12-18 13:14     ` Mike Looijmans
2014-12-17 22:03   ` Burton, Ross
2014-12-17 22:05     ` Burton, Ross
2014-12-17 22:08     ` Gary Thomas
2014-12-18  6:31       ` Mike Looijmans
2014-12-18 12:55         ` Otavio Salvador
2014-12-18 14:38           ` Mike Looijmans
2014-12-18  6:23     ` Mike Looijmans
2014-12-18 13:30       ` Burton, Ross
2014-12-18 13:54         ` Mike Looijmans [this message]
2014-12-18 14:17 ` [PATCH v2 0/4] Busybox mdev improvements Mike Looijmans
2014-12-18 14:17   ` [PATCH v2 1/4] busybox-mdev: Install missing find-touchscreen.sh Mike Looijmans
2014-12-18 14:43     ` Otavio Salvador
2014-12-18 14:17   ` [PATCH v2 2/4] busybox/find-touchscreen.sh: Simplify script and recognize USB devices Mike Looijmans
2014-12-18 14:17   ` [PATCH v2 4/4] busybox-mdev: Support automatic mounting of block devices Mike Looijmans
2015-01-01  2:49     ` Khem Raj
2015-01-01  9:07       ` Mike Looijmans
2015-01-01 13:08         ` Peter A. Bigot
2015-01-01 19:21           ` Mike Looijmans
2014-12-18 14:19   ` [PATCH v2 3/4] busybox-mdev: Add hotplug kernel module support to mdev.conf Mike Looijmans
2014-12-18 14:44     ` 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=5492DCB1.4000609@topic.nl \
    --to=mike.looijmans@topic.nl \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ross.burton@intel.com \
    /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.