Closed panlei-coder closed 1 month ago
整体更改涉及修改CMake文件以更新第三方依赖项的路径和配置。主要包括将ADDRESS_SANITIZER
默认设置从ON
改为OFF
,引入新的snappy.cmake
文件,更新多个库的路径配置,以及更改LevelDB
的管理方式。
文件 | 更改摘要 |
---|---|
CMakeLists.txt |
将ADDRESS_SANITIZER 默认设置从ON 改为OFF ,并引入snappy.cmake 文件。 |
cmake/braft.cmake 和 cmake/brpc.cmake |
更新prefix_path 变量以包含不同的leveldb 安装目录。 |
cmake/gflags.cmake |
将GFLAGS_BUILD_gflags_nothreads_LIB 标志从OFF 改为ON 。 |
cmake/leveldb.cmake |
从使用FETCHCONTENT 改为ExternalProject_Add 来管理LevelDB依赖,并设置相应路径和属性。 |
cmake/snappy.cmake |
新增文件,处理Snappy库的外部项目配置。 |
src/CMakeLists.txt |
调整TARGET_LINK_LIBRARIES 调用的格式,删除不必要的空格。 |
在代码的世界里,变化如风吹, 提供新的路径,新的库相随。 Snappy来了,依赖变更, LevelDB配置,简洁又清晰。 兔子欢庆,代码更迭, 效率提升,欢呼声不息。
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@CodiumAI-Agent /improve
@CodiumAI-Agent /review
⏱️ Estimated effort to review [1-5] | 3 |
🧪 Relevant tests | No |
🔒 Security concerns | No |
⚡ Key issues to review |
Dependency Management: The PR modifies the way dependencies are managed and built, specifically for leveldb and snappy. Ensure that these changes do not break the build process or create conflicts with other dependencies. |
Build Configuration: The changes in CMake configuration for multiple components (leveldb, snappy, gflags) need careful review to ensure they are correctly set up for different build environments and do not introduce any build-time or runtime errors. |
Add the snappy library and change leveldb to the specified version.
Summary by CodeRabbit
ADDRESS_SANITIZER
从ON
改为OFF
,并引入snappy
库。leveldb
安装路径配置。gflags
库的编译配置,启用gflags_nothreads
选项。leveldb
依赖管理方式,改用ExternalProject_Add
。snappy
库的 CMake 配置文件,设置路径和依赖项。TARGET_LINK_LIBRARIES
调用的格式。