Error in mysqld_safe script prevents startup

[modemmonkey@…]

In the most recent version of the percona port (percona @5.5.32-31.0_0), there is a bug in the mysqld_safe script that prevents MySQL from starting up. Upon running it, it gives the following error:

/opt/local/lib/percona/bin/mysqld_safe: line 734: syntax error near unexpected token `then'
/opt/local/lib/percona/bin/mysqld_safe: line 734: `    then    # The pid contains a mysqld process'

Looking into the script, there is an IF statement on line 733 that has no condition, and I'm assuming that's the issue. I've worked around this by re-activating a previous version of the port, but it's going to be a pain for users who are installing it for the first time.

[mf2k (Frank Schima)]

In the future, please Cc the port maintainer(s) and do not set the Priority field.

[pcollinson]

I've changed the code in mysqld_safe at line 723 from

if kill -0 $PID > /dev/null 2> /dev/null
  then
    if 
    then    # The pid contains a mysqld process                                 
      log_error "A mysqld process already exists"
      exit 1
    fi
  fi

to

  if kill -0 $PID > /dev/null 2> /dev/null
  then
    if ps -ef | grep -v mysqld_safe | grep -- $MYSQLD | grep $PID > /dev/null
    then    # The pid contains a mysqld process                                 
      log_error "A mysqld process already exists"
      exit 1
    fi
  fi

The ps line was taken from the previous release.

[pixilla (Bradley Giesbrecht)]

Cc Me!

[parzley@…]

I get the same for mysql55-server @5.5.33:

/opt/local/lib/mysql55/bin/mysqld_safe: line 726: syntax error near unexpected token `then'
/opt/local/lib/mysql55/bin/mysqld_safe: line 726: `    then    # The pid contains a mysqld process'

and with mariadb-server @5.5.32

/opt/local/lib/mariadb/bin/mysqld_safe: line 748: syntax error near unexpected token `then'
/opt/local/lib/mariadb/bin/mysqld_safe: line 748: `    then    # The pid contains a mysqld process'

[parzley@…]

Cc Me!

[mf2k (Frank Schima)]

See #57827

[chrstphrchvz (Christopher Chavez)]

In d8c15aa95a4d37310107078d69da01c69c7a4b82/macports-ports (master):

percona: delete port

Port is not known to build, has known_fail yes,
zero reported installations, no tickets in 7 years.
Outdated since release of 8.0.17-8 on 2019-10-30.
Other working MySQL ports are available.

Closes: #40018
Closes: #45136
Closes: #47614
See: #52557