feat mongo: add CMake support for mongo-c-driver 2.x.x#1175
Conversation
botanegg
force-pushed
the
userver-mongo-cmake-support-mongoc2
branch
3 times, most recently
from
5227387 to
0d61ce8
Compare
| # prefer static libs here for compat | ||
| # https://github.com/mongodb/mongo-c-driver/blob/2.0.0/src/libmongoc/etc/mongocConfig.cmake.in#L7-L12 | ||
| # mongoc::mongoc is mongoc::static first then mongoc::shared. Same for bson. | ||
| set(USERVER_MONGO_LIBMONGOC_NAME mongoc::mongoc) |
There was a problem hiding this comment.
Не будет ли луяше сделать тут интерфейсные библиотеки mongo::mongoc_static и mongo::bson_static, и не трогать тогда файл mongo/CMakeLists.txt ?
There was a problem hiding this comment.
Не оч понял, они и есть, если искать через v2 подход
Просто для них есть универсальный (на стороне либки) таргет, который делает то, что нужно (берет в приоритете статику, если статики нет, берет soшку)
А все трогание задумано, чтобы была совместимость между v1 и v2 и возможность сфоллбэчится на шаред либку для v2
There was a problem hiding this comment.
Зачем появилась надобность:
у нас на archlinux, нету .a (и вообще поломан cmake для mongo-c-driver)
мейнтейнеры арчи не разрешают добавить
https://gitlab.archlinux.org/archlinux/packaging/packages/mongo-c-driver/-/work_items/4
Чтобы запускать юсервер с монгой нативно (удобно разрабатываться) - нужен фолбэк на шаред версию :(
botanegg
force-pushed
the
userver-mongo-cmake-support-mongoc2
branch
from
0d61ce8 to
a10825c
Compare
|
LGTM |
|
Many thanks for the PR! @apolukhin is now importing your pull request into our internal upstream repository. |
|
✅ This pull request is being closed because it has been successfully merged into our internal monorepository. |
Tests: протестировано CI --- Pull Request resolved: #1175 commit_hash:88404055d44e6643ddc912434617f36d8f288854
2 participants
Note: by creating a PR or an issue you automatically agree to the CLA. See CONTRIBUTING.md. Feel free to remove this note, the agreement holds.