All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [RFC][PATCH v2] Alternative to busybox, using "proper tools"
@ 2010-03-22 16:08 Martyn Welch
  2010-03-22 16:33 ` Marcin Juszkiewicz
  2011-04-16 15:13 ` [oe,RFC,v2] " Ben Gamari
  0 siblings, 2 replies; 6+ messages in thread
From: Martyn Welch @ 2010-03-22 16:08 UTC (permalink / raw
  To: openembedded-devel

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?

 recipes/busybox/busybox-alt_1.bb   |   17 ++++++++
 recipes/tasks/task-proper-tools.bb |   76 ++++++++++++++++++++++++++++++++++++
 2 files changed, 92 insertions(+), 1 deletions(-)

diff --git a/recipes/busybox/busybox-alt_1.bb b/recipes/busybox/busybox-alt_1.bb
new file mode 100644
index 0000000..584cc46
--- /dev/null
+++ b/recipes/busybox/busybox-alt_1.bb
@@ -0,0 +1,17 @@
+DESCRIPTION = "Full versions of tools provided by busybox"
+
+ALLOW_EMPTY = "1"
+PACKAGES = "busybox"
+PACKAGE_ARCH = "all"
+
+PROVIDES = "busybox"
+DEFAULT_PREFERENCE = "1"
+
+RDEPENDS = "\
+		task-proper-tools \
+"
+
+DEPENDS = "\
+		task-proper-tools \
+"
+
diff --git a/recipes/tasks/task-proper-tools.bb b/recipes/tasks/task-proper-tools.bb
index e693c10..bac6b5b 100644
--- a/recipes/tasks/task-proper-tools.bb
+++ b/recipes/tasks/task-proper-tools.bb
@@ -4,33 +4,107 @@ PR = "r11"
 inherit task
 
 RDEPENDS_${PN} = "\
+		bash \
+		bc \
+		bind-utils \
+		binutils \
 		binutils-symlinks \
+		bzip2 \
+		console-tools \
 		coreutils \
 		cpio \
 		debianutils \
+		dhcp-client \
 		diffutils \
 		e2fsprogs \
+		e2fsprogs-fsck \
+		e2fsprogs-libs \
 		fbset \
 		findutils \
 		gawk \
 		grep \
 		gzip \
+		ifupdown-ubuntu \
+#		inetutils \
 		iproute2 \
 		iputils \
+		iputils-ping \
+		iputils-ping6 \
+		iputils-arping \
 		less \
+		minicom \
+		mktemp \
 		module-init-tools \
+		ncurses \
 		netcat \
 		net-tools \
 		patch \
 		procps \
 		psmisc \
+		openrdate \
+		realpath \
 		sed \
+		start-stop-daemon \
+		syslog-ng \
+		sysvinit \
 		tar \
+		tftp-hpa \
+		time \
 		tcptraceroute \
 		util-linux-ng \
+		unzip \
 		vim \
 		wget \
-        "
+"
+
+DEPENDS_${PN} = "\
+		bash \
+		bc \
+		bind \
+		binutils \
+		bzip2 \
+		console-tools \
+		coreutils \
+		cpio \
+		debianutils \
+		dhcp \
+		diffutils \
+		e2fsprogs \
+		fbset \
+		findutils \
+		gawk \
+		grep \
+		gzip \
+		ifupdown-ubuntu \
+#		inetutils \
+		iproute2 \
+		iputils \
+		less \
+		minicom \
+		mktemp \
+		module-init-tools \
+		ncurses \
+		netcat \
+		net-tools \
+		patch \
+		procps \
+		psmisc \
+		openrdate \
+		realpath \
+		sed \
+		start-stop-daemon \
+		syslog-ng \
+		sysvinit \
+		tar \
+		tftp-hpa \
+		time \
+		tcptraceroute \
+		util-linux-ng \
+		unzip \
+		vim \
+		wget \
+"
+
 #
 # binutils-symlinks provide ar in a way which conflict with busybox - #1465 for more info
 #


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



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [RFC][PATCH v2] Alternative to busybox, using "proper tools"
  2010-03-22 16:08 [RFC][PATCH v2] Alternative to busybox, using "proper tools" Martyn Welch
@ 2010-03-22 16:33 ` Marcin Juszkiewicz
  2010-03-23 10:32   ` Martyn Welch
  2011-04-16 15:13 ` [oe,RFC,v2] " Ben Gamari
  1 sibling, 1 reply; 6+ messages in thread
From: Marcin Juszkiewicz @ 2010-03-22 16:33 UTC (permalink / raw
  To: openembedded-devel

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

RDEPENDS = "task-proper-tools"

One line looks better. DEPENDS are not needed.

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

> +		bzip2 \
> +		console-tools \

> +		dhcp-client \

I added 'pump' - but thats minor.

> +		e2fsprogs-fsck \

In .dev this is in util-linux-ng now.

> +		e2fsprogs-libs \

What for this is?

> +		ifupdown-ubuntu \

Can we rename it to 'ifupdown'?

> +		iputils-ping \
> +		iputils-ping6 \
> +		iputils-arping \


> +		minicom \

Maybe 'picocom' would be nice too.

> +		mktemp \

> +		ncurses \

I added "ncurses-tools" - gives few utils more.

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

Regards, 
-- 
JID:      hrw@jabber.org
Website:  http://marcin.juszkiewicz.com.pl/
LinkedIn: http://www.linkedin.com/in/marcinjuszkiewicz





^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RFC][PATCH v2] Alternative to busybox, using "proper tools"
  2010-03-22 16:33 ` Marcin Juszkiewicz
@ 2010-03-23 10:32   ` Martyn Welch
  2010-11-06 14:00     ` Stefan Schmidt
  0 siblings, 1 reply; 6+ messages in thread
From: Martyn Welch @ 2010-03-23 10:32 UTC (permalink / raw
  To: openembedded-devel

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




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [RFC][PATCH v2] Alternative to busybox, using "proper tools"
  2010-03-23 10:32   ` Martyn Welch
@ 2010-11-06 14:00     ` Stefan Schmidt
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Schmidt @ 2010-11-06 14:00 UTC (permalink / raw
  To: openembedded-devel

Hello.

On Tue, 2010-03-23 at 10:32, Martyn Welch wrote:
> 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.

Martyn, did you sent out a newer version of this patch with the changes
integrated you acknowledged here? I can't find it in the repo only this outdated
version in patchwork.

regards
Stefan Schmidt



^ permalink raw reply	[flat|nested] 6+ messages in thread

* [oe,RFC,v2] Alternative to busybox, using "proper tools"
  2010-03-22 16:08 [RFC][PATCH v2] Alternative to busybox, using "proper tools" Martyn Welch
  2010-03-22 16:33 ` Marcin Juszkiewicz
@ 2011-04-16 15:13 ` Ben Gamari
  2011-04-18 12:31   ` Martyn Welch
  1 sibling, 1 reply; 6+ messages in thread
From: Ben Gamari @ 2011-04-16 15:13 UTC (permalink / raw
  To: openembedded-devel


Did anything ever happen with this[1] patch? If not, is there another
convenient way to use the proper tools in leiu of busybox? For larger
machines it's very nice to have a more civilized environment. Thanks,

 - Ben



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [oe,RFC,v2] Alternative to busybox, using "proper tools"
  2011-04-16 15:13 ` [oe,RFC,v2] " Ben Gamari
@ 2011-04-18 12:31   ` Martyn Welch
  0 siblings, 0 replies; 6+ messages in thread
From: Martyn Welch @ 2011-04-18 12:31 UTC (permalink / raw
  To: openembedded-devel

On 16/04/11 16:13, Ben Gamari wrote:
> 
> Did anything ever happen with this[1] patch? If not, is there another
> convenient way to use the proper tools in leiu of busybox? For larger
> machines it's very nice to have a more civilized environment. Thanks,
> 

I'm afraid I got pulled away from this before I could finish it.

The list of packages in task-proper-tools got updated by Marcin, I still have
a rather old tree containing the rest of it. Here's what I have (sorry my mail
client will have wrapped it a bit):

diff --git a/recipes/busybox/busybox-alt_1.bb b/recipes/busybox/busybox-alt_1.bb
new file mode 100644
index 0000000..811f073
--- /dev/null
+++ b/recipes/busybox/busybox-alt_1.bb
@@ -0,0 +1,19 @@
+DESCRIPTION = "Full versions of tools provided by busybox"
+
+PACKAGES = "busybox"
+PACKAGE_ARCH = "all"
+
+PROVIDES = "busybox"
+DEFAULT_PREFERENCE = "-1"
+
+RDEPENDS = "task-proper-tools"
+
+SRC_URI = "file://hwclock.sh"
+
+
+do_install () {
+       install -d ${D}${sysconfdir}/init.d/
+       install -m 0755 ${WORKDIR}/hwclock.sh ${D}${sysconfdir}/init.d/
+}
+
+FILES += "${sysconfdir}/init.d/hwclock.sh"
diff --git a/recipes/busybox/files/hwclock.sh b/recipes/busybox/files/hwclock.sh
index f9c9f9f..0a644fe 100644
--- a/recipes/busybox/files/hwclock.sh
+++ b/recipes/busybox/files/hwclock.sh
@@ -14,7 +14,7 @@

 . /etc/default/rcS

-[ "$UTC" = yes ] && UTC=-u || UTC=-l
+[ "$UTC" = yes ] && UTC=--utc || UTC=--localtime

 case "$1" in
         start)
diff --git a/recipes/tasks/task-proper-tools.bb
b/recipes/tasks/task-proper-tools.bb
index 40baf87..fd5e481 100644
--- a/recipes/tasks/task-proper-tools.bb
+++ b/recipes/tasks/task-proper-tools.bb
@@ -21,6 +21,7 @@ RDEPENDS_${PN} = "\
                grep \
                gzip \
                ifupdown \
+#              inetutils \
                iproute2 \
                iputils-arping \
                iputils-ping6 \
@@ -40,6 +41,7 @@ RDEPENDS_${PN} = "\
                psmisc \
                pump \
                realpath \
+               openrdate \
                sed \
                shadow \
                start-stop-daemon \
@@ -51,6 +53,7 @@ RDEPENDS_${PN} = "\
                time \
                unzip \
                util-linux-ng \
+               unzip \
                vim \
                wget \
         "


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



^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-04-18 14:19 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-22 16:08 [RFC][PATCH v2] Alternative to busybox, using "proper tools" Martyn Welch
2010-03-22 16:33 ` Marcin Juszkiewicz
2010-03-23 10:32   ` Martyn Welch
2010-11-06 14:00     ` Stefan Schmidt
2011-04-16 15:13 ` [oe,RFC,v2] " Ben Gamari
2011-04-18 12:31   ` Martyn Welch

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.