diff mbox series

network/traceroute01: make it compatible with Busybox

Message ID 20201112145636.31091-1-kory.maincent@bootlin.com
State Accepted
Headers show
Series network/traceroute01: make it compatible with Busybox | expand

Commit Message

Kory Maincent Nov. 12, 2020, 2:56 p.m. UTC
Busybox has not "-T" parameter, return TCONF in that case.

Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---
 testcases/network/traceroute/traceroute01.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Petr Vorel Nov. 12, 2020, 5:41 p.m. UTC | #1
Hi Kory,

> Busybox has not "-T" parameter, return TCONF in that case.
Thanks, merged with slightly updated message.
+ followup patch:
https://patchwork.ozlabs.org/project/ltp/patch/20201112173609.4123-1-petr.vorel@suse.com/

Kind regards,
Petr
Kory Maincent Nov. 12, 2020, 5:48 p.m. UTC | #2
Hi Petr,

On Thu, 12 Nov 2020 18:41:34 +0100
Petr Vorel <pvorel@suse.cz> wrote:

> Hi Kory,
> 
> > Busybox has not "-T" parameter, return TCONF in that case.  
> Thanks, merged with slightly updated message.
> + followup patch:
> https://patchwork.ozlabs.org/project/ltp/patch/20201112173609.4123-1-petr.vorel@suse.com/

Ok thanks.

Regards,

> 
> Kind regards,
> Petr
diff mbox series

Patch

diff --git a/testcases/network/traceroute/traceroute01.sh b/testcases/network/traceroute/traceroute01.sh
index 821853f0e..5b4a6d5a2 100755
--- a/testcases/network/traceroute/traceroute01.sh
+++ b/testcases/network/traceroute/traceroute01.sh
@@ -56,7 +56,11 @@  test1()
 test2()
 {
 	tst_res TINFO "run traceroute with TCP SYN"
-	run_trace -T
+	if traceroute -T 2>&1 | grep -q "invalid option"; then
+		tst_res TCONF "Wrong version of traceroute, traceroute from busybox can not use TCP SYN"
+	else
+		run_trace -T
+	fi
 }
 
 tst_run