From 189d85d20e831358a8ae960298d4e5b0329bbaba Mon Sep 17 00:00:00 2001
From: Anush Elangovan <anush@chromium.org>
Date: Fri, 15 Oct 2010 11:36:53 -0700
Subject: [PATCH] Revert "Fix issue with stale cache from bad builds."

This reverts commit 365d4b0cc8b21204e6e093bf910d53d3673f113c.

Fix preflight breakage, this seems like the most possible cause. Will recommit if not.

BUG=
TEST=

Review URL: http://codereview.chromium.org/3767012

Change-Id: I93fe463fec77c445e7ff0ec86db25d5ef2997ad7
---
 bin/cbuildbot.py        | 11 ++---------
 cros_mark_all_as_stable |  2 --
 cros_mark_as_stable.py  |  3 +--
 3 files changed, 3 insertions(+), 13 deletions(-)

diff --git a/bin/cbuildbot.py b/bin/cbuildbot.py
index f000a576c3..37a8b6e269 100755
--- a/bin/cbuildbot.py
+++ b/bin/cbuildbot.py
@@ -345,14 +345,6 @@ def _UprevCleanup(buildroot, error_ok=False):
               '--tracking_branch="cros/master"', 'clean'],
              cwd=cwd, error_ok=error_ok)
 
-  workon_packages = RunCommand(['./cros_workon', 'list'],
-                               enter_chroot=True, cwd=cwd, error_ok=error_ok,
-                               redirect_stdout=True)
-  if workon_packages:
-    Info('Cleaning up workon packages %s' % workon_packages)
-    RunCommand(['./cros_workon', 'stop'] + workon_packages.split(),
-               enter_chroot=True, cwd=cwd, error_ok=False)
-
 
 def _UprevPush(buildroot):
   """Pushes uprev changes to the main line."""
@@ -447,6 +439,7 @@ def main():
         # Master bot needs to check if the other slaves completed.
         if cbuildbot_comm.HaveSlavesCompleted(config):
           _UprevPush(buildroot)
+          _UprevCleanup(buildroot)
         else:
           # At least one of the slaves failed or we timed out.
           _UprevCleanup(buildroot)
@@ -456,7 +449,7 @@ def main():
         if buildconfig['important']:
           cbuildbot_comm.PublishStatus(cbuildbot_comm.STATUS_BUILD_COMPLETE)
 
-      _UprevCleanup(buildroot)
+        _UprevCleanup(buildroot)
   except:
     # Send failure to master bot.
     if not buildconfig['master'] and buildconfig['important']:
diff --git a/cros_mark_all_as_stable b/cros_mark_all_as_stable
index c01ffdcc6d..e85bb0f942 100755
--- a/cros_mark_all_as_stable
+++ b/cros_mark_all_as_stable
@@ -99,8 +99,6 @@ if [ -n "${PACKAGE_LIST}" ] ; then
   ./cros_mark_as_stable --board ${FLAGS_board} -p "${PACKAGE_LIST}" \
     -i "${COMMIT_ID_LIST}" -t ${FLAGS_tracking_branch} commit  || \
     die "Could not mark all packages as stable"
-
-  ./cros_workon start ${PACKAGE_LIST}
 else
   info "No candidate packages to be marked"
 fi
diff --git a/cros_mark_as_stable.py b/cros_mark_as_stable.py
index cacb6abfad..9e5666e612 100755
--- a/cros_mark_as_stable.py
+++ b/cros_mark_as_stable.py
@@ -123,8 +123,7 @@ def _PrintUsageAndDie(error_message=''):
     command_usage += '  %s: %s\n' % (command, _COMMAND_DICTIONARY[command])
   commands_str = '|'.join(commands)
   Warning('Usage: %s FLAGS [%s]\n\n%s\nFlags:%s' % (sys.argv[0], commands_str,
-                                                    command_usage,
-                                                    gflags.FLAGS))
+                                                  command_usage, gflags.FLAGS))
   if error_message:
     Die(error_message)
   else: