Did you have issues compiling?

Iirc, we tested to get this solution and at least the deb packages built fine for the past months since this change, after having couple of builds failed back then.

What it tries to do is that if pkg-config for cflags is not giving a path that has hiredis in it, then include <hiredis/hiredis.h>. If what pkg-config returns has hiredis, then just include <hiredis.h>.

I got on different systems and versions, either (1):

-I/opt/local/include/hiredis

or (2):

-I/opt/local/include

So for (1), include <hiredis.h> and for (2), include <hiredis/hiredis.h>.

Am I wrong in what is doing it right now?

Maybe there is something else to fix, but I find the current behaviour as expected.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.