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

Doesn't respect LC_MESSAGES #183

Open
p-baum opened this issue Oct 29, 2019 · 5 comments
Open

Doesn't respect LC_MESSAGES #183

p-baum opened this issue Oct 29, 2019 · 5 comments

Comments

@p-baum
Copy link

p-baum commented Oct 29, 2019

I'm getting a German language interface because of my (mostly) German settings. My LC_MESSAGES is however set to en_GB.UTF-8 which 95% of software respects. Sadly it appears lazydocker does not.

For now I will workaround with

LC_ALL=$LC_MESSAGES lazydocker

@dawidd6
Copy link
Collaborator

dawidd6 commented Oct 29, 2019

We are using github.com/cloudfoundry/jibber_jabber package for detecting what locale a user is using. If I see correctly in their source, they do not check LC_MESSAGES, in fact I'm surprised that other software is respecting this environment variable higher that LC_ALL, but maybe I just don't know something.

A quick workaround for that would be to check for LC_MESSAGES after this line:

userLang := detectLanguage(jibber_jabber.DetectLanguage)
, but I don't really think it is the ideal solution.

@mjarkk
Copy link
Contributor

mjarkk commented Oct 29, 2019

@dawidd6 I don't think it's a good idea we do this.
This is really just a problem of jibber_jabber and we or @p-baum should report the problem there.

It's much cleaner if they fix the problem and we just update our libraries,
No hacking required to fix the problem.

@dawidd6
Copy link
Collaborator

dawidd6 commented Oct 29, 2019

@mjarkk judging by activity in their repository, it might not be fixed anytime soon.

@mjarkk
Copy link
Contributor

mjarkk commented Oct 29, 2019

I'll open an issue and wait for a few days.
If no-one will reply i think it's best we fork it and make the changes our-self.

@mjarkk
Copy link
Contributor

mjarkk commented Oct 29, 2019

Created the issue: cloudfoundry-attic/jibber_jabber#5

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

No branches or pull requests

3 participants