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

Failed to install cchardet #1946

Closed
chrisgoddard opened this issue Feb 24, 2024 · 7 comments
Closed

Failed to install cchardet #1946

chrisgoddard opened this issue Feb 24, 2024 · 7 comments
Labels
needs-mre Needs more information for reproduction

Comments

@chrisgoddard
Copy link

Error install cchardet==2.1.7

src/cchardet/_cchardet.cpp:196:12: fatal error: 'longintrepr.h' file not found
  #include "longintrepr.h"
           ^~~~~~~~~~~~~~~
1 error generated.
error: command '/usr/bin/clang' failed with exit code 1```

Mac (intel) OSX 14.2.1 - though I had the same issue on my M1.
@charliermarsh
Copy link
Member

Can you please include your uv version, the output of running uv pip install with --verbose, and confirmation that you're able to install this package with pip and --no-cache?

@charliermarsh charliermarsh added the needs-mre Needs more information for reproduction label Feb 24, 2024
@hwong557
Copy link

hwong557 commented Feb 24, 2024

Are you on 3.11? It seems cchardet is unsupported on 3.11 and the package seems to be abandoned.

PyYoshi/cChardet#81

But there seems to be a fork: https://github.com/faust-streaming/cChardet

@chrisgoddard
Copy link
Author

Oh weird - 3.12 actually. Normal pip works but I'll check out the fork

@hauntsaninja
Copy link
Contributor

Yeah, cchardet is dead. Fwiw, seems like a lot of people are switching to charset_normalizer anyway

@chrisgoddard
Copy link
Author

I'll check that out - quite a few scraping libs have it as an optional dependency for performance.

@notatallshaw-gts
Copy link

I'll check that out - quite a few scraping libs have it as an optional dependency for performance.

FYI there is a mypyc mode for charset-normalizer for some performance improvement: https://charset-normalizer.readthedocs.io/en/latest/community/speedup.html

@zanieb
Copy link
Member

zanieb commented Feb 26, 2024

It looks like there's nothing for us to address here. (please let us know if that's not the case)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-mre Needs more information for reproduction
Projects
None yet
Development

No branches or pull requests

6 participants