[LTP] [PATCH 1/1] network: handle ping6 vs. ping -6 compatibility

Petr Vorel pvorel@suse.cz
Wed Dec 21 18:31:48 CET 2016


ping6 was merged into ping and removed in iputils-s20150815. Distros
with older iputils version needs to define variable PING6_CMD=ping6

TODO: handle $tping in testcases/kernel/containers/netns/netns_helper.sh
(doesn't use test_net.sh).

Signed-off-by: Petr Vorel <pvorel@suse.cz>
Cc: Alexey Kodanev <alexey.kodanev@oracle.com>
---
IMHO would be nice to have ping for IPv6 on one place.
This will break old distros, which have iputils older than iputils-s20150815.
Do we care (want default PING6_CMD be ping6)?
---
 testcases/lib/test_net.sh                       | 1 +
 testcases/network/stress/ns-tools/ns-echoclient | 2 +-
 testcases/network/tcp_cmds/ping/ping01.sh       | 9 +++++----
 testcases/network/tcp_cmds/ping/ping02.sh       | 5 +++--
 4 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/testcases/lib/test_net.sh b/testcases/lib/test_net.sh
index 1d350fdd1..94872d186 100644
--- a/testcases/lib/test_net.sh
+++ b/testcases/lib/test_net.sh
@@ -426,6 +426,7 @@ export UPLOAD_BIGFILESIZE=${UPLOAD_BIGFILESIZE:-"2147483647"}
 export UPLOAD_REGFILESIZE=${UPLOAD_REGFILESIZE:-"1024"}
 export MCASTNUM_NORMAL=${MCASTNUM_NORMAL:-"20"}
 export MCASTNUM_HEAVY=${MCASTNUM_HEAVY:-"40000"}
+export PING6_CMD="${PING6_CMD:-ping -6}"
 
 [ -n "$TST_USE_NETNS" -a "$TST_INIT_NETNS" != "no" ] && init_ltp_netspace
 
diff --git a/testcases/network/stress/ns-tools/ns-echoclient b/testcases/network/stress/ns-tools/ns-echoclient
index 1f3187823..15e62876d 100644
--- a/testcases/network/stress/ns-tools/ns-echoclient
+++ b/testcases/network/stress/ns-tools/ns-echoclient
@@ -120,7 +120,7 @@ case $family in
     ping_command="ping"
     ;;
     6)
-    ping_command="ping6"
+    ping_command="$PING6_CMD"
     ;;
     *)
     echo "protocol family should be 4 or 6."
diff --git a/testcases/network/tcp_cmds/ping/ping01.sh b/testcases/network/tcp_cmds/ping/ping01.sh
index 73f1180f5..7505cdedc 100755
--- a/testcases/network/tcp_cmds/ping/ping01.sh
+++ b/testcases/network/tcp_cmds/ping/ping01.sh
@@ -35,17 +35,18 @@ do_setup()
 	COUNT=${COUNT:-3}
 	PACKETSIZES=${PACKETSIZES:-"8 16 32 64 128 256 512 1024 2048 4064"}
 
-	PING_CMD=ping${TST_IPV6}
+	PING=ping
+	[ "$TST_IPV6" ] && PING=$PING6_CMD
 
-	tst_check_cmds $PING_CMD
+	tst_check_cmds $(echo $PING | sed 's/\([^ ]\+\).*/\1/')
 }
 
 do_test()
 {
-	tst_resm TINFO "$PING_CMD with $PACKETSIZES ICMP packets"
+	tst_resm TINFO "$PING with $PACKETSIZES ICMP packets"
 	local ipaddr=$(tst_ipaddr rhost)
 	for packetsize in $PACKETSIZES; do
-		EXPECT_PASS $PING_CMD -c $COUNT -s $packetsize $ipaddr \>/dev/null
+		EXPECT_PASS $PING -c $COUNT -s $packetsize $ipaddr \>/dev/null
 	done
 }
 
diff --git a/testcases/network/tcp_cmds/ping/ping02.sh b/testcases/network/tcp_cmds/ping/ping02.sh
index 6a9878e42..47877476c 100755
--- a/testcases/network/tcp_cmds/ping/ping02.sh
+++ b/testcases/network/tcp_cmds/ping/ping02.sh
@@ -24,9 +24,10 @@ do_setup()
 	COUNT=${COUNT:-3}
 	PACKETSIZES=${PACKETSIZES:-"8 16 32 64 128 256 512 1024 2048 4064"}
 
-	PING=ping${TST_IPV6}
+	PING=ping
+	[ "$TST_IPV6" ] && PING=$PING6_CMD
 
-	tst_check_cmds $PING
+	tst_check_cmds $(echo $PING | sed 's/\([^ ]\+\).*/\1/')
 }
 
 do_test()
-- 
2.11.0



More information about the ltp mailing list