[Buildroot] [patch] udev broken, update and fix

[Buildroot] [patch] udev broken, update and fix

[Brad House]

/etc/udev/rules.d/ is empty with the current udev.  It should be
populated by default with one of the examples provided with udev.
Also, /dev/pts should be mounted by default from the udev (S10udev)
start script, otherwise 'dropbear' won't allow logins as it requires
/dev/pts...  (and this shouldn't be done automatically in fstab since
udev creates a tmpfs which would mount over it).

Also, udev_root isn't configured by default in /etc/udev/udev.conf as
the start script requires.

I've attached a patch which fixes those and goes ahead and bumps the
version to 114 (from 101).  Also, you can remove the
'udev-install.patch', as I've modified the makefile to do the same
thing...

-Brad
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: udev-114.patch
Url: http://busybox.net/lists/buildroot/attachments/20070806/bbc9d7b2/attachment.diff 

[Buildroot] [patch] udev broken, update and fix

[Ulf Samuelsson]

----- Original Message ----- 
From: "Brad House" <brad@mainstreetsoftworks.com>
To: <buildroot@uclibc.org>
Sent: Tuesday, August 07, 2007 2:35 AM
Subject: [Buildroot] [patch] udev broken, update and fix


> /etc/udev/rules.d/ is empty with the current udev.  It should be
> populated by default with one of the examples provided with udev.
> Also, /dev/pts should be mounted by default from the udev (S10udev)
> start script, otherwise 'dropbear' won't allow logins as it requires
> /dev/pts...  (and this shouldn't be done automatically in fstab since
> udev creates a tmpfs which would mount over it).
> 
> Also, udev_root isn't configured by default in /etc/udev/udev.conf as
> the start script requires.
> 
> I've attached a patch which fixes those and goes ahead and bumps the
> version to 114 (from 101).  Also, you can remove the
> 'udev-install.patch', as I've modified the makefile to do the same
> thing...
> 
> -Brad
>


> --- package/udev/udev.mk (revision 19415)
> +++ package/udev/udev.mk (working copy)
> @@ -46,6 +46,9 @@
>  USE_LOG=false USE_SELINUX=false \
>  udevdir=$(UDEV_ROOT) -C $(UDEV_DIR) install
>  $(INSTALL) -m 0755 package/udev/S10udev $(TARGET_DIR)/etc/init.d
> + $(INSTALL) -m 0644 $(UDEV_DIR)/etc/udev/frugalware/* $(TARGET_DIR)/etc/udev/rules.d
> + echo 'udev_root=/dev' >> $(TARGET_DIR)/etc/udev/udev.conf

If you build udev, and then delete the source directory, and build udev again
you will get multiple lines "udev_root=/dev" in the udev.conf.

It would be better to check if it is there and add it, if it is not.

Best Regards
Ulf Samuelsson

[Buildroot] [patch] udev broken, update and fix

[Brad House]

> If you build udev, and then delete the source directory, and build udev again
> you will get multiple lines "udev_root=/dev" in the udev.conf.
> 
> It would be better to check if it is there and add it, if it is not.

Yep, you're right. I've attached a patch to fix that.
I also fixed the udev-clean/dirclean issue if you didn't have
volume_id and scsi_id selected.

-Brad
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: udev-114-2.patch
Url: http://busybox.net/lists/buildroot/attachments/20070807/4f3c8ee9/attachment.diff 

[Buildroot] [patch] udev broken, update and fix

[Brad House]

Any word on getting this committed?

Thanks.
-Brad

Brad House wrote:
>> If you build udev, and then delete the source directory, and build
>> udev again
>> you will get multiple lines "udev_root=/dev" in the udev.conf.
>>
>> It would be better to check if it is there and add it, if it is not.
> 
> Yep, you're right. I've attached a patch to fix that.
> I also fixed the udev-clean/dirclean issue if you didn't have
> volume_id and scsi_id selected.
> 
> -Brad
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> buildroot mailing list
> buildroot at uclibc.org
> http://busybox.net/mailman/listinfo/buildroot