-
Notifications
You must be signed in to change notification settings - Fork 481
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
fatal error: 'mysql.h' file not found #1152
Comments
I have the same problem trying to build xeus-sql 0.2.1 with soci 4.0.3
|
This seems rather different because I'd expect You will need to check how/where does CMake find this header. |
Yes, you're right. I don't know how this instruction is coming for you and not for me. |
This is likely an issue with the SOCI-provided In #1118 I have improved the find-module. |
Hi @Krzmbrzl Build instruction was:
|
Another problem is that /usr/lib64/libsoci_mysql.so is the library instead of being a link to the library, which have to be /usr/lib64/libsoci_mysql.so.4.1.0 |
why is this a problem?
Yeah, I can reproduce that. It seems that this only affects MySQL though - I get
so for the other backends things seem to work as expected. I'll look into that |
Okay that was easy. It's fixed already:
(not that I force-pushed my PR branch, so you'd have to do a hard reset to get the latest changes without merge conflicts) |
Fine, this is fixed. |
I don't know yet, but I wonder if the change is wanted, and why. |
Oh, right. 👀
I made this choice deliberately because I think it's more organized than every project just blindly installing into |
After applying your last fix, I think not all header files are installed. This was already the case before the fix.
I had previously header files in |
Indeed. I missed the |
I have now these header files: /usr/include/soci/mysql/soci-mysql.h
/usr/include/soci/odbc/soci-odbc.h
/usr/include/soci/postgresql/soci-postgresql.h
/usr/include/soci/sqlite3/soci-sqlite3.h The bad news is that xeus-sql doesn't recognize SOCI.
|
Now, I have an argument for the "why not": The change breaks the detection of |
And I still have this: |
That is addressed in a different PR
You shouldn't be using a |
But Lines 47 to 98 in 6b6db13
I will try that. |
Yes, because it is currently a private header (which it shouldn't be). I'll add it once the PR putting it into the private includes is merged. |
OK, I removed FindSOCI, used
Note that previously package to find was |
Hm. Maybe we have to install the FindMySQL module along with SOCI in order for it to find that 🤔
Yeah, SOCI is the correct name and case-sensitive filesystems this becomes important. This was an oversight of the previous cmake implementation. |
Hello,
MySQL is said found, but the variable MySQL_FOUND is false or not set. |
clone soci into
/Users/alexey/CLionProjects/untitled1/lib/soci
with terminalgit clone
and write a CMakeLists.txt as inexample/subdir-include/CMakeLists.txt
Here is my CMakeLists.txt:
Here is my error:
The text was updated successfully, but these errors were encountered: