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

[Natanael Copa]

On Fri, 2008-05-02 at 14:47 +0200, Denys Vlasenko wrote:
> 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)?

yup. works.

i switched back to 1.9.2 last minute because i need to make a release of
alpine (my busybox/uclibc based distro) today. But that was related to
other things.

> > 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.

fair enough.

Thanks!

> --
> vda