Hi, Monty! On Apr 05, Michael Widenius wrote:
revision-id: e3971006cca (mariadb-10.5.2-547-ge3971006cca) parent(s): ae009435e72 author: Michael Widenius
committer: Michael Widenius timestamp: 2021-03-24 19:25:09 +0200 message: Give a readable error in mtr if resolve_at_variable fails
This change is wrong. Clearly, the || '' part was intentional.
Check the history, it was added in the commit
commit 897b51db434
Author: Sergei Golubchik
diff --git a/mysql-test/lib/My/ConfigFactory.pm b/mysql-test/lib/My/ConfigFactory.pm index 3249a06256c..5a5b691998c 100644 --- a/mysql-test/lib/My/ConfigFactory.pm +++ b/mysql-test/lib/My/ConfigFactory.pm @@ -347,8 +347,15 @@ sub resolve_at_variable { or croak "There is no group named '$group_name' that ", "can be used to resolve '$option_name' for test '$self->{testname}'";
- my $value= $from_group->value($option_name) || ''; - $res .= $before.$value; + my $value= $from_group->value($option_name); + if (!defined($value)) + { + ::mtr_verbose("group: $group_name option_name: $option_name is undefined"); + } + else + { + $res .= $before.$value; + } } $res .= $after;
Regards, Sergei VP of MariaDB Server Engineering and security@mariadb.org