| Message ID | 1469020751-12413-5-git-send-email-ian.jackson@eu.citrix.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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; }
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@eu.citrix.com> --- mg-list-all-branches | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)