diff mbox

SV: [base] busybox-ifplugd reference set til /usr/bin/ifplugd instead of /usr/sbin/ifplugd.

Message ID BD22328D0EB182418C2A0952DC2454F8791B7957@VMPREVAS1.prevas.se
State Not Applicable
Headers show

Commit Message

kim.hansen@prevas.dk Dec. 17, 2014, 7:32 a.m. UTC 
Hi Mads,

I can't figure out what this is trying to solve, can you elaborate?

/Kim

Comments

Mads Doré Hansen Dec. 17, 2014, 7:37 a.m. UTC | #1 
Hi Kim,

When going from using ifplugd to use busybox ifplugd the installation 
path are of, as the two different ifplugd's are install in different places.

First I tried to let them co-exist on the target - bad idea ;-)

Then I went for the busybox ifplugd, but the users of it crashed as the 
where looking in the wrong place for it, expecting ifplugd to be where 
the non-busybox one was placed.

Best regards
Mads

On 2014-12-17 08:32, Kim Højgaard-Hansen wrote:
> Hi Mads,
>
> I can't figure out what this is trying to solve, can you elaborate?
>
> /Kim
> ____________________________
> KIM HØJGAARD-HANSEN
> Software Developer
>
> Prevas A/S
> Hedeager 1
> DK-8200 Århus N
>
> Central +45 87438070
> Direct/Mobile +45 51547387
> kim.hansen@prevas.dk
> www.prevas.dk
> ________________________________________
> Fra: dev-bounces@oe-lite.org [dev-bounces@oe-lite.org] på vegne af Mads Doré [dore@madsdore.dk]
> Sendt: 30. juli 2014 08:17
> Til: dev@oe-lite.org
> Emne: [base] busybox-ifplugd reference set til /usr/bin/ifplugd instead of      /usr/sbin/ifplugd.
>
> ---
>   recipes/busybox/files/busybox-ifplugd | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/recipes/busybox/files/busybox-ifplugd b/recipes/busybox/files/busybox-ifplugd
> index 6fd6230..a99668c 100644
> --- a/recipes/busybox/files/busybox-ifplugd
> +++ b/recipes/busybox/files/busybox-ifplugd
> @@ -2,7 +2,7 @@
>
>   CFG=/etc/ifplugd/ifplugd.conf
>
> -IFPLUGD=/usr/sbin/ifplugd
> +IFPLUGD=/usr/bin/ifplugd
>   test -x $IFPLUGD || exit 0
>
>   [ -f $CFG ] && . $CFG
> --
> 2.0.3
>
> _______________________________________________
> dev mailing list
> dev@oe-lite.org
> http://lists.oe-lite.org/mailman/listinfo/dev
kim.hansen@prevas.dk Dec. 17, 2014, 1:40 p.m. UTC | #2 
Could you try if it works fine with:

IFPLUGD="/bin/busybox ifplugd"

That would ensure that we always use the busybox version. I will fix the ifplugd recipe to provide the same service, so you would get a conflict if you try to include both

/Kim
diff mbox

Patch

diff --git a/recipes/busybox/files/busybox-ifplugd b/recipes/busybox/files/busybox-ifplugd
index 6fd6230..a99668c 100644
--- a/recipes/busybox/files/busybox-ifplugd
+++ b/recipes/busybox/files/busybox-ifplugd
@@ -2,7 +2,7 @@ 

 CFG=/etc/ifplugd/ifplugd.conf

-IFPLUGD=/usr/sbin/ifplugd
+IFPLUGD=/usr/bin/ifplugd
 test -x $IFPLUGD || exit 0

 [ -f $CFG ] && . $CFG