Bug#676001: Processed: reassign 676001 to busybox

[Michael Tokarev <mjt@tls.msk.ru>]

severity 676001 wishlist
tags 676001 + wontfix
thanks

As shown, historically it was util-linux which
implemented functionality previously found in busybox,
not the other way around, and as noted several times,
it is none of switch_root business to deal with other
filesystems (it is more a misfeature in util-linux
implementation).

So marking this bug as wontfix, and appropriately
lowering severity.

Thanks,

/mjt

On 08.06.2012 21:36, Michael Tokarev wrote:
> On 08.06.2012 15:22, maximilian attems wrote:
>> On Fri, Jun 08, 2012 at 03:10:42PM +0400, Michael Tokarev wrote:
>>> On 08.06.2012 14:52, maximilian attems wrote:
>>>> dude care to have a bit of patience before reassigning back,
>>>> that be really nice.
>>>
>>> I gave a few days, maybe it was too few, I dunno.
>>>
>>>> On Tue, Jun 05, 2012 at 08:45:59AM +0400, Michael Tokarev wrote:
>>> []
>>>>> I disagree it is a busybox problem, and don't think it is a
>>>>> switch_root business (be it from busybox or from util-linux).
>>>>
>>>> switch_root in util-linux does it.
>>>
>>> Yes, but it is still none of its business.
>>
>> that is your personal opinon and shown to be wrong. (:
> 
>>>> If you name a command switch-root and not run-init, you'd have
>>>> to take care to emmulate what the original command does.
>>>>
>>>> In this case it is util-linux is clearly predating busybox and thus
>>>> busybox is buggy not fully implementing the command.
>>>
>>> Almost no of busybox commands implements fully the corresponding
>>> "big brother" behavour.
>>
>> well in the cases where it is needed and as busybox doesn't do it is a
>> bug.
> 
> Switch_root utility in util-linux appeared _after_ the same
> utility appeared in busybox.  In util-linux it was implemented
> in 2009, http://git.kernel.org/?p=utils/util-linux/util-linux.git;a=commit;h=711ea7307d54caa74aa89fc7e8614236e3721f1c
> This command there were taken from dracut apparently.
> 
> In there, it has been written in 2002 (I guess), has been
> called "switchroot" (no underscore), and it does _umounting_
> of /dev, /proc, /sys, as can be seen at
> http://pjones.fedorapeople.org/mkstart/usr/lib/mkstart/switchroot.c
> 
> In busybox it has been implemented as switch_root in 2005, in this commit:
> http://git.busybox.net/busybox/commit/util-linux/switch_root.c?id=0f34a821ab99e4936c7aa4974f58784442172211
> which, obviously, pre-dates the same command in util-linux.
> 
> There in busybox, it is named switch_root right from the
> beginning, and were made after run-init behavour (instead
> of the switchroot from dracut).
> 
> So we can conclude these implementations (in util-linux
> and busybox) are independent and not follows or modelled
> from one another.
> 
> In dractu, were it umounted /dev /proc /sys instead of
> moving these, I guess it was modelled after nash which
> was a bad example of shell to use in initrd, it didn't
> have most standard shell constructs so it weren't easy
> to program in it.  But I can only guess.
> 
> []
>> run-init doesn't move mount things, you could rename switch_root
>> to run-init, then it would be correct.
>> You confuse things, switch_root is the new command name by util-linux
>> and it does a certain number of things. It doesn't matter, if
>> you personally agree with them or not.
>>
>> Ah and please stop evading into the init script.
> 
> I'm not.
> 
> Thanks,
> 
> /mjt