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

Issue 839472 link

Starred by 3 users
Status: Fixed
Owner:
jkardatzke@chromium.org
Closed: May 2018
Cc:
█ reve...@chromium.org
tbuck...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Add BROWSER env var for garcon url handling

Project Member Reported by jkardatzke@chromium.org, May 3 2018 
Some apps will use the $BROWSER env variable for launching the default browser. We should set this to be:
/usr/bin/garcon-url-handler

This should be easy for someone who knows where to do that. :)

Comment 1 by vapier@chromium.org, May 9 2018

Components: OS>Systems>Containers

Comment 2 by vapier@chromium.org, May 9 2018

Components: OS>Systems>Containers

Comment 3 by smbar...@chromium.org, May 9 2018

Labels: -Restrict-View-Google

Comment 4 by tbuck...@chromium.org, May 9 2018

Status: Assigned (was: Untriaged)
<triage>smbarber, is this fixed?</triage>

Comment 5 by tbuck...@chromium.org, May 14 2018

Labels: Hotlist-Crostini-Platform

Comment 6 by jkardatzke@chromium.org, May 16 2018

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

Comment 8 by jkardatzke@chromium.org, May 16 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment