hiredis
hiredis copied to clipboard
Fix cmake config path on Linux.
CMake config files were installed to /usr/local/share/hiredis
, which is not recognizable by find_package()
.
I'm not sure why it was set that way.
Given the commit introducing it is for Windows, I keep that behavior consistent there, but fix the rest.
I'm pretty unfamiliar with CMake but this does appear correct. I'll read up a bit on CMake's find_package
and then get it merged.
Emmm....Actually after double checking I might be wrong. /usr/local/share/hiredis
could be supported as well. Originally I saw it after packaging into rpm and think that doesn't look right, probably because rarely anyone uses that.
![image](https://user-images.githubusercontent.com/5203025/136160026-1c4cdf6e-1902-454e-b162-1d9ca05f54ef.png)
So can this be closed then?
Up to you, but I would recommend to merge. This PR fixed a "package not found" issue, although I'm not sure about the root cause because either way should be OK based on the doc.
Merged, thanks!
@chayim Thanks for the help!