[Xen-devel] [OSSTEST PATCH 5/5] mg-list-all-branches: Do not match ${BRANCHES+= ... }

[Ian Jackson <ian.jackson@xxxxxxxxxxxxx>]

This is not valid shell syntax and should not appear.  The confusion
seems to have arisen because of the need for to match BRANCHES+=...
(without the surrounding { }).

This results in no change to the output.  (I seem to have collected
this patch some time ago as part of some fixes to mg-list-all-branches
which have by now been applied, but not managed to write up and post
this specific change.)

Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
---
 mg-list-all-branches | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mg-list-all-branches b/mg-list-all-branches
index 62d3ff1..989b5ae 100755
--- a/mg-list-all-branches
+++ b/mg-list-all-branches
@@ -14,7 +14,7 @@ foreach my $f (qw(cr-for-branches crontab)) {
     open C, $f or die $!;
     while (<C>) {
         next unless
-           m/\$\{$branchvar_re[:+]?=()($branchchr_re+)\b/ ||
+           m/\$\{$branchvar_re:=()($branchchr_re+)\b/ ||
            m/$branchvar_re[:+]?=('?)($branchchr_re+?)\1\s/;
         $branches{$_}=1 foreach split /\s+/, $2;
     }
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel