Re: [PATCH 1/2] BUG/MINOR: reg-tests: fix service dependency script

[Christopher Faulet]

Le 10/01/2021 à 21:13, William Dauchy a écrit :

I badly tested my previous patch forgetting to remove the "+" testing
present in options, and not in services; the list of services do not
have any "+" at the beginning of each service

this patch is fixing commit aabde7133242563109b4f36c42e732f083 ("MINOR:
reg-tests: add a way to add service dependency")

Signed-off-by: William Dauchy <wdau...@gmail.com>
---
  scripts/run-regtests.sh | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/run-regtests.sh b/scripts/run-regtests.sh
index e6cb0e6cf..27bb13cbf 100755
--- a/scripts/run-regtests.sh
+++ b/scripts/run-regtests.sh
@@ -204,7 +204,7 @@ _findtests() {
        alternatives=$(echo "$requiredservice" | sed -e 's/|/ /g')
        found=
        for alt in $alternatives; do
-        if echo "$SERVICES" | grep -qw "\+$alt"; then
+        if echo "$SERVICES" | grep -qw "$alt"; then
            found=1;
        fi
        done

Now merged, thanks William !

--
Christopher Faulet