Created attachment 145781 [details] mysql-server rc script patch to pass mysql_user to mysql_install_db With mysql56, the rc script attempts to bootstrap using mysql_install_db but the created directories mysql/ and test/ are not chown'ed to the mysql user and thus cause subsequent failures. I believe the mysql_install_db (perl) script expects the user option to be passed. The attached patch fixes this issue. Thanks & cheerio, Harry.
over to maintainer
Hi Alex, can you please incorporate this fix – I am continued to get user complaints :-/ Thanks & cheerio, Harry.
Can _someone_ please incorporate this patch!? Thanks & cheerio, Harry.
The created directories *are* chown'ed by mysql user: mysql_create_auth_tables() { eval $mysql_install_db $mysql_install_db_args >/dev/null 2>/dev/null [ $? -eq 0 ] && chown -R ${mysql_user}:${mysql_user} ${mysql_dbdir} } and in fact it works for 99.9% of users. Please investigate why it's not working for you.
Hi Alex, mySQL introduced a regression of http://bugs.mysql.com/bug.php?id=36462. It re-occured after switching from a shell to a Perl-based installation (bootstraping) script. The original mySQL fix for the shell script was http://lists.mysql.com/commits/70593. Cheerio, Harry.
Created attachment 170458 [details] backported fix from databases/mysql57 Fix backported from databases/mysql57 Tested, ready to commit
Hi Markus, although nearly two years after submitting the original patch is an unacceptable timeframe, I am glad it (eventually) got resolved. Thanks for picking it up! Cheerio, Harry.
Comment on attachment 145781 [details] mysql-server rc script patch to pass mysql_user to mysql_install_db Obsoleting this as it's not a patch against the files in the tree.
Setting proper maintainer feedback query.
Comment on attachment 170458 [details] backported fix from databases/mysql57 Setting approval request from proper maintainer.
Could the maintainer please incorporate the fix? Thanks!
A commit references this bug: Author: feld Date: Thu Jan 19 22:29:06 UTC 2017 New revision: 431919 URL: https://svnweb.freebsd.org/changeset/ports/431919 Log: databases/mysql56: Update to 5.6.35 - Port improvements from MySQL 5.7 port - Use system libs instead of bundled - Fix many open PRs - Change MAINTAINER PR: 216244 192657 198812 199751 205093 PR: 209618 211248 205983 209338 Changes: head/databases/mysql56-client/Makefile head/databases/mysql56-client/files/patch-CMakeLists.txt head/databases/mysql56-client/files/patch-extra_CMakeLists.txt head/databases/mysql56-client/files/patch-man_CMakeLists.txt head/databases/mysql56-client/files/patch-mysys_ssl_my_default.cc head/databases/mysql56-client/files/patch-scripts_CMakeLists.txt head/databases/mysql56-client/files/patch-support-files_CMakeLists.txt head/databases/mysql56-client/pkg-message head/databases/mysql56-server/Makefile head/databases/mysql56-server/distinfo head/databases/mysql56-server/files/my.cnf.sample.in head/databases/mysql56-server/files/mysql-server.in head/databases/mysql56-server/files/patch-mysys_ssl_my_default.cc head/databases/mysql56-server/pkg-message head/databases/mysql56-server/pkg-plist
A commit references this bug: Author: brnrd Date: Thu Jan 26 19:58:06 UTC 2017 New revision: 432535 URL: https://svnweb.freebsd.org/changeset/ports/432535 Log: MFH: r431919 r431968 r431975 r432035 r432066 r432458 databases/mysql56: Update to 5.6.35 - Port improvements from MySQL 5.7 port - Use system libs instead of bundled - Fix many open PRs - Change MAINTAINER PR: 216244 192657 198812 199751 205093 PR: 209618 211248 205983 209338 databases/mysql56-server: Rollback rc script changes databases/mysql56-server: Do not install my.cnf sample An issue was discovered where users of mysql did not have a my.cnf and the recent update was causing mysqld to find a sample my.cnf and load its parameters. This was causing errors on startup for users of innodb as the parameters used to init the database did not match the ones in the sample config file it was now reading. databases/mysql56-server: Fix build with LibreSSL - Fix CMake SSL detection - Always set WITH_SSL=${OPENSSLBASE} PR: 216311 Approved by: Mahdi Moktari <mokhi64@gmail.com> (maintainer) Differential Revision: D9272 Revert r432035 part 2, it breaks build with openssl from base With hat: portmgr databases/mysql56-server: Fix OpenSSL linking - Force dynamic linking with OpenSSL Approved by: ports-secteam (feld) Changes: _U branches/2017Q1/ branches/2017Q1/databases/mysql56-client/Makefile branches/2017Q1/databases/mysql56-client/files/patch-CMakeLists.txt branches/2017Q1/databases/mysql56-client/files/patch-cmake_ssl.cmake branches/2017Q1/databases/mysql56-client/files/patch-extra_CMakeLists.txt branches/2017Q1/databases/mysql56-client/files/patch-man_CMakeLists.txt branches/2017Q1/databases/mysql56-client/files/patch-mysys_ssl_my_default.cc branches/2017Q1/databases/mysql56-client/files/patch-scripts_CMakeLists.txt branches/2017Q1/databases/mysql56-client/files/patch-support-files_CMakeLists.txt branches/2017Q1/databases/mysql56-client/pkg-message branches/2017Q1/databases/mysql56-server/Makefile branches/2017Q1/databases/mysql56-server/distinfo branches/2017Q1/databases/mysql56-server/files/my.cnf.sample.in branches/2017Q1/databases/mysql56-server/files/patch-cmake_ssl.cmake branches/2017Q1/databases/mysql56-server/files/patch-mysys_ssl_my_default.cc branches/2017Q1/databases/mysql56-server/pkg-message branches/2017Q1/databases/mysql56-server/pkg-plist