commit 80eefa9b82532318cbbfcbb52faa9ac9c063d146
author Mike Frysinger <vapier@chromium.org> Mon Sep 17 23:51:12 2012 -0400
committer Gerrit <chrome-bot@google.com> Tue Sep 18 09:38:58 2012 -0700
tree 0593170a8e9d428c44092c80e60335ca98392c49
parent 158289eb3a6d98848378d94668cb1810c63ddb12

cros_choose_profile: do nothing if selected profile is the same

This lets us not worry about calling this script if the computed
path hasn't changed which simplifies calling code.

BUG=chromium-os:34519
TEST=running multiple times doesn't warn:
	cros_choose_profile --board stumpy --profile ''
TEST=running warns:
	cros_choose_profile --board stumpy --profile ''
	cros_choose_profile --board stumpy --profile 'kernel-next'

Change-Id: Ib052b4a6163a013739dea22abc30c32d3b4c4fc8
Reviewed-on: https://gerrit.chromium.org/gerrit/33483
Reviewed-by: Chris Sosa <sosa@chromium.org>
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

host/cros_choose_profile

diff --git a/host/cros_choose_profile b/host/cros_choose_profile
index 0b2476f..0ba1c08 100755
--- a/host/cros_choose_profile
+++ b/host/cros_choose_profile
@@ -136,7 +136,8 @@
   warn "will fail"
 fi
 
-if [ -e "${MAKE_PROFILE}" ]; then
+CURR_PROFILE=$(readlink "${MAKE_PROFILE}" 2>/dev/null)
+if [[ ${CURR_PROFILE} != "${PROFILES_DIR}" ]]; then
   warn "You are switching profiles for a board that is already setup.  This"
   warn "can cause trouble for Portage.  If you experience problems with"
   warn "build_packages you may need to run:"
@@ -147,6 +148,5 @@
   warn "'emerge-${BOARD_VARIANT} -c' or 'emerge-${BOARD_VARIANT} -C <ebuild>'"
 
   sudo rm -f "${MAKE_PROFILE}"
+  sudo ln -sf "${PROFILES_DIR}" "${MAKE_PROFILE}"
 fi
-
-sudo ln -sf "${PROFILES_DIR}" "${MAKE_PROFILE}"