commit 46d424482d0eac081401bec8330e83e809677038
author Ryan Cui <rcui@google.com> Fri Oct 21 16:03:23 2011 -0700
committer Gerrit <chrome-bot@google.com> Mon Oct 24 12:17:58 2011 -0700
tree 484976ea526e39f79b06140d3e9a7845472057ef
parent 7e44ca25e7b4945cc1a57f2d49273ef004fb9446

Don't set CHROME_ORIGIN environment variable

CL http://gerrit.chromium.org/gerrit/#change,10530 removes dependency of
chrome ebuild on CHROME_ORIGIN being set to GERRIT_SOURCE when user runs
cros_workon.  So remove this functionality.

BUG=chromium-os:21969
TEST=Ran cros_workon start/stop for chrome and cros-devutil

Change-Id: I04f7093b45f8e2871f369331bdafbba36443e21d
Reviewed-on: http://gerrit.chromium.org/gerrit/10531
Reviewed-by: David James <davidjames@chromium.org>
Commit-Ready: Ryan Cui <rcui@chromium.org>
Tested-by: Ryan Cui <rcui@chromium.org>

host/cros_workon

diff --git a/host/cros_workon b/host/cros_workon
index ce7a983..ee4d925 100755
--- a/host/cros_workon
+++ b/host/cros_workon
@@ -70,7 +70,6 @@
 WORKON_DIR=${CHROOT_TRUNK_DIR}/.config/cros_workon
 KEYWORDS_DIR=${BOARD_DIR}/etc/portage/package.keywords
 UNMASK_DIR=${BOARD_DIR}/etc/portage/package.unmask
-PACKAGE_ENV_DIR=${BOARD_DIR}/etc/portage/env
 CHROMITE_DIR=${CHROOT_TRUNK_DIR}/chromite
 WORKON_FILE=${WORKON_DIR}/${FLAGS_board:-host}
 KEYWORDS_FILE=${KEYWORDS_DIR}/cros-workon
@@ -259,23 +258,11 @@
 
 chrome_to_live () {
   # Switch to using repo manifest checkout of chromium src.
-  # TODO(rcui): Remove this when CHROME_ORIGIN defaults to GERRIT_SOURCE.
-  # Tracked in chromium-os:19259.
-  env_dir=${PACKAGE_ENV_DIR}/chromeos-base
-  sudo mkdir -p ${env_dir}
-  sudo_clobber "${env_dir}/chromeos-chrome" <<EOF
-CHROME_ORIGIN=GERRIT_SOURCE
-EOF
   # Run chrome_set_ver to set DEPS
   info "Setting Chrome dependencies to the correct revision."
   ${CHROME_SET_VER}
 }
 
-chrome_to_stable () {
-  env_file=${PACKAGE_ENV_DIR}/${CHROME_ATOM}
-  sudo rm ${env_file}
-}
-
 # Move a stable ebuild to the live development catgeory.  The ebuild
 # src_unpack step fetches the package source for local development.
 ebuild_to_live () {
@@ -309,11 +296,6 @@
     if grep -qx "=${atom}-9999" "${WORKON_FILE}" ; then
       if sudo sed -e "/^=${atom/\//\\/}-9999\$/d" -i "${WORKON_FILE}"; then
         atoms_success="${atoms_success} ${atom}"
-        # TODO(rcui): Remove special case when CHROME_ORIGIN defaults to
-        # GERRIT_SOURCE. Tracked in chromium-os:19259.
-        if [ "${atom}" = "${CHROME_ATOM}" ]; then
-          chrome_to_stable
-        fi
       fi
     else
       warn "Not working on ${atom}"