-
Notifications
You must be signed in to change notification settings - Fork 13
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
Issue #25 Abstract mysql types. #33
Conversation
@@ -0,0 +1 @@ | |||
swift build -Xlinker -L/usr/local/lib -Xlinker -lmysqlclient -Xcc -I/usr/include/** |
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.
Why is this file and .swift-test-linux
not the same? They have a different include path (/usr/include/**
vs /usr/local/include/
)
These files are a Package-Builder thing, not an SPM thing, right? Is there an SPM mechanism that relieves normal users from having to include the options?
osx_image: xcode9.4 | ||
sudo: required | ||
env: | ||
- SWIFT_SNAPSHOT=swift-4.2-DEVELOPMENT-SNAPSHOT-2018-06-04-a MYSQL_VER=5 |
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.
Now that Travis has an xcode10
image I think we can move up to a newer 4.2 snapshot
int wrapper_mysql_library_init(int argc, char **argv, char **groups) | ||
{ | ||
return mysql_library_init(0, NULL, NULL); | ||
} |
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.
Do we need to wrapper every mysql function? Can we not just wrapper the few where a conversion needs to happen?
This has been superseded by pull request 36 |
This PR adds a c wrapper which provides type consistency irrespective of the level of mysql plugin.