From fbc715200e6acc96de1ec0d091abadbd5bfb6546 Mon Sep 17 00:00:00 2001 From: Antonio Ospite Date: Mon, 14 Apr 2025 14:40:13 +0200 Subject: [PATCH] ci/android: strip tailing spaces in Android CTS expectation files Trailing spaces in include and exclude filters can confuse cts-tradefed, so make sure to strip those from the lines in Android CTS expectation files, since those are used to build the filter arguments. Part-of: --- .gitlab-ci/android-cts-runner.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.gitlab-ci/android-cts-runner.sh b/.gitlab-ci/android-cts-runner.sh index 6f714b9880d..2f714d28116 100755 --- a/.gitlab-ci/android-cts-runner.sh +++ b/.gitlab-ci/android-cts-runner.sh @@ -14,13 +14,13 @@ SKIP_FILE="$INSTALL/${GPU_VERSION}-android-cts-skips.txt" EXCLUDE_FILTERS="" if [ -e "$SKIP_FILE" ]; then - EXCLUDE_FILTERS="$(grep -v -E "(^#|^[[:space:]]*$)" "$SKIP_FILE" | sed -s 's/.*/--exclude-filter "\0" /g')" + EXCLUDE_FILTERS="$(grep -v -E "(^#|^[[:space:]]*$)" "$SKIP_FILE" | sed -e 's/\s*$//g' -e 's/.*/--exclude-filter "\0" /g')" fi INCLUDE_FILE="$INSTALL/${GPU_VERSION}-android-cts-include.txt" if [ -e "$INCLUDE_FILE" ]; then - INCLUDE_FILTERS="$(grep -v -E "(^#|^[[:space:]]*$)" "$INCLUDE_FILE" | sed -s 's/.*/--include-filter "\0" /g')" + INCLUDE_FILTERS="$(grep -v -E "(^#|^[[:space:]]*$)" "$INCLUDE_FILE" | sed -e 's/\s*$//g' -e 's/.*/--include-filter "\0" /g')" else INCLUDE_FILTERS=$(printf -- "--include-filter %s " $ANDROID_CTS_MODULES | sed -e 's/ $//g') fi