You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildstream.apache.org by ak...@apache.org on 2022/03/17 07:39:23 UTC

[buildstream] 01/01: .github/run-ci.sh: remove extra quotes

This is an automated email from the ASF dual-hosted git repository.

akitouni pushed a commit to branch abderrahim/fix-tests
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 8ff0efbf2f2ae4684c192dca2d2f0adc99c8044d
Author: Abderrahim Kitouni <ak...@gnome.org>
AuthorDate: Thu Mar 17 08:39:01 2022 +0100

    .github/run-ci.sh: remove extra quotes
    
    The quotes prevent passing multiple tests (or no test) on the command line
    
    Fixes ec84e3f113e991c6824d3fa16c105cf2be0e7f50
---
 .github/run-ci.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/run-ci.sh b/.github/run-ci.sh
index 4769a36..c273523 100755
--- a/.github/run-ci.sh
+++ b/.github/run-ci.sh
@@ -102,13 +102,13 @@ function runServiceTest() {
 
 
 if [ -z "${test_names}" ]; then
-    for test_name in "mypy debian-10 fedora-34 fedora-35 fedora-missing-deps"; do
+    for test_name in mypy debian-10 fedora-34 fedora-35 fedora-missing-deps; do
 	if ! runTest "${test_name}"; then
 	    echo "Tests failed"
 	    exit 1
 	fi
     done
-    for test_name in "buildgrid buildbarn"; do
+    for test_name in buildgrid buildbarn; do
 	if ! runServiceTest "${test_name}"; then
 	    echo "Tests failed"
 	    exit 1
@@ -116,14 +116,14 @@ if [ -z "${test_names}" ]; then
     done
 else
     if $arg_service; then
-	for test_name in "${test_names}"; do
+	for test_name in ${test_names}; do
 	    if ! runServiceTest "${test_name}"; then
 		echo "Tests failed"
 		exit 1
 	    fi
 	done
     else
-	for test_name in "${test_names}"; do
+	for test_name in ${test_names}; do
 	    if ! runTest "${test_name}"; then
 		echo "Tests failed"
 		exit 1