diff options
| author | John Crispin <john@openwrt.org> | 2014-10-14 12:21:11 +0000 |
|---|---|---|
| committer | John Crispin <john@openwrt.org> | 2014-10-14 12:21:11 +0000 |
| commit | 20940138ac41e3eca54afefc34f65e40834be3ea (patch) | |
| tree | 3c34846e48daad4595ab2ab2c00f1068022ee62f /target/linux/ixp4xx/base-files/lib/preinit | |
| parent | 8267c0e2ac371fe0f93ba34f745976acf358a253 (diff) | |
| download | upstream-20940138ac41e3eca54afefc34f65e40834be3ea.tar.gz upstream-20940138ac41e3eca54afefc34f65e40834be3ea.tar.bz2 upstream-20940138ac41e3eca54afefc34f65e40834be3ea.zip | |
scripts: fix wrong usage of '==' operator
[base-files] shell-scripting: fix wrong usage of '==' operator
normally the '==' is used for invoking a regex parser and is a bashism.
all of the fixes just want to compare a string. the used busybox-ash
will silently "ignore" this mistake, but make it portable/clean at least.
this patch does not change the behavior/logic of the scripts.
Signed-off-by: Bastian Bittorf <bittorf@bluebottle.com>
SVN-Revision: 42911
Diffstat (limited to 'target/linux/ixp4xx/base-files/lib/preinit')
| -rw-r--r-- | target/linux/ixp4xx/base-files/lib/preinit/05_set_ether_mac_ixp4xx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/target/linux/ixp4xx/base-files/lib/preinit/05_set_ether_mac_ixp4xx b/target/linux/ixp4xx/base-files/lib/preinit/05_set_ether_mac_ixp4xx index e707dabf05..adda197b59 100644 --- a/target/linux/ixp4xx/base-files/lib/preinit/05_set_ether_mac_ixp4xx +++ b/target/linux/ixp4xx/base-files/lib/preinit/05_set_ether_mac_ixp4xx @@ -6,7 +6,7 @@ set_ether_mac() { for npe in eth0 eth1 eth2 do - if [ "$(ifconfig $npe 2>/dev/null | grep -c 00:00:00:00:00:00)" == "1" ]; then + if [ "$(ifconfig $npe 2>/dev/null | grep -c 00:00:00:00:00:00)" = "1" ]; then ifconfig $npe hw ether $(fconfig -s -r -d /dev/$RB_CONFIG -n npe_"$npe"_esa) fi done @@ -14,16 +14,16 @@ set_ether_mac() { # Some developers should be shot on sight at Zcom/Netgear # -- Fixup for the WG302v1, need someone with a WAG302v1 to fix that, too - if [ "$(ifconfig eth0 2>/dev/null | grep -c 00:00:00:00:00:00)" == "1" ]; then + if [ "$(ifconfig eth0 2>/dev/null | grep -c 00:00:00:00:00:00)" = "1" ]; then ifconfig eth0 hw ether $(fconfig -s -r -d /dev/$RB_CONFIG -n zcom_npe_esa) fi # Others (*cough*, Tonze) are dumb enough to not handle mac addresses at all - if [ "$(ifconfig eth0 2>/dev/null | grep -c 00:00:00:00:00:00)" == "1" ]; then + if [ "$(ifconfig eth0 2>/dev/null | grep -c 00:00:00:00:00:00)" = "1" ]; then ifconfig eth0 hw ether 00:11:22:33:44:55 fi - if [ "$(ifconfig eth1 2>/dev/null | grep -c 00:00:00:00:00:00)" == "1" ]; then + if [ "$(ifconfig eth1 2>/dev/null | grep -c 00:00:00:00:00:00)" = "1" ]; then ifconfig eth1 hw ether 00:11:22:33:44:56 fi } |