[OE-core] [PATCH] busybox: fix conflict with runlevel applet

[Khem Raj]

On Mon, Oct 8, 2018 at 8:39 AM Burton, Ross <ross.burton at intel.com> wrote:
>
> On Thu, 4 Oct 2018 at 12:28, Martin Jansa <martin.jansa at gmail.com> wrote:
>>
>> On Thu, Oct 04, 2018 at 12:59:37PM +0200, Andrej Valek wrote:
>> > This init feature was introduced via
>> > http://lists.openembedded.org/pipermail/openembedded-core/2016-January/116109.html
>> > which is from 2016. Busybox has this runlevel applet available from 2005.
>> >
>> > I think, Raj was trying to make this system really tiny. So, I am not
>> > sure, what would be the impact, if we dropped this runlevel script.
>>
>> OK, fair enough. I don't use busybox init anyway.
>>
>> I was just curious, because there is also runlevel from upstart which we were
>> using (and I have u-a for runlevel in busybox.bbappend because of that).
>> So I was surprised that there is actually 3rd implementation as OE
>> specific shell script.
>
>
> Khem, do you have an opinion?  I suspect that shell script including comment is longer than the compiled size of the runlevel applet in busybox...  We could simplify by just having the applet and removing that runlevel stub.

we should enable runlevel unconditionally when selecting busybox init
and delete the file that I added for runlevel, that way we will be
inline with busybox for this applet.

>
> Ross