Don't call FixJavaHome() during dart-editor* builds

client/tools/buildbot_annotated_steps.py

 # Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 #!/usr/bin/python
 
 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
@@ skipping 133 matching lines @@
   if len(sys.argv) == 0:
     print 'Script pathname not known, giving up.'
     return 1
 
   scriptdir = os.path.dirname(sys.argv[0])
   # Get at the top-level directory. This script is in client/tools
   os.chdir(os.path.abspath(os.path.join(scriptdir, os.pardir, os.pardir)))
 
   #TODO(sigmund): remove this indirection once we update our bots
   (name, version) = GetBuildInfo()
-  # The buildbot will set a BUILDBOT_JAVA_HOME relative to the dart
-  # root directory, set JAVA_HOME based on that.
-  FixJavaHome()
   if name.startswith('dart-editor'):
+    # TODO (danrubel) Fix dart-editor builds so that we can call FixJavaHome() before the build

[ricow1, 2012/06/21 05:56:02]

long line

     status = ProcessTools('release', name, version)
   else:
+    # The buildbot will set a BUILDBOT_JAVA_HOME relative to the dart
+    # root directory, set JAVA_HOME based on that.
+    FixJavaHome()
     status = ProcessCompiler(name)
 
   if status:
     print '@@@STEP_FAILURE@@@'
 
   return status
 
 
 if __name__ == '__main__':
   sys.exit(main())