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

Fixing error with installation on windows with cp1251 #1489

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

vmartyanov
Copy link

Faced with installation error on Windows, last lines in exception log:

File "C:\Users\...\AppData\Local\Temp\tmp87tpzffd\.venv\lib\site-packages\setuptools\_core_metadata.py", line 136, in write_pkg_info self.write_pkg_file(f) File "<string>", line 420, in _write_pkg_file File "C:\Users\...\AppData\Local\Temp\tmp87tpzffd\.venv\lib\site-packages\setuptools\_core_metadata.py", line 210, in write_pkg_file file.write("\n%s" % long_description) File "C:\Python310\lib\tempfile.py", line 483, in func_wrapper return func(*args, **kwargs) File "C:\Python310\lib\encodings\cp1251.py", line 19, in encode return codecs.charmap_encode(input,self.errors,encoding_table)[0] UnicodeEncodeError: 'charmap' codec can't encode character '\xe9' in position 20918: character maps to <undefined>

I realized that problem was in setup.py: long_description was read with utf-8, but written with local encoding which doesn't support diacritics from readme.md

@mrexodia
Copy link
Contributor

Looks like a good change to me!

@serpilliere serpilliere merged commit 3e22013 into cea-sec:master Jul 17, 2024
2 of 3 checks passed
@serpilliere
Copy link
Contributor

Thankt you for the fix @vmartyanov !

@serpilliere
Copy link
Contributor

and thank you for the review @mrexodia 😄

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

Successfully merging this pull request may close these issues.

3 participants