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

use hashkit library provided by libmemcached #313

Closed
wants to merge 1 commit into from

Conversation

p-alik
Copy link
Collaborator

@p-alik p-alik commented Nov 22, 2020

Remove libhashkit subdirectory and use instead
officially by libmemcached provided hashkit
http://docs.libmemcached.org/hashkit_functions.html

See #293 (comment)

Remove libhashkit subdirectory and use instead
officially by libmemcached provided hashkit
http://docs.libmemcached.org/hashkit_functions.html

See discussion in issue
gearman#293 (comment)
@esabol
Copy link
Member

esabol commented Nov 22, 2020

I'm good with this as long as none of the symbols in libgearman.so have changed. Did you check that, @p-alik?

@esabol
Copy link
Member

esabol commented Nov 23, 2020

Well, Travis CI shows build failures for gcc/g++ 4.8.5 and 4.9.x on Trusty (Ubuntu 14.04.6 LTS), unfortunately. Maybe sudo apt-get install libmemcached-dev (or sudo apt-get install libhashkit-dev?) on Trusty only in .travis.yml to fix those failures?

But that does prompt the question: Do we really want gearmand to be dependent on libmemcached?

@p-alik
Copy link
Collaborator Author

p-alik commented Nov 23, 2020

I'm good with this as long as none of the symbols in libgearman.so have changed. Did you check that, @p-alik?

No, I didn't

@p-alik
Copy link
Collaborator Author

p-alik commented Nov 23, 2020

apt-get install libhashkit-dev doesn't help to solve the Issue with gcc 4.8 and 4.9

@esabol
Copy link
Member

esabol commented Nov 23, 2020

apt-get install libhashkit-dev doesn't help to solve the Issue with gcc 4.8 and 4.9

I suggest trying apt-get install libmemcached-dev as well.

@p-alik
Copy link
Collaborator Author

p-alik commented Nov 24, 2020

I suggest trying apt-get install libmemcached-dev as well.

It won't help because libmemcached-dev doesn't matter to the issue and elder libhashkit doesn't provide libhashkit_murmur3.

$ cat /etc/issue
Ubuntu 14.04.6 LTS \n \l

$ grep -r libhashkit_murmur /usr/include/libhashkit* /usr/include/libmemcached*
/usr/include/libhashkit-1.0/algorithm.h:uint32_t libhashkit_murmur(const char *key, size_t key_length);
$ dpkg -S /usr/include/libhashkit-1.0/algorithm.h 
libhashkit-dev: /usr/include/libhashkit-1.0/algorithm.h
$ dpkg -l libhashkit-dev
Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name                                          Version                     Architecture                Description
+++-=============================================-===========================-===========================-===============================================================================================
ii  libhashkit-dev                                1.0.8-1ubuntu2              amd64                       libmemcached hashing functions and algorithms (development files)

@SpamapS
Copy link
Member

SpamapS commented Sep 23, 2024

This is pretty old. Happy to revisit if it makes sense.

@SpamapS SpamapS closed this Sep 23, 2024
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.

3 participants