[PATCH] testbot/build: Allow both commas and colons in the target lists.

Francois Gouget fgouget at codeweavers.com
Mon Aug 20 02:08:10 CDT 2018


This paves the way to a future switch to colons.
It is only needed for Build.pl and WineTest.pl since the TestBot does
not pass a target list to the Reconfig scripts.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/bin/build/Build.pl    | 2 +-
 testbot/bin/build/WineTest.pl | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/testbot/bin/build/Build.pl b/testbot/bin/build/Build.pl
index 920fc8204..b729d95f4 100755
--- a/testbot/bin/build/Build.pl
+++ b/testbot/bin/build/Build.pl
@@ -250,7 +250,7 @@ if (!defined $Usage)
   }
 
   $TargetList = join(",", keys %AllTargets) if (!defined $TargetList);
-  foreach my $Target (split /,/, $TargetList)
+  foreach my $Target (split /[,:]/, $TargetList)
   {
     $Target = "exe$1" if ($Target =~ /^(32|64)$/);
     if (!$AllTargets{$Target})
diff --git a/testbot/bin/build/WineTest.pl b/testbot/bin/build/WineTest.pl
index 101061917..3310b4c53 100755
--- a/testbot/bin/build/WineTest.pl
+++ b/testbot/bin/build/WineTest.pl
@@ -282,7 +282,7 @@ if (!defined $Usage)
 {
   if (defined $TargetList)
   {
-    foreach my $Target (split /,/, $TargetList)
+    foreach my $Target (split /[,:]/, $TargetList)
     {
       if (!$AllTargets{$Target})
       {
-- 
2.18.0



More information about the wine-devel mailing list