-
Notifications
You must be signed in to change notification settings - Fork 132
Compiler errors switching from GCC to Clang #29
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
Comments
Those errors are indeed odd. It seems like there is possibly a conflict with the ndk version you have, though that is admittedly just a guess. Can you check which version of the ndk you have (you can check /opt/android-sdk/current/ndk-bundle/CHANGELOG.md, which should list it near the top). Also, have you tried out one of the Quickstart testapps we provide at https://github.com/firebase/quickstart-cpp? They are relatively fast to set up, and recently verified, so it'd be interesting to know if you encounter a similar problem with them. |
@a-maurice the thing is that there is no ndk installed in |
It looks like errors coming from the closed source binary lib blobs - they usually have those weird paths. I'm guessing they show the paths from the container they were compiled on. We (QtFirebase project) haven't tried Clang yet - but it look like it's setup related indeed |
I came across a similar problem. It turned out that in QtFirebase in one of the .pri file it is hardcoded to use the "gnustl" versions of the firebase libraries from the firebase sdk. Changing that "gnustl" to "c++" made things work using clang. To make things a litle bit more flexible I added an extra variable (QTFIREBASE_STL_VARIANT) so that the STL variant to be used can be specified in the app main .pro file (a bit like the QTFIREBASE_SDK_PATH variable). Also issued a pull request for this tiny change, but just wanted to mention it already here as it might be usefull Update: putting the variable in the main .pro file does not always work, depending on the order in which the .pro files are processed. Changing in the location where the original string was does give the expected results! |
Perfect catch! |
I think - and hope we nailed it with a7f8247 |
So I have a Qt project in which I incorporated Firebase C++ using the wrapper here https://github.com/Larpon/QtFirebase. I decided to switch my compiler from GCC (which was working ok) to Clang as Google deprecated the former, but now I am getting these strange errors:
I have tried to add this in my
build.gradle
but it doesn't work:The text was updated successfully, but these errors were encountered: