Re: [RFC][PATCH v2] Alternative to busybox, using "proper tools"

[Martyn Welch <martyn.welch@ge.com>]

Marcin Juszkiewicz wrote:
> Dnia poniedziałek, 22 marca 2010 o 17:08:20 Martyn Welch napisał(a):
>
>   
>> This patch provides a mechanism to replace busybox by defining a busybox
>> package which actually just requires a set of other packages.
>>
>>     To get it to be used rather than the standard busybox, define:
>>
>>     PREFERRED_PROVIDER_busybox = "busybox-alt"
>> ---
>>
>>  Marcin: Something like this you mean?
>>     
>
> Yes, few notes:
>  
>   

No prob.

>> diff --git a/recipes/busybox/busybox-alt_1.bb
>> b/recipes/busybox/busybox-alt_1.bb new file mode 100644
>>     
>
> busybox-alt.bb is fine too - PV will be 1.0 then
>
>   
>> +++ b/recipes/busybox/busybox-alt_1.bb
>>     
>
>   
>> +ALLOW_EMPTY = "1"
>> +PACKAGES = "busybox"
>> +PACKAGE_ARCH = "all"
>> +
>> +PROVIDES = "busybox"
>> +DEFAULT_PREFERENCE = "1"
>>     
>
> "-1" is more proper - we do not want people to get busybox-alt instead of 
> busybox by mistake.
>
>   

Agreed - definitely left over from prototyping.

> RDEPENDS = "task-proper-tools"
>
> One line looks better. DEPENDS are not needed.
>
>   

Ok.

I understand that RDEPENDS are runtime dependancies. Am I right to
surmise that DEPENDS are configure/compile/packaging time dependences?

>> +++ b/recipes/tasks/task-proper-tools.bb
>> @@ -4,33 +4,107 @@ PR = "r11"
>>     
>
> Bump PR.
>
>   
>>  RDEPENDS_${PN} = "\
>> +		bash \
>> +		bc \
>> +		bind-utils \
>>     
>
>   
>> +		binutils \
>>     
>
> Not needed - 'binutils-symlinks' depends on it.
>
>   

Ok.

>> +		bzip2 \
>> +		console-tools \
>>     
>
>   
>> +		dhcp-client \
>>     
>
> I added 'pump' - but thats minor.
>
>   

As long as it works, I'm not fussed either way.

>> +		e2fsprogs-fsck \
>>     
>
> In .dev this is in util-linux-ng now.
>   

So I assume I can remove e2fsprogs from this list?

>   
>> +		e2fsprogs-libs \
>>     
>
> What for this is?
>
>   

Shared libraries for e2fsprogs. I'll remove this, I assume this is
either automatically added or not needed.

>> +		ifupdown-ubuntu \
>>     
>
> Can we rename it to 'ifupdown'?
>
>   

We added a recipe to our overlay which used the Ubuntu version of
ifupdown rather than using the Debian version already in OE. I'm afraid
I didn't create the original recipe for our overlay (one of my
colleagues did), my understanding is as follows:

The ifupdown tool does not have an active upstream maintainer, as a
result (at least Debian and Ubuntu) have differently patched copies of
version 0.6.8. The functionality provided by the Ubuntu version provided
a better drop in replacement for the required functionality than the
Debian version and hence we have used this one. As the version numbers
were the same, I created a "ifupdown-ubuntu" recipe for this package. To
ensure it was used here instead of the Debian version, I put
"ifupdown-ubuntu" in the RDEPENDS. Whilst we were prototyping this we
had a "PREFERRED_PROVIDER_ifupdown" entry in our local.conf to force
it's use.

If you think this is unwarranted, we can try using the Debian version
instead.

>> +		iputils-ping \
>> +		iputils-ping6 \
>> +		iputils-arping \
>>     
>
>
>   
>> +		minicom \
>>     
>
> Maybe 'picocom' would be nice too.
>
>   

Never heard of it - up to you.

>> +		mktemp \
>>     
>
>   
>> +		ncurses \
>>     
>
> I added "ncurses-tools" - gives few utils more.
>
>   

Ok.

>> +		openrdate \
>> +		realpath \
>>     
>
>
>   
>> +		start-stop-daemon \
>>     
>
> Nice - I added 'dpkg' instead but it was overkill.
>
>   
>> +		syslog-ng \
>> +		sysvinit \
>> +		tftp-hpa \
>> +		time \
>>     
>
> I added 'shadow' to the list.
>
>
>   
>> +DEPENDS_${PN} = "\
>>     
>
> DEPENDS are not needed.
>   

Martyn


-- 
Martyn Welch (Principal Software Engineer)   |   Registered in England and
GE Intelligent Platforms                     |   Wales (3828642) at 100
T +44(0)127322748                            |   Barbirolli Square, Manchester,
E martyn.welch@ge.com                        |   M2 3AB  VAT:GB 927559189