diff options
| author | Kristian Nielsen <knielsen@knielsen-hq.org> | 2016-09-10 17:50:32 +0200 |
|---|---|---|
| committer | Otto Kekäläinen <otto@mariadb.org> | 2016-11-20 14:52:49 +0200 |
| commit | f77bd5faf53fe89f3ad7fe53bb0cee44ddfb1cec (patch) | |
| tree | 8ac6decb8a3dd160472050ae89df365d9a6b61e0 /mysql-test/t/show_explain_ps.test | |
| parent | d49cffa124199a056ba8d4e97bb6c36084aa1fd7 (diff) | |
| download | mariadb-git-f77bd5faf53fe89f3ad7fe53bb0cee44ddfb1cec.tar.gz | |
Fix use of `require` in mysql-test-run.
The motivation for this is that Perl is moving towards not having
current directory ./ in @INC by default. This is causing
mysql-test-run.pl to fail in latest Debian Unstable:
https://lists.debian.org/debian-devel-announce/2016/08/msg00013.html
However, we have `use "lib"`, there is no need for current directory
in @INC, except for a gross hack. In mtr_cases.pm, there is a
`require "mtr_misc.pl"`, which hides mtr_misc.pl away in mtr_cases
namespace. And things only work because mysql-test-run.pl loads it
with a different name, `require "lib/mtr_misc.pl"`! (Perl will
`require` only once for each unique filename).
Fix this by only using `require` in main program, and referencing
functions with :: scope from other namespaces. For multi-use in
different namespaces, proper `use` modules should be used.
Signed-off-by: Kristian Nielsen <knielsen@knielsen-hq.org>
Diffstat (limited to 'mysql-test/t/show_explain_ps.test')
0 files changed, 0 insertions, 0 deletions