Use the nix path for mysql/mariadb 'basedir'. Fixes #2524 #2525
+8
−8
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
As described on #2524,
mysqld
are using the local/usr
forbasedir
(e.g. reading/usr/lib/mysql/plugins
if present), when it should be using the nix equivalent.How was it tested?
Warning: this is untested. I don't know how to recompile flake.nix, so I cheated and edited the
mysqld
wrapper withsudo vim $(which mysqld)
. This patch may be wrong, e.g. I'm not sure if$out
will be expanded in those single-quotes.But if the patch does what I hope, this is the effect it should have:
With this patch, the command above should print something like:
previously it would print:
Also, one can fully launch mysqld, connect with mysql and show the basedir with:
The basedir and related paths are now correctly pointing to
/nix/store
paths, and everything still works!