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

Please update dependencies; cchardet is abandoned; build on Python 3.11 currently impossible #1651

Closed
HWiese1980 opened this issue Feb 26, 2023 · 5 comments

Comments

@HWiese1980
Copy link

HWiese1980 commented Feb 26, 2023

Original issue

aiohttp 3.8.1 has issues with Python 3.11, cannot be built. aiohttp 3.8.2 works.

Building aiohttp 3.8.1 with Python 3.11 results in

yarl/_quoting_c.c:198:12: fatal error: 'longintrepr.h' file not found
        #include "longintrepr.h"
                 ^~~~~~~~~~~~~~~

Update

cchardet is also affected by this error. And contrary to aiohttp it cannot be updated beyond its current state as it seems to be abandoned.

Update 2

https://github.com/Ousret/charset_normalizer is named as alternative

@frenck
Copy link
Contributor

frenck commented Feb 26, 2023

Please upgrade beyond that... 3.8.2 has memory leaks when used in conjunction with SSL

@HWiese1980
Copy link
Author

HWiese1980 commented Feb 26, 2023

cchardet 2.1.7 has the same issue; unfortunately it's the latest version available

Update: cchardet appears to be abandoned... PyYoshi/cChardet#77

@HWiese1980 HWiese1980 changed the title Please update aiohttp==3.8.2 Please update dependencies; cchardet is abandoned; build on Python 3.11 currently impossible Feb 26, 2023
@acockburn
Copy link
Member

Dev currently has aiohttp==3.8.4, I'll be putting out a release soon.

AFAIK, cchardet isn't a specific dependency but I'll check into why we need it and ensure the next release supports python 3.11.

@HWiese1980
Copy link
Author

Awesome! For a local install that I needed for code completion in Visual Studio Code, it was already enough to remove cchardet from the requirements.txt. But I don't know if appdaemon would actually run like that. Haven't tested it.

@acockburn
Copy link
Member

acockburn commented Feb 26, 2023 via email

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

No branches or pull requests

3 participants