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

Issue 839106 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

PATH is not correct in garcon

Project Member Reported by jkardatzke@chromium.org, May 2 2018

Issue description

Chrome OS version: R68-10635.0.0
OS: Chrome OS

Repro steps:
1. Install gnoma-mahjongg
2. Click on the icon in the launcher, it won't run


If you look at the output of 'sudo journalctl' and see that it can't find the gnome-mahjongg binary. If you look at the PATH var for the running garcon process, it does not include /usr/games which is where gnome-mahjongg is at. In the terminal, the PATH does have /usr/games in it.

I even restarted the VM/container and that did not change the PATH that garcon was using. I have a feeling there'
 
Components: OS>Systems>Containers
Components: OS>Systems>Containers
Labels: -Restrict-View-Google
Status: Assigned (was: Untriaged)
<triage>smbarber, is this already fixed?</triage>
Labels: Hotlist-Crostini-Platform
Cc: -jkardatzke@chromium.org
Owner: jkardatzke@chromium.org
Project Member

Comment 7 by bugdroid1@chromium.org, May 16 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/containers/cros-container-guest-tools/+/d6749f01714c76522de321f1ef6d08d8f077750f

commit d6749f01714c76522de321f1ef6d08d8f077750f
Author: Jeffrey Kardatzke <jkardatzke@google.com>
Date: Wed May 16 21:39:59 2018

Add env vars for garcon for BROWSER, XDG_DATA_DIRS and PATH

BUG= chromium:839472 
BUG= chromium:839106 
BUG= chromium:843515 
TEST=Verified garcon's env has the right vars in it

Change-Id: I6c7af19bdfbaa493483f4bcf094e78d642706cf7
Reviewed-on: https://chromium-review.googlesource.com/1062792
Tested-by: Jeffrey Kardatzke <jkardatzke@google.com>
Reviewed-by: Jeffrey Kardatzke <jkardatzke@google.com>

[add] https://crrev.com/d6749f01714c76522de321f1ef6d08d8f077750f/cros-garcon/conffiles
[modify] https://crrev.com/d6749f01714c76522de321f1ef6d08d8f077750f/cros-garcon/BUILD
[add] https://crrev.com/d6749f01714c76522de321f1ef6d08d8f077750f/cros-garcon/cros-garcon-override.conf

Status: Fixed (was: Assigned)

Sign in to add a comment