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

Review default LC_ALL setting #507

Closed
hadley opened this issue Jun 26, 2014 · 2 comments
Closed

Review default LC_ALL setting #507

hadley opened this issue Jun 26, 2014 · 2 comments

Comments

@hadley
Copy link
Member

@hadley hadley commented Jun 26, 2014

Now that all cran check machines run in UTF-8.

@yihui
Copy link
Contributor

@yihui yihui commented Jun 26, 2014

I did not know devtools was forcing the locale to be C, which is a bad idea. I believe it makes a lot of sense to remove this restriction. The locale can be temporarily set to C in a few cases to ensure consistency, such as sorting, but in general, the C locale will make R unable to read/write Unicode characters correctly.

@hadley
Copy link
Member Author

@hadley hadley commented Jun 26, 2014

I made it C a long time ago because that's what CRAN used to use - and I had a horrible problem that took hours to resolve. But I now think that time has passed :)

@lock lock bot locked and limited conversation to collaborators Sep 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants