Skip to content

Add MySQL port, fixes #4#5

Open
lumbric wants to merge 2 commits intodanapsimer:masterfrom
lumbric:master
Open

Add MySQL port, fixes #4#5
lumbric wants to merge 2 commits intodanapsimer:masterfrom
lumbric:master

Conversation

@lumbric
Copy link
Copy Markdown

@lumbric lumbric commented Aug 20, 2017

I don't undestand why this ever worked before, but this fix seems to work for me. Also it is a nice thing to be able to select port via argument on CLI.

@lumbric lumbric mentioned this pull request Aug 20, 2017
@skwashd
Copy link
Copy Markdown

skwashd commented Jun 4, 2018

While this fixes #4 it also contains a commit from #6. I have proposed #10 that fixes the port issue. To keep it consistent with the other values being concatenated into the string I used a pointer.

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.

2 participants