Build Tools Cleanup

tools/create_sdk.py

 #!/usr/bin/env python
 #
-# Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
+# Copyright (c) 2014, 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.
 #
 # A script which will be invoked from gyp to create an SDK.
 #
 # Usage: create_sdk.py sdk_directory
 #
 # The SDK will be used either from the command-line or from the editor.
 # Top structure is
 #
@@ skipping 27 matching lines @@
 # ......io/
 # ......isolate/
 # ......js/
 # ......math/
 # ......mirrors/
 # ......typed_data/
 # ....util/
 # ......(more will come here)
 
 
-import glob
 import optparse
 import os
 import re
 import sys
 import subprocess
-import tempfile
+
 import utils
 
+
 HOST_OS = utils.GuessOS()
 
 # TODO(dgrove): Only import modules following Google style guide.
-from os.path import basename, dirname, join, realpath, exists, isdir
+from os.path import basename, dirname, join, realpath, exists
 
 # TODO(dgrove): Only import modules following Google style guide.
 from shutil import copyfile, copymode, copytree, ignore_patterns, rmtree, move
 
 
 def GetOptions():
   options = optparse.OptionParser(usage='usage: %prog [options]')
   options.add_option("--sdk_output_dir",
       help='Where to output the sdk')
   options.add_option("--snapshot_location",
       help='Location of the snapshots.')
   return options.parse_args()
 
 
 def ReplaceInFiles(paths, subs):
-  '''Reads a series of files, applies a series of substitutions to each, and
-     saves them back out. subs should by a list of (pattern, replace) tuples.'''
+  """Reads a series of files, applies a series of substitutions to each, and
+     saves them back out. subs should by a list of (pattern, replace) tuples."""
   for path in paths:
     contents = open(path).read()
     for pattern, replace in subs:
       contents = re.sub(pattern, replace, contents)
 
     dest = open(path, 'w')
     dest.write(contents)
     dest.close()
 
 
 def Copy(src, dest):
   copyfile(src, dest)
   copymode(src, dest)
 
 
 def CopyShellScript(src_file, dest_dir):
-  '''Copies a shell/batch script to the given destination directory. Handles
-     using the appropriate platform-specific file extension.'''
+  """Copies a shell/batch script to the given destination directory. Handles
+     using the appropriate platform-specific file extension."""
   file_extension = ''
   if HOST_OS == 'win32':
     file_extension = '.bat'
 
   src = src_file + file_extension
   dest = join(dest_dir, basename(src_file) + file_extension)
   Copy(src, dest)
 
 
 def CopyDartScripts(home, sdk_root):
   for executable in ['dart2js', 'dartanalyzer', 'dartfmt', 'docgen', 'pub']:
     CopyShellScript(os.path.join(home, 'sdk', 'bin', executable),
                     os.path.join(sdk_root, 'bin'))
 
 
 def CopySnapshots(snapshots, sdk_root):
   for snapshot in ['dart2js', 'dartanalyzer', 'dartfmt', 'utils_wrapper',
                    'pub']:
     snapshot += '.dart.snapshot'
     copyfile(join(snapshots, snapshot),
              join(sdk_root, 'bin', 'snapshots', snapshot))
 
 
-def Main(argv):
+def Main():
   # Pull in all of the gypi files which will be munged into the sdk.
   HOME = dirname(dirname(realpath(__file__)))
 
   (options, args) = GetOptions()
 
   SDK = options.sdk_output_dir
   SDK_tmp = '%s.tmp' % SDK
 
   SNAPSHOT = options.snapshot_location
 
@@ skipping 115 matching lines @@
   if revision is not None:
     with open(os.path.join(SDK_tmp, 'revision'), 'w') as f:
       f.write(revision + '\n')
       f.close()
 
   Copy(join(HOME, 'README.dart-sdk'), join(SDK_tmp, 'README'))
 
   move(SDK_tmp, SDK)
 
 if __name__ == '__main__':
-  sys.exit(Main(sys.argv))
+  sys.exit(Main())