Bug 280841 - databases/mariadb1011-server: remove the --db_dir from rc.script
Summary: databases/mariadb1011-server: remove the --db_dir from rc.script
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Bernard Spil
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-08-16 02:29 UTC by pradeep.michchi
Modified: 2024-08-18 17:07 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (brnrd)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description pradeep.michchi 2024-08-16 02:29:37 UTC 
Why rc.script using the dbdir in startup command line when this is specified in my.cnf (server.cnf)?.

I suggest to remove the --db_dir from rc.script to avoid confusion/conflict with my.cnf setting as per mariadb recommendations.
Comment 1 Bernard Spil freebsd_committer freebsd_triage 2024-08-18 12:03:39 UTC 
Can't really do that because of the _instance capability (running multiple mysql instances in parallel from rc-script).

Will comment out in the config-file instead. Update to land shortly with 10.11.9
Comment 2 commit-hook freebsd_committer freebsd_triage 2024-08-18 12:31:53 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=91f311910cae4e5d87db8cdf202c918ba57ca437

commit 91f311910cae4e5d87db8cdf202c918ba57ca437
Author:     Bernard Spil <brnrd@FreeBSD.org>
AuthorDate: 2024-08-18 12:28:53 +0000
Commit:     Bernard Spil <brnrd@FreeBSD.org>
CommitDate: 2024-08-18 12:28:53 +0000

    databases/mariadb1011-server: Update to 10.11.9

    * Don't set db_dir twice [1]
    * Fix build with ports LibreSSL [2]

    PR:             280841 [1], 274494 [2]
    Reported by:    <pradeep michchi gmail com> [1], Ivan Rozhuk <rozhuk im gmail com> [2]

 databases/mariadb1011-server/Makefile              |  3 +-
 databases/mariadb1011-server/distinfo              |  6 +-
 .../mariadb1011-server/files/patch-PR279362 (gone) | 85 ----------------------
 .../files/patch-mysys__ssl_openssl.c               | 15 ++++
 .../mariadb1011-server/files/server.cnf.sample.in  |  2 +-
 5 files changed, 20 insertions(+), 91 deletions(-)