Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removing --tmpdir argument from mysql_install_db as it's used in mysqld #647

Closed
wants to merge 2 commits into from

Conversation

@vorburger
Copy link
Collaborator

Resolving #632 and #634

That would #632 and #646 (#634 was just a dependabot)...

... I will test this by creating a PR which combines this with #638!

@MazenAmria Thank you for your further investigation of this issue, and for having raised this PR.

@vorburger
Copy link
Collaborator

@MazenAmria given that #649 failed, I am (have to be) reluctant to accept and merge this as-is - hope that makes sense?

Would you be interesting in contributing further and help figure out what's going on? /Cc @jejanim

PS: To be clear, I AM very happy to merge this IFF I can "verify" that it fully fixes #632 "via" #638.

@MazenAmria
Copy link
Contributor Author

@vorburger yeah that's totally fine
btw, I'm looking into the problem right now, and I'll be happy to help.

@TareqK
Copy link

TareqK commented Dec 10, 2022

Dying because of this as well

@vorburger
Copy link
Collaborator

Sorry for the long silence on this. I've fixed a number of others things in this project today, and am hoping to find time another weekend to upgrade the project overall from Java 11 to Java 17 and then be able to pick this PR up....

PS: If you use MariaDB4j professionally, please consider either taking a Tidelift subscription or GitHub Sponsor to support future maintenance!

@vorburger vorburger mentioned this pull request Mar 6, 2023
@vorburger
Copy link
Collaborator

@MazenAmria I have FINALLY (just now) merged your work here as part of #649 as vorburger@60d1d62 (with you still credited as the Author of it).! FYI @TareqK. Therefore closing this PR now. Many many thanks again for this contribution!

@vorburger vorburger closed this Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants