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

On debian testing, python 3.8.3rc1, version parsing fails. #33

Closed
bdauvergne opened this issue Jul 16, 2020 · 0 comments · Fixed by #34
Closed

On debian testing, python 3.8.3rc1, version parsing fails. #33

bdauvergne opened this issue Jul 16, 2020 · 0 comments · Fixed by #34
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@bdauvergne
Copy link

I get the following trace on any use of the library (using latest version from pypi)


tabularfile/csv.py:44: in __init__
    self.encoding = charset_normalizer.detect(header).get('encoding')
/tmp/tox-bdauvergne/tabularfile/py3-charsetnormalizer/lib/python3.8/site-packages/charset_normalizer/legacy.py:20: in detect
    r = CnM.from_bytes(byte_str).best().first()
/tmp/tox-bdauvergne/tabularfile/py3-charsetnormalizer/lib/python3.8/site-packages/charset_normalizer/normalizer.py:395: in from_bytes
    py_v = [int(el) for el in python_version_tuple()]
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

.0 = <tuple_iterator object at 0x7f51145cc8e0>

>   py_v = [int(el) for el in python_version_tuple()]
E   ValueError: invalid literal for int() with base 10: '3rc1'

Maybe you could simply use sys.version_info ?

@bdauvergne bdauvergne added bug Something isn't working help wanted Extra attention is needed labels Jul 16, 2020
@Ousret Ousret mentioned this issue Feb 8, 2021
@Ousret Ousret closed this as completed in #34 Feb 8, 2021
Ousret added a commit that referenced this issue Feb 8, 2021
* 🔧 Dependencies refactor and add py 3.9 and 3.10 to the supported list

* 🐛 Fix python 3.8.3rc1, version parsing fails #33

* 🔧 travis add py 3.8, 3.9, and 3.10 and remove 2.7

* 🔖 Bump to version 1.3.5

* 📝 Update readme.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Development

Successfully merging a pull request may close this issue.

1 participant