On 09/18/2014 03:42 PM, Khem Raj wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1



On 9/1/14 8:18 AM, Robert Yang wrote:
Bashism:
possible bashism in arptables/etc/init.d/arptables line 14 (should be '.', not 
'source'):
source /etc/init.d/functions
possible bashism in arptables/etc/init.d/arptables line 96 ($"foo" should be eval_gettext 
"foo"):
         echo $"Usage $0 {start|stop|restart|condrestart|save|status}"

could these be patched instead of rdepending on bash

Thanks, I updated in the repo (V2, but V1 and V2 uses the same repo):

git://git.openembedded.org/meta-openembedded-contrib rbt/rdeps

And here is the patch:

commit e8848953ff6815c2335dc23d18ee40277e04103f
Author: Robert Yang <[email protected]>
Date:   Sun Aug 31 23:22:11 2014 -0700

    arptables: remove bashism

    Bashism:
possible bashism in arptables/etc/init.d/arptables line 14 (should be '.', not 'source'):
    source /etc/init.d/functions
possible bashism in arptables/etc/init.d/arptables line 96 ($"foo" should be eval_gettext "foo"):
            echo $"Usage $0 {start|stop|restart|condrestart|save|status}"

    Use "." to replace of "source", and change /bin/bash to /bin/sh, the
    echo $"foo" works well in our busybox.

    Signed-off-by: Robert Yang <[email protected]>

diff --git a/meta-networking/recipes-support/arptables/arptables-0.0.3-4/arptables-remove-bashism.patch b/meta-networking/recipes-support/arptables/arptables-0.0.3-4/arptables-remove-bashism.
new file mode 100644
index 0000000..f332658
--- /dev/null
+++ b/meta-networking/recipes-support/arptables/arptables-0.0.3-4/arptables-remove-bashism.patch
@@ -0,0 +1,37 @@
+From cd312bc0e3686404428878d23b8888cba09a20e1 Mon Sep 17 00:00:00 2001
+From: Robert Yang <[email protected]>
+Date: Thu, 18 Sep 2014 19:46:58 -0700
+Subject: [PATCH] arptables.sysv: remove bashism
+
+Use "." to replace of "source", and change /bin/bash to /bin/sh, the
+echo $"foo" works well in busybox.
+
+Upstream-Status: Pending
+
+Signed-off-by: Robert Yang <[email protected]>
+---
+ arptables.sysv |    4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/arptables.sysv b/arptables.sysv
+index 7a90bd2..7710376 100644
+--- a/arptables.sysv
++++ b/arptables.sysv
+@@ -1,4 +1,4 @@
+-#!/bin/bash
++#!/bin/sh
+ #
+ # init script for arptables
+ #
+@@ -11,7 +11,7 @@
+ #
+ # config: __SYSCONFIG__/arptables
+
+-source /etc/init.d/functions
++. /etc/init.d/functions
+ # source /etc/sysconfig/network
+
+ # Check that networking is up.
+--
+1.7.9.5
+

// Robert


Signed-off-by: Robert Yang <[email protected]>
---
  .../recipes-support/arptables/arptables_0.0.3-4.bb |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb 
b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
index 74f81b5..94920f9 100644
--- a/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
+++ b/meta-networking/recipes-support/arptables/arptables_0.0.3-4.bb
@@ -5,7 +5,7 @@ SECTION = "console/network"

  PR = "r2"

-RDEPENDS_${PN} += "perl"
+RDEPENDS_${PN} += "perl bash"

  LIC_FILES_CHKSUM = 
"file://${COREBASE}/meta/files/common-licenses/GPL-2.0;md5=801f80980d171dd6425610833a22dbe6"
  SRC_URI = " \


- --
- -Khem
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.22 (Darwin)
Comment: GPGTools - https://gpgtools.org

iEYEARECAAYFAlQajN0ACgkQuwUzVZGdMxSHuACdFnp3LZpRjZsN3IVLm2tCCb8L
y2kAnRFWsf2R9HDvvAp1xUoSnQB9bZfQ
=ZfQX
-----END PGP SIGNATURE-----

--
_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Reply via email to