Merge lp:~dveeden/mysql-sandbox/mysql-sandbox into lp:mysql-sandbox

Proposed by Daniël van Eeden
Status: Merged
Merge reported by: Giuseppe Maxia
Merged at revision: not available
Proposed branch: lp:~dveeden/mysql-sandbox/mysql-sandbox
Merge into: lp:mysql-sandbox
Diff against target: 11 lines (+1/-0)
1 file modified
lib/MySQL/Sandbox/Scripts.pm (+1/-0)
To merge this branch: bzr merge lp:~dveeden/mysql-sandbox/mysql-sandbox
Reviewer Review Type Date Requested Status
Giuseppe Maxia Pending
Review via email: mp+192344@code.launchpad.net

Description of the change

Don't use defaults-file for mysql_config_editor.

This prevents errors like this:
$ ./my sql_config_editor print --all
  /home/some_user/opt/mysql/5.6.14/bin/mysql_config_editor: unknown variable 'defaults-file=/home/some_user/sandboxes/msb_5_6_14/my.sandbox.cnf'

To post a comment you must log in.
Revision history for this message
Giuseppe Maxia (giuseppe-maxia) wrote :

Fixed as part of bug#1155517

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/MySQL/Sandbox/Scripts.pm'
2--- lib/MySQL/Sandbox/Scripts.pm 2013-09-30 09:15:54 +0000
3+++ lib/MySQL/Sandbox/Scripts.pm 2013-10-23 15:27:51 +0000
4@@ -1491,6 +1491,7 @@
5 NODEFAULT=(myisam_ftdump
6 myisamlog
7 mysql_config
8+mysql_config_editor
9 mysql_convert_table_format
10 mysql_find_rows
11 mysql_fix_extensions

Subscribers

People subscribed via source and target branches