Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 14/16] udev: run depmod if modules.dep doesn't exist
Date: Wed, 04 Jan 2012 17:15:36 +0000	[thread overview]
Message-ID: <1325697336.20759.40.camel@ted> (raw)
In-Reply-To: <a4ffb960e298bf423adcda013c5b58c1db2f6b48.1325101675.git.otavio@ossystems.com.br>

On Wed, 2011-12-28 at 19:55 +0000, Otavio Salvador wrote:
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>  meta/recipes-core/udev/udev/init   |    5 +++++
>  meta/recipes-core/udev/udev_164.bb |    2 +-
>  2 files changed, 6 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
> index 322e7e7..8d89d51 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -52,6 +52,11 @@ if [ "$DEVCACHE" != "" ]; then
>  	fi
>  fi
>  
> +if [ -x /sbin/depmod ] && [ ! -e "/lib/modules/$(uname -r)"/modules.dep ]; then
> +	mkdir -p /lib/modules/$(uname -r)
> +	depmod -ae
> +fi
> +
>  # make_extra_nodes
>  kill_udevd > "/dev/null" 2>&1

Why does udev need to do this instead of the kernel module postinstalls?
This strikes me as the wrong thing to be doing here...

Cheers,

Richard




  reply	other threads:[~2012-01-04 17:23 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-28 19:55 [PATCH 00/16] Misc Qt4 and udev changes pending from O.S. Systems Otavio Salvador
2011-12-28 19:55 ` [PATCH 01/16] qt4-graphics-system: add Otavio Salvador
2012-01-02 18:23   ` Saul Wold
2012-01-02 18:38     ` Martin Jansa
2012-01-04  0:49       ` Otavio Salvador
2011-12-28 19:55 ` [PATCH 02/16] qt4e.bbclass: add QT_BASE_NAME for use in recipes Otavio Salvador
2011-12-28 19:55 ` [PATCH 03/16] qt4x11.bbclass: " Otavio Salvador
2011-12-28 19:55 ` [PATCH 04/16] qt4-embedded.inc: use QT_BASE_NAME from qt4e.bbclass Otavio Salvador
2011-12-28 19:55 ` [PATCH 05/16] qt4-x11-free.inc: use QT_BASE_NAME from qt4x11.bbclass Otavio Salvador
2011-12-28 19:55 ` [PATCH 06/16] udev: drop 145 version Otavio Salvador
2011-12-28 19:55 ` [PATCH 07/16] udev-extraconf: move mount.blacklist to udev-extraconf dir Otavio Salvador
2011-12-28 19:55 ` [PATCH 08/16] udev: reorganize files of 164 version Otavio Salvador
2011-12-28 19:55 ` [PATCH 09/16] udev: use tabs for init script (no code changes) Otavio Salvador
2011-12-28 19:55 ` [PATCH 10/16] udev: skip mounting /dev on tmpfs if it is on devtmpfs Otavio Salvador
2011-12-28 19:55 ` [PATCH 11/16] udev: ensure /dev/pts and /dev/shm does exists Otavio Salvador
2012-01-04 17:12   ` Richard Purdie
2012-01-05 16:59     ` Otavio Salvador
2011-12-28 19:55 ` [PATCH 12/16] udev: improve udev-cache robustness Otavio Salvador
2012-01-04 17:14   ` Richard Purdie
2012-01-05 17:06     ` Otavio Salvador
2011-12-28 19:55 ` [PATCH 13/16] udev: use 'echo' instead of 'echo -n' in init script Otavio Salvador
2011-12-28 19:55 ` [PATCH 14/16] udev: run depmod if modules.dep doesn't exist Otavio Salvador
2012-01-04 17:15   ` Richard Purdie [this message]
2012-01-04 18:51     ` Koen Kooi
2012-01-04 23:04       ` Richard Purdie
2011-12-28 19:55 ` [PATCH 15/16] udev: split utilities onto udev-utils Otavio Salvador
2011-12-28 19:55 ` [PATCH 16/16] udev: remove commented code from init script (no code changes) Otavio Salvador
2012-01-04  0:24 ` [PATCH 00/16] Misc Qt4 and udev changes pending from O.S. Systems Saul Wold
2012-01-05 20:25   ` Saul Wold

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=1325697336.20759.40.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --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