New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 652326 link

Starred by 6 users

Issue metadata

Status: Verified
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

no locales available in system anymore: warning: setlocale: LC_ALL: cannot change locale (en_US.utf8)

Project Member Reported by w...@chromium.org, Oct 3 2016

Issue description

Version: 55.0.2858.0
OS: ChromeOS

What steps will reproduce the problem?
(1) Open crosh (Ctrl+Alt+T).

What is the expected output?

What do you see instead?

A new warning has appeared when crosh is launched:

/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.utf8)
 
Components: -Platform>Apps>Default>Hterm OS>Systems
Labels: -M-55
Owner: vapier@chromium.org
Status: Available (was: Untriaged)
Summary: no locales available in system anymore: warning: setlocale: LC_ALL: cannot change locale (en_US.utf8) (was: crosh logs a warning when opened)
we stopped generating the en_US.UTF8 locale before as it was a workaround for a bug in Chrome UI input.  but that means we don't ship any locales at all.  would be nice if we could figure out a way to pull or generate locales on demand, but that requires tools/data files that we don't ship in release images since they add significant overhead.
Cc: rjahagir@chromium.org helenzhang@chromium.org ka...@chromium.org shrawan@chromium.org snanda@chromium.org
 Issue 650403  has been merged into this issue.
 Issue 642464  has been merged into this issue.
Labels: M-55
So this is what triggered the regression?

https://chromium-review.googlesource.com/#/c/368200/

I love the comment here:

https://bugs.chromium.org/p/chromium/issues/detail?id=117330#c43

"assuming it doesn't get reverted, we're good to go :)"

Comment 6 by vapier@chromium.org, Oct 13 2016

yes, that CL did it.  looks like i need to cleanup the workaround that's in the session manager launcher too:
  https://gerrit.chromium.org/gerrit/17490
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/c0be6d55d5aa9454bd848705dbd7ba27507e76d4

commit c0be6d55d5aa9454bd848705dbd7ba27507e76d4
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Oct 13 03:31:18 2016

libchromeos-ui: drop LC_ALL env var setting

We no longer create this locale, nor do we use it, so we don't need
to set it either.

BUG= chromium:117330 
BUG= chromium:652326 
TEST=precq still passes

Change-Id: I5afce1359371034a6af483fbf748d055b836be5a
Reviewed-on: https://chromium-review.googlesource.com/397682
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Brian Norris <briannorris@chromium.org>

[modify] https://crrev.com/c0be6d55d5aa9454bd848705dbd7ba27507e76d4/libchromeos-ui/chromeos/ui/chromium_command_builder.cc
[modify] https://crrev.com/c0be6d55d5aa9454bd848705dbd7ba27507e76d4/libchromeos-ui/chromeos/ui/chromium_command_builder_unittest.cc

Comment 8 by vapier@chromium.org, Oct 14 2016

Labels: Merge-Request-55
should be fairly benign change to merge back

Comment 9 by dimu@chromium.org, Oct 15 2016

Labels: -Merge-Request-55 Merge-Approved-55 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M55 (branch: 2883)
Project Member

Comment 10 by bugdroid1@chromium.org, Oct 17 2016

Labels: merge-merged-release-R55-8872.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/1e9c3dac5c903b32026f247e9af1b7fb8d66a069

commit 1e9c3dac5c903b32026f247e9af1b7fb8d66a069
Author: Mike Frysinger <vapier@chromium.org>
Date: Thu Oct 13 03:31:18 2016

libchromeos-ui: drop LC_ALL env var setting

We no longer create this locale, nor do we use it, so we don't need
to set it either.

BUG= chromium:117330 
BUG= chromium:652326 
TEST=precq still passes
(cherry picked from commit c0be6d55d5aa9454bd848705dbd7ba27507e76d4)

Change-Id: I5901f5a2fff9379ec9757811d266e081f4214fea
Reviewed-on: https://chromium-review.googlesource.com/399543
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/1e9c3dac5c903b32026f247e9af1b7fb8d66a069/libchromeos-ui/chromeos/ui/chromium_command_builder.cc
[modify] https://crrev.com/1e9c3dac5c903b32026f247e9af1b7fb8d66a069/libchromeos-ui/chromeos/ui/chromium_command_builder_unittest.cc

Status: Fixed (was: Available)
Project Member

Comment 12 by sheriffbot@chromium.org, Oct 18 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Merge-Approved -Merge-Approved-55 Merge-Merged
I guess the auto-labeling features of the merge queue haven't adjusted to our newfangled labeling scheme?
Status: Verified (was: Fixed)
FWIW, no problem:

Version 55.0.2883.17 dev
Platform 8872.15.0
Project Member

Comment 15 by bugdroid1@chromium.org, Oct 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/99461a21a2193c2afbb2e685f85f7cb3e4dd722f

commit 99461a21a2193c2afbb2e685f85f7cb3e4dd722f
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Wed Oct 26 18:37:40 2016

Revert "libchromeos-ui: drop LC_ALL env var setting"

This reverts commit 1e9c3dac5c903b32026f247e9af1b7fb8d66a069.

For now revert only on R55. For master discussion on  crbug.com/649787 .

BUG= chromium:117330 
BUG= chromium:652326 
BUG= chromium:649787 

Change-Id: I38f752247add52de3d7c02d2ab2a7adf8ee850c3
Reviewed-on: https://chromium-review.googlesource.com/403834
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Bernie Thompson <bhthompson@chromium.org>
Tested-by: Bernie Thompson <bhthompson@chromium.org>

[modify] https://crrev.com/99461a21a2193c2afbb2e685f85f7cb3e4dd722f/libchromeos-ui/chromeos/ui/chromium_command_builder.cc
[modify] https://crrev.com/99461a21a2193c2afbb2e685f85f7cb3e4dd722f/libchromeos-ui/chromeos/ui/chromium_command_builder_unittest.cc

Project Member

Comment 16 by bugdroid1@chromium.org, Nov 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/ea9399310a8dcd048442e102f1ad56e8c5367dce

commit ea9399310a8dcd048442e102f1ad56e8c5367dce
Author: Ilja H. Friedel <ihf@chromium.org>
Date: Tue Nov 08 06:22:56 2016

Revert "libchromeos-ui: drop LC_ALL env var setting"

This reverts commit c0be6d55d5aa9454bd848705dbd7ba27507e76d4.

This should fix Facebook games on master.

BUG= chromium:117330 
BUG= chromium:652326 
BUG= chromium:649787 

Change-Id: I160376aa99cb956fa11a1f1c9b265f20fe56771f
Reviewed-on: https://chromium-review.googlesource.com/408818
Commit-Ready: Ilja H. Friedel <ihf@chromium.org>
Tested-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>

[modify] https://crrev.com/ea9399310a8dcd048442e102f1ad56e8c5367dce/libchromeos-ui/chromeos/ui/chromium_command_builder.cc
[modify] https://crrev.com/ea9399310a8dcd048442e102f1ad56e8c5367dce/libchromeos-ui/chromeos/ui/chromium_command_builder_unittest.cc

Sign in to add a comment