All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yegor Yefremov <yegor_sub1@visionsystems.de>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 2/2] udev: bump to 181 and other fixes
Date: Thu, 09 Feb 2012 08:26:53 +0100	[thread overview]
Message-ID: <4F33753D.7030002@visionsystems.de> (raw)
In-Reply-To: <87k43x3ufp.fsf@macbook.be.48ers.dk>

Am 08.02.2012 22:07, schrieb Peter Korsgaard:
>>>>>> "yegorslists" == yegorslists  <yegorslists@googlemail.com> writes:
> 
>  yegorslists> From: Yegor Yefremov <yegorslists@googlemail.com>
> 
>  yegorslists> Changes:
>  yegorslists> - Linux kernel requirement: 2.6.34 and above (devtmpfs is mandatory)
>  yegorslists> (this also applies to the chosen toolchain, because of Kernel headers)
>  yegorslists> - optional dependency on libusb and usbutils removed
>  yegorslists> - added dependency on kmod
>  yegorslists> - added dependency on util-linux for libblkid
>  yegorslists> - install in /lib/udev instead of /usr/libexec/udev
>  yegorslists> - udevd moved to /lib/udev
>  yegorslists> - fixed path to pci.ids and usb.ids
>  yegorslists> - persistent rules generator is disabled by default, so option
>  yegorslists> is introduced to enable this option if desired
> 
> Thanks, committed with minor tweaks (see below).
> 
> 
>  yegorslists> +++ b/package/udev/Config.in
>  yegorslists> @@ -1,31 +1,39 @@
>  yegorslists>  config BR2_PACKAGE_UDEV
>  yegorslists>  	bool "udev"
>  yegorslists>  	depends on BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
>  yegorslists> +	depends on BR2_LARGEFILE # util-linux
>  yegorslists> +	depends on BR2_USE_WCHAR # util-linux
>  yegorslists> +	select BR2_PACKAGE_UTIL_LINUX
>  yegorslists> +	select BR2_PACKAGE_UTIL_LINUX_LIBBLKID
>  yegorslists> +	select BR2_PACKAGE_KMOD
>  yegorslists>  	help
>  yegorslists>  	  Userspace device daemon.
>  
>  yegorslists> +	  udev requires a Linux kernel >= 2.6.34: it relies on devtmpfs.
>  yegorslists> +
> 
> The devtmpfs dependency is not really an issue as we already depend on
> it for dynamic /dev handling.

ACK
 
>   yegorslists>  comment "udev requires /dev mgmnt set to udev under System configuration"
>  yegorslists>  	depends on !BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV
>  yegorslists> +
>  yegorslists> +comment "udev requires a toolchain with LARGEFILE + WCHAR support"
>  yegorslists> +    depends on !(BR2_LARGEFILE && BR2_USE_WCHAR)
>  
> 
> This doesn't really make sense to show here, so I moved it
> target/generic where the user selects the /dev handling method. This
> line would only be shown in situations where he couldn't select udev
> anyway (as he hadn't selected BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_UDEV).

ACK
 
>  yegorslists> +++ b/package/udev/udev.mk
>  yegorslists> @@ -3,7 +3,7 @@
>  yegorslists>  # udev
>  yegorslists>  #
>  yegorslists>  #############################################################
>  yegorslists> -UDEV_VERSION = 173
>  yegorslists> +UDEV_VERSION = 181
>  yegorslists>  UDEV_SOURCE = udev-$(UDEV_VERSION).tar.bz2
>  yegorslists>  UDEV_SITE = $(BR2_KERNEL_MIRROR)/linux/utils/kernel/hotplug/
>  yegorslists>  UDEV_INSTALL_STAGING = YES
>  yegorslists> @@ -11,20 +11,24 @@ UDEV_INSTALL_STAGING = YES
>  yegorslists>  UDEV_CONF_OPT =			\
>  yegorslists>  	--sbindir=/sbin		\
>  yegorslists>  	--with-rootlibdir=/lib	\
>  yegorslists> -	--libexecdir=/lib/udev	\
>  yegorslists> +	--libexecdir=/lib	\
>  yegorslists> +	--with-usb-ids-path=/usr/share/hwdata/usb.ids	\
>  yegorslists> +	--with-pci-ids-path=/usr/share/hwdata/pci.ids	\
>  yegorslists> +	--with-firmware-path=/lib/firmware		\
>  yegorslists>  	--disable-introspection
> 
> udevd now needs to link with librt as it calls mq_getattr() (which isn't
> automatically done on uClibc), so I added UDEV_CONF_ENV += LIBS='-lt'

O.K. Thanks for reviewing and committing. Due to kernel version dependency we can almost retire these toolchains in case of udev:

Sourcery CodeBench ARM 2010q1
Sourcery CodeBench ARM 2009q3
Sourcery CodeBench ARM 2009q1 

Yegor

  reply	other threads:[~2012-02-09  7:26 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-08  9:50 [Buildroot] [PATCH v2 1/2] kmod: bump to 5 yegorslists at googlemail.com
2012-02-08  9:50 ` [Buildroot] [PATCH v2 2/2] udev: bump to 181 and other fixes yegorslists at googlemail.com
2012-02-08 21:07   ` Peter Korsgaard
2012-02-09  7:26     ` Yegor Yefremov [this message]
2012-02-09  9:02       ` Thomas Petazzoni
2012-02-09  9:08         ` Yegor Yefremov
2012-02-09 11:50           ` Yegor Yefremov
2012-02-08 20:12 ` [Buildroot] [PATCH v2 1/2] kmod: bump to 5 Peter Korsgaard

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=4F33753D.7030002@visionsystems.de \
    --to=yegor_sub1@visionsystems.de \
    --cc=buildroot@busybox.net \
    /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.