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

1.30 on linux picks wrong locale #65037

Closed
gudvinr opened this issue Dec 13, 2018 · 3 comments
Closed

1.30 on linux picks wrong locale #65037

gudvinr opened this issue Dec 13, 2018 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal Integrated terminal issues verified Verification succeeded

Comments

@gudvinr
Copy link

gudvinr commented Dec 13, 2018

  • VSCode Version: 1.30
  • OS Version: Linux 4.19.8-arch1-1-ARCH x86_64; KDE 5.14

Steps to Reproduce:

  1. Open any other terminal running bash (Konsole, Terminator) and see this:
$ locale
LANG=C
LC_CTYPE="C"
LC_NUMERIC=C
LC_TIME=C
LC_COLLATE=C
LC_MONETARY=ru_RU.UTF-8
LC_MESSAGES="C"
LC_PAPER="C"
LC_NAME="C"
LC_ADDRESS="C"
LC_TELEPHONE="C"
LC_MEASUREMENT=C
LC_IDENTIFICATION="C"
LC_ALL=
$ ls /not
ls: cannot access '/not': No such file or directory
  1. Open integrated terminal in VSCode and you'll see that:
$ locale
LANG=en_US.UTF-8
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC=C
LC_TIME=C
LC_COLLATE=C
LC_MONETARY=ru_RU.UTF-8
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT=C
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=
$ ls /not
ls: невозможно получить доступ к '/not': Нет такого файла или каталога

I have terminal.integrated.setLocaleVariables set to false. I'm not sure if it became a thing in 1.30 but probably yes.

Does this issue occur when all extensions are disabled?: Yes (And I have zero extensions related to locales).

@vscodebot vscodebot bot added the terminal Integrated terminal issues label Dec 13, 2018
@Tyriar Tyriar added this to the January 2019 milestone Dec 14, 2018
@Tyriar Tyriar added the bug Issue identified by VS Code Team member as probable bug label Dec 14, 2018
@Tyriar Tyriar closed this as completed in ece3a90 Dec 14, 2018
@Tyriar
Copy link
Member

Tyriar commented Dec 14, 2018

I made a change to make the default true to all platforms and removing the fallback when setLocaleVariables is false, so setting it to terminal.integrated.setLocaleVariables set to false should leave your setup alone.

If you could test it out on the Insider's build tomorrow and let me know how it works that would be helpful 😃 https://code.visualstudio.com/insiders

@gudvinr
Copy link
Author

gudvinr commented Dec 14, 2018

@Tyriar
Yep, that works.
Hope to see that in 1.30.1

@vscodebot vscodebot bot locked and limited conversation to collaborators Jan 28, 2019
@dbaeumer
Copy link
Member

dbaeumer commented Feb 1, 2019

Marking as verified based on comment #65037 (comment)

@dbaeumer dbaeumer added the verified Verification succeeded label Feb 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug terminal Integrated terminal issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants