the busybox-1.10.1-mdev.patch in fixes-1.10.1 break things

[Denys Vlasenko]

On Friday 02 May 2008 14:12, Natanael Copa wrote:
> Attatched is a patch that fixes a bug introduced in the
> fixes-1.10.1/busybox-1.10.1-mdev.patch that make mdev -s fail on
> existing mdev.conf files.
> 
> It should be applied after the busybox-1.10.1-mdev.patch.
> 
> Below is a diff with --ignore-space-change option to show exactly what
> this patch does.
> 
> Please put the attatched diff in the fixes-1.10.1 dir.

Ok... can you test attached patch (atop busybox-1.10.1-mdev.patch)?

> btw... why was the mdev patch added there in firstplace? I thought fixes
> should only contain fixes and not new features.

mdev had a serious bug - it was creating wrong type of device nodes.
Also it didn't stop parsing config after first match.
Other smaller problems too.

I decided that it's too buggy to let people wait for 1.11.
--
vda
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 4.patch
Type: text/x-diff
Size: 771 bytes
Desc: not available
Url : http://busybox.net/lists/busybox/attachments/20080502/132113d8/attachment.patch