Skip to content

Conversation

@devbharat
Copy link

Instead of relying it to be installed on system.

@david-drinn
Copy link

david-drinn commented Oct 25, 2023

Would like to see this updated with the latest FetchContent features, specifically OVERRIDE_FIND_PACKAGE option in FetchContent_Declare(), and calling FetchContent_MakeAvailable(hiredis). I think it would simplify the changes significantly.

message(STATUS "redis-plus-plus build type: ${CMAKE_BUILD_TYPE}")

set(REDIS_PLUS_PLUS_DEFAULT_CXX_STANDARD 17)
set(REDIS_PLUS_PLUS_DEFAULT_CXX_STANDARD 11)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why ?

@sewenew
Copy link
Owner

sewenew commented Jun 11, 2025

FetchContent feature has been supported with #639

Regards

@sewenew sewenew closed this Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants