mkinitrd unification across distributions
 help / color / mirror / Atom feed
* [PATCH] busybox: fix installation from /sbin/busybox
@ 2012-09-25 22:42 Daniel Drake
       [not found] ` <20120925224228.25BC8FAABE-2+9YHz4BXxlLDiiyqF6/jw@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Daniel Drake @ 2012-09-25 22:42 UTC (permalink / raw)
  To: initramfs-u79uwXL29TY76Z2rM5mHXA, harald-H+wXaHxf7aLQT0dZR+AlfA

"inst busybox" no longer seems to work when busybox is at
/sbin/busybox. Reproduced on Fedora 18 (dracut-023-39.git20120910).

Use type -P to find the full path to busybox to solve this problem.
---
 modules.d/05busybox/module-setup.sh | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/modules.d/05busybox/module-setup.sh b/modules.d/05busybox/module-setup.sh
index 2e9ead8..e0ef178 100755
--- a/modules.d/05busybox/module-setup.sh
+++ b/modules.d/05busybox/module-setup.sh
@@ -13,9 +13,10 @@ depends() {
 }
 
 install() {
-    local _i _progs _path
-    inst busybox /usr/bin/busybox
-    for _i in `/sbin/busybox | sed -ne '1,/Currently/!{s/,//g; s/busybox//g; p}'`
+    local _i _progs _path _busybox
+    _busybox=$(type -P busybox)
+    inst $_busybox /usr/bin/busybox
+    for _i in `$_busybox | sed -ne '1,/Currently/!{s/,//g; s/busybox//g; p}'`
     do
         _progs="$_progs $_i"
     done
-- 
1.7.11.4

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

* Re: [PATCH] busybox: fix installation from /sbin/busybox
       [not found] ` <20120925224228.25BC8FAABE-2+9YHz4BXxlLDiiyqF6/jw@public.gmane.org>
@ 2012-09-26  9:50   ` Harald Hoyer
  0 siblings, 0 replies; 2+ messages in thread
From: Harald Hoyer @ 2012-09-26  9:50 UTC (permalink / raw)
  To: Daniel Drake; +Cc: initramfs-u79uwXL29TY76Z2rM5mHXA

Am 26.09.2012 00:42, schrieb Daniel Drake:
> "inst busybox" no longer seems to work when busybox is at
> /sbin/busybox. Reproduced on Fedora 18 (dracut-023-39.git20120910).
> 
> Use type -P to find the full path to busybox to solve this problem.
> ---
>  modules.d/05busybox/module-setup.sh | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/modules.d/05busybox/module-setup.sh b/modules.d/05busybox/module-setup.sh
> index 2e9ead8..e0ef178 100755
> --- a/modules.d/05busybox/module-setup.sh
> +++ b/modules.d/05busybox/module-setup.sh
> @@ -13,9 +13,10 @@ depends() {
>  }
>  
>  install() {
> -    local _i _progs _path
> -    inst busybox /usr/bin/busybox
> -    for _i in `/sbin/busybox | sed -ne '1,/Currently/!{s/,//g; s/busybox//g; p}'`
> +    local _i _progs _path _busybox
> +    _busybox=$(type -P busybox)
> +    inst $_busybox /usr/bin/busybox
> +    for _i in `$_busybox | sed -ne '1,/Currently/!{s/,//g; s/busybox//g; p}'`
>      do
>          _progs="$_progs $_i"
>      done
> 

Thanks! Pushed.

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

end of thread, other threads:[~2012-09-26  9:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-25 22:42 [PATCH] busybox: fix installation from /sbin/busybox Daniel Drake
     [not found] ` <20120925224228.25BC8FAABE-2+9YHz4BXxlLDiiyqF6/jw@public.gmane.org>
2012-09-26  9:50   ` Harald Hoyer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox