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

python: set file encoding #126

Merged
merged 1 commit into from
Jul 29, 2022
Merged

python: set file encoding #126

merged 1 commit into from
Jul 29, 2022

Conversation

fargies
Copy link

@fargies fargies commented Feb 28, 2022

  • on some containers LC_ALL is not set, python fallback on ascii
    encoder, which fails on AUTHORS file

@lemire
Copy link
Member

lemire commented Mar 1, 2022

Looks reasonable. There appears to be a strange VS16 error that is probably unrelated.

@fargies
Copy link
Author

fargies commented Mar 1, 2022

Yes, maybe a shortage of builders on Github side, not sure.

I've pushed a similar PR on biojppm/cmake#6

@lemire
Copy link
Member

lemire commented Mar 3, 2022

Please sync your fork with our main branch, it should make the error go away.

- on some containers LC_ALL is not set, python fallsback on ascii
  encoder, which fails on AUTHORS file
@lemire lemire merged commit 6283cf7 into fastfloat:main Jul 29, 2022
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.

2 participants