Bug #55060 Addition of innodb_plugin to mysql-test-run default suites caused mess
Submitted: 7 Jul 2010 18:00 Modified: 15 Oct 2012 14:02
Reporter: Karen Langford Email Updates:
Status: Closed Impact on me:
Category:MySQL Server: Tests Severity:S7 (Test Cases)
Version:5.1.49 OS:Any
Assigned to: Assigned Account CPU Architecture:Any
Tags: innodb_plugin, mysql-test-run
Triage: Needs Triage: D1 (Critical) / R1 (None/Negligible) / E1 (None/Negligible)

[7 Jul 2010 18:00] Karen Langford
Marko Mäkelä's change: revision 3351.14.134 to add innodb_plugin to mysql-test-run default suites was not complete, files needed to perform tests
where not included by Bootstrap and caused all tests to fail. RPM creation also failed.

How to repeat:
Run mysql-test-run.

Suggested fix:
Needs the following extra line added to Makefile.am so bootstrap
pulls in the necessary files to test the plugin.

bteam@astra04:~/bzr/mysql-5.1.49-release> bzr diff
=== modified file 'mysql-test/Makefile.am'
--- mysql-test/Makefile.am      2010-04-01 11:19:15 +0000
+++ mysql-test/Makefile.am      2010-07-07 17:20:24 +0000
@@ -100,6 +100,7 @@ TEST_DIRS = t r include std_data std_dat
        suite/rpl_ndb suite/rpl_ndb/t suite/rpl_ndb/r \
        suite/parts suite/parts/t suite/parts/r suite/parts/inc \
        suite/innodb suite/innodb/t suite/innodb/r suite/innodb/include \
+       suite/innodb_plugin suite/innodb_plugin/t suite/innodb_plugin/r suite/innodb_plugin/include \
        suite/engines suite/engines/funcs suite/engines/iuds suite/engines/rr_trx \
        suite/engines/funcs/r suite/engines/funcs/t suite/engines/iuds/r \
        suite/engines/iuds/t suite/engines/rr_trx/include suite/engines/rr_trx/r \
[7 Jul 2010 18:32] Joerg Bruehe
Patch is correct and approved, will be used in the 5.1.49 release build.

Basic rule:
Whenever a directory is added or becomes mandatory
(like "mysql-test/suite/innodb_plugin"),
make sure that it is also part of a "make dist".

Release builds are not run within a BZR tree but from a
"bzr export ; BUILD/compile-dist ; make dist"
so better check that before pushing.
[8 Jul 2010 20:54] Joerg Bruehe
Patch is used in the build of 5.1.49, and it fixes the problem.
[15 Oct 2012 14:02] Erlend Dahl
Fixed in 5.1.49