Bug #30624 Syntax error in mysqld_safe script

• Submitted: 24 Aug 2007 16:57
• Modified: 9 Oct 2007 20:13
• Reporter: Jonathan Perkin
• Status: Closed
• Category: Client
• Severity: S3 (Non-critical)
• Version: 5.1.21-beta
• OS: Any
• Assigned to: Jonathan Perkin

[24 Aug 2007 16:57] Jonathan Perkin

Description:
A recent mysqld_safe update missed a change, resulting in a syntax error in the script. 
set -x output:

...
USER_OPTION=--user=mysql
+ [ -eq 0 ]
/usr/local/mysql/bin/mysqld_safe: test: argument expected

This is caused by $syslog still being used after the rest were renamed $want_syslog

As a side note, the problem only manifests itself either if the current user can write to
/ or $USER is set to "root".

How to repeat:
Run the mysqld_safe script.

[24 Aug 2007 17:01] Jonathan Perkin

The patch in http://lists.mysql.com/commits/33032 solves this problem.

[14 Sep 2007 0:59] Bugs System

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/34228

ChangeSet@1.2602, 2007-09-14 00:59:22+02:00, joerg@trift2. +1 -0
  scripts/mysqld_safe.sh
      Copy a fix by jperkin for bug#30624 into the 5.1.22 build tree.

[26 Sep 2007 15:21] Bugs System

Pushed into 5.1.23-beta

[5 Oct 2007 19:57] Bugs System

Pushed into 5.1.23-beta

[9 Oct 2007 20:13] Paul DuBois

Noted in 5.1.23 changelog.

The mysqld_safe script had a syntax error in a test operation.

[10 Oct 2007 12:08] Jon Stephens

Also documented in mysql-5.1.22-ndb-6.2.7 changelog.

[15 Oct 2007 19:57] Jon Stephens

Also documented in mysql-5.1.22-ndb-6.3.4 changelog (already closed).