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

Compilation error LIBEVENT_INCLUDES-NOTFOUND #170

Closed
selcukaltinay opened this issue Jun 20, 2023 · 3 comments
Closed

Compilation error LIBEVENT_INCLUDES-NOTFOUND #170

selcukaltinay opened this issue Jun 20, 2023 · 3 comments

Comments

@selcukaltinay
Copy link

Hey everyone,

I'm taking an error when I try to compile hiredis-cluster with latest version of both hiredis and hircluster.

The compilation steps listed below;

mkdir build
cmake .. -Dhiredis_DIR:PATH=$HIREDISPATH

Then the error occurs like listed below;

CMake Error in tests/CMakeLists.txt:
Found relative path while evaluating include directories of "clusterclient_reconnect_async":
"LIBEVENT_INCLUDES-NOTFOUND"
I'm trying to use libevent library from /usr/lib64 and its already added to LD_LIBRARY_PATH.

I'm trying to fix another error. When I use the async apis for subscription, I'm taking crash. During publishing a message, the subscriber process giving a message like that redisProcessCallbacks: Assertion `(c->flags & 0x20 || c->flags & 0x40)' failed.

I assumed that the problem can be caused for this compilation error.
Thanks for answers :)

@bjosv
Copy link
Collaborator

bjosv commented Jun 20, 2023

Lets see if I can cover all your issues.

To avoid the LIBEVENT_INCLUDES-NOTFOUND error you can use -DDISABLE_TESTS=ON as an argument to cmake, see build instructions. This will skip tests which require libevent.

If I understand correctly you have a specific hiredis downloaded which you want to use.
Would this example help? Its an example of building hiredis and hiredis-cluster separate using CMake.

By default hiredis-cluster will download and build hiredis v1.1.0 in a CMake build, unless you disable that with -DDOWNLOAD_HIREDIS=OFF, see build options.

The assert seems to indicate that your build uses an old, possibly system-installed, hiredis.

@zuiderkwast
Copy link
Collaborator

Additionally, pubsub doesn't work in hiredis-cluster...

@bjosv
Copy link
Collaborator

bjosv commented Oct 30, 2023

Closing since the build instructions covers the issues.
The missing pubsub support in covered by #27.

@bjosv bjosv closed this as completed Oct 30, 2023
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

No branches or pull requests

3 participants