From eaea2bce024fa6ae0db54af1e78b4d477d422791 Mon Sep 17 00:00:00 2001 From: Quentin Monnet Date: Tue, 26 Oct 2021 22:23:56 +0100 Subject: [PATCH] sync: remove redundant test on $BPF_BRANCH The sync-kernel.sh script has two consecutive tests for $BPF_BRANCH being provided by the user (and so the second one can currently never fail). Looking at the error message displayed in each case, we want to keep the second one. Let's remove the first check. Signed-off-by: Quentin Monnet --- scripts/sync-kernel.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/sync-kernel.sh b/scripts/sync-kernel.sh index 749c741..b5fa2aa 100755 --- a/scripts/sync-kernel.sh +++ b/scripts/sync-kernel.sh @@ -17,7 +17,7 @@ BPF_BRANCH=${3-""} BASELINE_COMMIT=${BPF_NEXT_BASELINE:-$(cat ${LIBBPF_REPO}/CHECKPOINT-COMMIT)} BPF_BASELINE_COMMIT=${BPF_BASELINE:-$(cat ${LIBBPF_REPO}/BPF-CHECKPOINT-COMMIT)} -if [ -z "${LIBBPF_REPO}" ] || [ -z "${LINUX_REPO}" ] || [ -z "${BPF_BRANCH}" ]; then +if [ -z "${LIBBPF_REPO}" ] || [ -z "${LINUX_REPO}" ]; then echo "Error: libbpf or linux repos are not specified" usage fi