commit 984d0628aba8e7560c0665c9f2216a03e11240ae
author Mike Frysinger <vapier@chromium.org> Fri Jun 01 16:08:44 2012 -0400
committer Gerrit <chrome-bot@google.com> Mon Jun 04 10:31:57 2012 -0700
tree 59d2e9916d890155b13224b1baa73d5214e4d6a9
parent a5d82f08e0f5e31edd29f7939cc99268bb08407e

cros_setup_toolchains: do not require root for --show-board-cfg

The --show-board-cfg option just parses config files, so no need to
require it to be run as root.

BUG=None
TEST=`~/trunk/chromite/bin/cros_setup_toolchains --show-board-cfg=x86-generic` works as non-root

Change-Id: Ia6b992229a311850fc34886b4243364cfef4771d
Reviewed-on: https://gerrit.chromium.org/gerrit/24316
Reviewed-by: Zdenek Behan <zbehan@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Ready: Mike Frysinger <vapier@chromium.org>

scripts/cros_setup_toolchains.py

diff --git a/scripts/cros_setup_toolchains.py b/scripts/cros_setup_toolchains.py
index b7c1264..7c91b76 100644
--- a/scripts/cros_setup_toolchains.py
+++ b/scripts/cros_setup_toolchains.py
@@ -628,14 +628,14 @@
 
   (options, _remaining_arguments) = parser.parse_args(argv)
 
-  # This has to be always ran as root.
-  if not os.getuid() == 0:
-    print "%s: This script must be run as root!" % sys.argv[0]
-    sys.exit(1)
-
   if options.board_cfg:
     print ','.join(GetToolchainsForBoard(options.board_cfg))
     return 0
 
+  # This has to be always run as root.
+  if not os.getuid() == 0:
+    print "%s: This script must be run as root!" % sys.argv[0]
+    sys.exit(1)
+
   targets = set(options.targets.split(','))
   UpdateToolchains(options.usepkg, options.deleteold, options.hostonly, targets)