Skip to contentSkip to conversation

selftests: do not require bash to run bpf tests

PATCH In review Rolf Eike Beer 3 participants 3 messages Dec 14, 2016

Subsystems (5)
lore
Share & tools
lore
#0 lore raw

PATCH 2/3 selftests: do not require bash to run bpf tests

Quoted 1 line

From b9d6c1b7427d708ef2d4d57aac17b700b3694d71 Mon Sep 17 00:00:00 2001

From: Rolf Eike Beer <[email protected]>
Date: Wed, 14 Dec 2016 09:58:12 +0100
Subject: [PATCH 2/3] selftests: do not require bash to run bpf tests

Nothing in this minimal script seems to require bash. We often run these tests on embedded devices where the only shell available is the busybox ash.

---
 tools/testing/selftests/bpf/test_kmod.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tools/testing/selftests/bpf/test_kmod.sh +1 -1
index 92e627a..6d58cca 100755
@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/bin/sh
 
 SRC_TREE=../../../../
 
Signature · 11 lines
2.8.3


-- 
Rolf Eike Beer, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Bertha-von-Suttner-Str. 9, 37085 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055

emlix – smart embedded open source
maintainer Acked-by on v1 2/3
#1 lore raw
Quoted 9 lines · Rolf Eike Beer

From b9d6c1b7427d708ef2d4d57aac17b700b3694d71 Mon Sep 17 00:00:00 2001
From: Rolf Eike Beer <[email protected]>
Date: Wed, 14 Dec 2016 09:58:12 +0100
Subject: [PATCH 2/3] selftests: do not require bash to run bpf tests

Nothing in this minimal script seems to require bash. We often run these tests on embedded devices where the only shell available is the busybox ash.

#2 lore raw
Quoted 6 lines · Daniel Borkmann
Quoted 5 lines · Rolf Eike Beer

From b9d6c1b7427d708ef2d4d57aac17b700b3694d71 Mon Sep 17 00:00:00 2001
From: Rolf Eike Beer <[email protected]>
Date: Wed, 14 Dec 2016 09:58:12 +0100
Subject: [PATCH 2/3] selftests: do not require bash to run bpf tests

Quoted 6 lines
Quoted 4 lines

Nothing in this minimal script seems to require bash. We often run these tests on embedded devices where the only shell available is the busybox ash.

Thanks. I will get these into 4.10-rc1 or rc2

-- Shuah