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

Use os-locale to define $LANG in the terminal #605

Closed
wants to merge 1 commit into from
Closed

Conversation

MrRio
Copy link
Contributor

@MrRio MrRio commented Aug 10, 2016

No description provided.

@MrRio MrRio force-pushed the os-locale branch 3 times, most recently from b0d2524 to d46e731 Compare August 10, 2016 16:04
@MrRio
Copy link
Contributor Author

MrRio commented Aug 10, 2016

This isn't the whole picture unfortunately... will update PR with more details soon

@MrRio MrRio added the 👩‍🔬 Status: In Progress Issue or PR is currently active and work is in progress label Aug 10, 2016
@timothyis
Copy link
Contributor

timothyis commented Aug 29, 2016

I think using os-locale might, for now, fix problems for a few users even if this isn't the whole issue. Should we merge this in in that case?

Ping @MrRio

@rauchg
Copy link
Member

rauchg commented Aug 29, 2016

LGTM!

@timothyis timothyis added Status: Rebase Needed and removed 👩‍🔬 Status: In Progress Issue or PR is currently active and work is in progress labels Aug 29, 2016
@lordgiotto
Copy link
Contributor

I think os-locale is not the right solution for our issue: I tested this PR talking with @MrRio on Slack last month, and I tested it again now.
I guess the problem is due the fact that it uses locale check in first instance. So this is what happens in 3 different cases:

  • npm start form iTerm2 (where the right locale is set by default): in hyperterm LANG inherits the right one LANG="it_IT.UTF-8"
  • npm start from hyperterm 0.7.1 (where my locale is wrong set to it.UTF-8): LANG is set to LANG="it.UTF-8"
  • packaged app: LANG is completely wrong formatted LANG=""C".UTF-8"

@leo
Copy link
Contributor

leo commented Aug 30, 2017

Shouldn't the dependency be defined in app/package.json?

@chabou chabou mentioned this pull request Nov 20, 2017
@Stanzilla Stanzilla added the 👀 Awaiting Response Issue or PR is awaiting a response from the author label Apr 23, 2018
@Stanzilla
Copy link
Collaborator

I made a rebased version at #2913

@Stanzilla
Copy link
Collaborator

Thanks for your contribution, now committed as #2913

@Stanzilla Stanzilla closed this Apr 27, 2018
@leo leo deleted the os-locale branch August 9, 2018 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 Awaiting Response Issue or PR is awaiting a response from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants