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
Abstract MySQL types #36
Conversation
else | ||
export DEBIAN_FRONTEND="noninteractive" | ||
if [[ $MYSQL_VER == 5 ]]; then | ||
mysql --version || { apt-get update && apt-get install -y pkg-config mysql-server libmysqlclient-dev && service mysql start && mysql --version; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this going to break once Ubuntu starts installing v8 by default? (or do we just fix it if and when that happens)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code changes look good, and I can see they work on macOS. There's a wrinkle to be worked out with the travis for mysql v8 on Ubuntu.
This PR abstracts the mysql data-types that changed between v5 and v8.
It also updates to use the pkgConfig functionality to remove the need for specifying explicit paths to headers and link libraries.