Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

setup.py: Fallback to use the system hiredis library. #161

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Apteryks
Copy link
Contributor

Fixes #158 fully, including using a system-provided hiredis.

When the hiredis git submodule hasn't been initialized, print a message about it, and attempt to link against the a system-provided hiredis library instead.

  • setup.py (is_hiredis_bundled): New procedure. (get_hiredis_bundled_sources): Likewise. Print a message when bundled_hiredis_sources is empty.
    (get_sources): Adjust to use the above procedure.
    (get_linker_args): Add -lhiredis when the bundled hiredis is not used.

Fixes redis#158 fully, including using a system-provided hiredis.

When the hiredis git submodule hasn't been initialized, print a
message about it, and attempt to link against the a system-provided
hiredis library instead.

* setup.py (is_hiredis_bundled): New procedure.
(get_hiredis_bundled_sources): Likewise.  Print a message when
bundled_hiredis_sources is empty.
(get_sources): Adjust to use the above procedure.
(get_linker_args): Add -lhiredis when the bundled hiredis is not used.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sdsalloc.h doesn't seem to be installed as part of regular hiredis
2 participants