Bug #51961 mysqld_multi master fails to load - has unmatched right curly bracket

• Submitted: 11 Mar 2010 16:43
• Modified: 16 Mar 2010 15:46
• Reporter: John Smith
• Status: Duplicate
• Category: MySQL Server: General
• Severity: S3 (Non-critical)
• Version: 5.1.44, 5.1.46-bzr
• OS: Linux
• CPU Architecture: Any
• Tags: Contribution, mysqld_multi, regression

[11 Mar 2010 16:43] John Smith

Description:

Unmatched right curly bracket at /opt/mysql/bin/mysqld_multi line 171, at end of line
syntax error at /opt/mysql/bin/mysqld_multi line 171, near "}"
Execution of /opt/mysql/bin/mysqld_multi aborted due to compilation errors.

(The compiler is complaining...  the actual missing bracket seems to be around line 74.)

How to repeat:
execute:
/opt/mysql/bin/mysqld_multi

Suggested fix:
Change the following:
-----------------
  # Handle deprecated --config-file option: convert to --defaults-extra-file
  foreach my $arg (@ARGV)
  {
    if ($arg =~ m/^--config-file=(.*)/)
    {
      # Put it at the beginning of the list, so it has lower precedence
      # than a correct --defaults-extra-file option

      unshift @defaults_options, "--defaults-extra-file=$1";
      print "WARNING: --config-file is deprecated and will be removed\n";
      print "in MySQL 5.6.  Please use --defaults-extra-file instead\n";
    }
   }
  }

To:
----------------
  # Handle deprecated --config-file option: convert to --defaults-extra-file
  foreach my $arg (@ARGV)
  {
    if ($arg =~ m/^--config-file=(.*)/)
    {
      # Put it at the beginning of the list, so it has lower precedence
      # than a correct --defaults-extra-file option

      unshift @defaults_options, "--defaults-extra-file=$1";
      print "WARNING: --config-file is deprecated and will be removed\n";
      print "in MySQL 5.6.  Please use --defaults-extra-file instead\n";
    }
#   }
  }

[11 Mar 2010 17:21] Valeriy Kravchuk

Thank you for the bug report. Verified just as described by code review.

[11 Mar 2010 17:22] Valeriy Kravchuk

5.0.x does not have this problem, so it is a regression.

[16 Mar 2010 15:46] Omer Barnir

Duplicate of bug#51468