diff options
| author | John Crispin <blogic@openwrt.org> | 2014-10-14 12:21:11 +0000 |
|---|---|---|
| committer | John Crispin <blogic@openwrt.org> | 2014-10-14 12:21:11 +0000 |
| commit | 81d9f2d5856395476060d7107e0bfba1696228d1 (patch) | |
| tree | d3a0f88b38cdfc9e7a6cb1246fd98d8b0249990a /target/linux/ixp4xx | |
| parent | 82fa1674e0f97e634c0be3a345cfc7575c36f659 (diff) | |
| download | upstream-81d9f2d5856395476060d7107e0bfba1696228d1.tar.gz upstream-81d9f2d5856395476060d7107e0bfba1696228d1.tar.bz2 upstream-81d9f2d5856395476060d7107e0bfba1696228d1.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>
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@42911 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target/linux/ixp4xx')
| -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 } |