-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix ABI compability when linking against OCCI when using clang. #460
Conversation
Build started |
@phsft-bot build just on slc6/clang_gcc62 with flags -Dcxxmodules=On -Doracle=On -Dccache=Off -DVc=Off |
Build started |
sql/oracle/CMakeLists.txt
Outdated
# Check if libstdc++ uses the new string ABI and fall back to the old one if yes. | ||
# We have to check for this because on the nightly builds we have an old occi | ||
# installation that uses the old ABI, which results in linking errors. | ||
CHECK_CXX_SOURCE_COMPILES( |
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.
I think, it would make more sense to move this check in CheckCompiler.cmake
be50f1f
to
c1a4afe
Compare
Starting build on |
@phsft-bot build also on slc6/clang_gcc62 with flags -Dcxxmodules=On -Doracle=On -Dccache=Off |
Starting build on |
This patch improves the previous check whether the current compiler is using an incompatible libstdc++ version or not.
c1a4afe
to
ce7a90f
Compare
@phsft-bot build also on slc6/clang_gcc62 with flags -Dcxxmodules=On -Doracle=On -Dccache=Off |
Starting build on |
It seems that the failures are not related to the PR. Merging. |
…-project#460) This patch improves the previous check whether the current compiler is using an incompatible libstdc++ version or not.
This patch improves the previous check whether the current compiler
is using an incompatible libstdc++ version or not.