Skip to content
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

Allow rocksdb library to be usable with CMake's FetchContent API #11575

Closed
wants to merge 2 commits into from
Closed

Allow rocksdb library to be usable with CMake's FetchContent API #11575

wants to merge 2 commits into from

Conversation

Mis1eader-dev
Copy link
Contributor

This adds proper support for using rocksdb with FetchContent, without this PR the user must include the following with their own CMakeLists.txt file:

include_directories(./build/_deps/rocksdb-src/include)

Copy link
Contributor

@ajkr ajkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@facebook-github-bot
Copy link
Contributor

@ajkr has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@ajkr merged this pull request in 977aae5.

rockeet pushed a commit to topling/toplingdb that referenced this pull request Dec 18, 2023
…11575)

Summary:
This adds proper support for using rocksdb with FetchContent, without this PR the user must include the following with their own `CMakeLists.txt` file:
```cmake
include_directories(./build/_deps/rocksdb-src/include)
```

Pull Request resolved: facebook/rocksdb#11575

Reviewed By: jowlyzhang

Differential Revision: D47163520

Pulled By: ajkr

fbshipit-source-id: a202dcf435ecc9dd8d51c88f90e98c04814721ca
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants