-
Notifications
You must be signed in to change notification settings - Fork 77
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
Unsupported leveldb 1.21 on MacOS #95
Comments
could you try adding Lines 18 to 19 in 1a04cfb
and then try building plyvel from source? (typing |
or better yet, try out the branch from #91 |
pip3 install git+https://github.com/ryandesign/[email protected] ` clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/System/Library/Frameworks/Tk.framework/Versions/8.5/Headers -I/usr/local/include -I/usr/local/opt/openssl/include -I/usr/local/opt/sqlite/include -I/usr/local/Cellar/python/3.7.3/Frameworks/Python.framework/Versions/3.7/include/python3.7m -c plyvel/_plyvel.cpp -o build/temp.macosx-10.13-x86_64-3.7/plyvel/_plyvel.o -Wall -g -x c++
Command "/usr/local/opt/python/bin/python3.7 -u -c "import setuptools, tokenize;file='/private/var/folders/44/qt_sltss2ld3mr9dtfhhd21r0000gn/T/pip-req-build-kd6mc_t4/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /private/var/folders/44/qt_sltss2ld3mr9dtfhhd21r0000gn/T/pip-record-lj52dzzg/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /private/var/folders/44/qt_sltss2ld3mr9dtfhhd21r0000gn/T/pip-req-build-kd6mc_t4/` |
to downgrade leveldb use: |
@fenidik please make a normal checkout and follow the build steps as described in the developer/contributor docs. there is an extra cython build step for git checkouts. |
@wbolster I also had this issue installing from PyPi. I added the switch you indicated, built locally and installed, and it works fine. |
I should clarify - by works fine, I mean "import plyvel" now returns without error. It used to throw errors about a missing dylib. |
@kyuupichan thanks. re the checkout, did you actually use the branch from #91 or current master? |
I just used master - simply git cloned it. |
ok so that doesn't add much then. it's #91 that supposedly fixes this (i don't develop on osx myself) |
@wbsolster I can confirm that #91, but only with an extra
I can also confirm that keeping the platform code and adding |
Save my life! |
Hello guys.
I had an issue with plyvel and leveldb on macos. It was about installing plyvel.
The issue was similar to the #80.
However, I discovered that leveldb was updated from 1.20 (build from 2017) to the newest 1.21 (https://github.com/google/leveldb/releases/tag/1.21 - 10 days ago).
So I tried to downgrade leveldb to 1.20 and it works now, I assume that the newest version is not compatible with plyvel on macos.
Installation log:
The text was updated successfully, but these errors were encountered: