-
Notifications
You must be signed in to change notification settings - Fork 10
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
Build fails #12
Comments
Humm is that compiling with clang? And where dies cython come into the mix (we use cffi) |
All I'm doing is running the above commands to try to install cld2-cffi using pip. I'm not running cython or any other command. I'd be happy to test compiling with clang, but I don't know how that would work, since I've never used it. |
Can you send me
thanks |
.... Nevermind
That is a very very very new compiler, but let me see what can be done with it. |
So I am thinking the best solution to this is to get round to implementing #11, then you wont have to compile |
FWIW, here's
|
BTW, it does look like this is a compiler issue--I was able to successfully compile the cfd2-cffi with an old GCC, using |
I am guessing that GCC 6 is more strict with narrowed strings (notice
that is the error)
I dont have the time to look into it right now, but I will try to figure
out how to cope with gcc6.
On Tue, Dec 6, 2016, at 05:06 PM, Jonathan Reeve wrote:
BTW, it does look like this is a compiler issue--I was able to
successfully compile the cfd2-cffi with an old GCC, using sudo env CC=/usr/bin/gcc-
5 pip install -U cld2-cffi.
— You are receiving this because you commented. Reply to this email
directly, view it on GitHub[1], or mute the thread[2].
|
I just ran into this error with GCC 6 also. As a workaround I did a |
* add failing en specs * add cld2 gem * Replace WhatLanguage with CLD
The following worked for me: pip install --upgrade 'git+https://github.com/GregBowyer/cld2-cffi.git@d048c5fa98066bdb130f1c7e52641a0645aef5ac' gcc (Gentoo 8.3.0-r1 p1.1) 8.3.0 |
Inlines the function strcasecmp from strings.h, which is not present on Windows. This should close Issue GregBowyer#12 and PR GregBowyer#15.
Was going to use textpipe, but dependency was too large and failed to install with halfway modern GCC (due to CLD2 issue): GregBowyer/cld2-cffi#12 So instead basically pulled out the clean_text function, which is quite short.
I had the same installation issue. The fix in branch Fix-21 worked for me. |
When I try to install cld2-cffi via
sudo pip install cld2-cffi
orpip install -U cld2-cffi
, this is what happens:The text was updated successfully, but these errors were encountered: