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

Fix nix devShell warning #4564

Closed
wants to merge 1 commit into from
Closed

Conversation

RCoeurjoly
Copy link
Contributor

Fix "bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UT…F-8)" warning on nix devShell

What are the reasons/motivation for this change?
Quality of life.
Explain how this is achieved.
We set the locale correctly in the nix devShell.
If applicable, please suggest to reviewers how they can test the change.

@widlarizer
Copy link
Collaborator

Can you explain the problem some more and point to standard nix practice?

@RCoeurjoly
Copy link
Contributor Author

Can you explain the problem some more and point to standard nix practice?

After reconsidering, this issue is better handled locally by each user setting LOCALE_ARCHIVE with nix profile, so there's no need for a global change. I've added documentation on how to set this with nix profile (since nix-env is deprecated): https://wiki.nixos.org/wiki/Locales.

Closing the PR.

@RCoeurjoly RCoeurjoly closed this Sep 30, 2024
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