Adding helper script to make it easy to run common DOM tasks.

tools/dom/dom.py

Index: tools/dom/dom.py
diff --git a/tools/dom/dom.py b/tools/dom/dom.py
new file mode 100755
index 0000000000000000000000000000000000000000..847c399dd0795a5794ab88668bca5a050e93a376
--- /dev/null
+++ b/tools/dom/dom.py
@@ -0,0 +1,136 @@
+#!/usr/bin/python
+
+# Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file

[vsm, 2013/02/05 20:29:24]

2013 :-)

[blois, 2013/02/06 22:36:27]

Done.

+# 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 makes it easy to execute common DOM-related tasks
+
+import subprocess
+import sys
+import re
+import os
+import glob

[vsm, 2013/02/05 20:29:24]

nit: alphabetize imports

[blois, 2013/02/06 22:36:27]

Done.

+
+dart_out_dir = 'out/ReleaseIA32/'

[vsm, 2013/02/05 20:29:24]

'out' is 'xcodebuild' on the Mac.  I think most of the dom scripts have worked
on the mac.  Also, os.path.join instead of '/'?

[Emily Fortuna, 2013/02/06 18:36:14]

and 'out' is called 'build' on Windows. the file in tools/utils.py has a
function called GetBuildDir that can be used for just this purpose.

[blois, 2013/02/06 22:36:27]

Done.

+dart_bin = os.path.join(dart_out_dir, 'dart')

[Emily Fortuna, 2013/02/06 18:36:14]

same here ... on Windows it's dart.exe see the function DartBinary in
tools/utils.py

[blois, 2013/02/06 22:36:27]

Done.

+
+class Processor:
+  argv = []

[Anton Muhin, 2013/02/06 11:52:28]

this is not necessary and does slightly different thing than you might expect. 
overall, why do you need a class in the first place?

[blois, 2013/02/06 22:36:27]

Done.

+
+  def process(self, argv):

[vsm, 2013/02/05 20:29:24]

You might consider the optparse python module for args.  I think most of our
python scripts (dom or otherwise) use it.

[Emily Fortuna, 2013/02/06 18:36:14]

+1 Then you won't need to to add the logic for help() -- argparse/optparse will
do that for you.

[blois, 2013/02/06 22:36:27]

From what I can tell, opt parse primarily handles flags, where the primary case
here is passing a series of commands.

for instance, I commonly run:
dom.py gen test_drt html/element_test

or:
dom.py analyze test_drt html

From what I can tell, optparse doesn't really help for non-flags, but I really
wanted to eliminate all flags and just have chaining of common commands.

[Emily Fortuna, 2013/02/06 22:52:39]

Gotcha. Argparse is just what you want then, but it only works for Python 2.7,
and our bots are running 2.6. Sigh.

+    self.argv = argv
+
+    if not argv:
+      self.help()
+
+    while (len(argv) > 0):
+      init_dir()
+      command = argv.pop(0)
+
+      if not hasattr(self, command):

[Anton Muhin, 2013/02/06 11:52:28]

I'd rather have an explicit map 'command name' : function and used a patter
like:

COMMANDS.get(commandName, help)()

which returns help function is there is no mapping for commandName

[blois, 2013/02/06 22:36:27]

Done.

+        self.help()
+        return
+      getattr(self, command)()
+
+  def help(self):
+    print('Commands: ')
+    privates = ['argv', 'process']
+    for attr in dir(self):
+      if not attr in privates and not attr.startswith('__'):

[Anton Muhin, 2013/02/06 11:52:28]

map will help here too

[blois, 2013/02/06 22:36:27]

Done.

+        print('\t%s' % attr)
+
+  def analyze(self):
+    ''' Runs the dart analyzer. '''
+    call([
+      './out/ReleaseIA32/dart-sdk/bin/dart_analyzer',
+      './tests/html/element_test.dart',
+      '--dart-sdk', 'sdk/',
+      '--show-sdk-warnings',
+    ])
+
+  def build(self):
+    ''' Builds the SDK '''
+    call([
+      './tools/build.py',
+      '--mode=release',
+      '--arch=ia32',
+    ])
+
+  def gen(self):
+    os.chdir('tools/dom/scripts/')
+    call([
+      './go.sh',
+    ])
+
+  def size_check(self):
+    ''' Displays the dart2js size of swarm. '''
+    dartfile = 'samples/swarm/swarm.dart'
+    dartfile = os.path.abspath(dartfile)
+    outfile = dartfile + '.js'
+
+    dart2js_path = os.path.join(dart_out_dir, 'dart-sdk', 'bin', 'dart2js')
+
+    call([
+      dart2js_path,
+      dartfile,
+      '--library-root=sdk/',
+      '-o%s' % outfile
+    ])
+    subprocess.call([
+      'du',
+      '-kh',
+      '--apparent-size',
+      outfile,
+    ])
+
+    os.remove(outfile)
+    os.remove(outfile + '.deps')
+    os.remove(outfile + '.map')
+
+  def test_ff(self):
+    test_dart2js('ff', self.argv)
+
+  def test_drt(self):
+    test_dart2js('drt', self.argv)
+
+  def test_chrome(self):
+    test_dart2js('chrome', self.argv)
+
+
+def call(args):
+  print (' '.join(args))
+  subprocess.call(args)
+
+def init_dir():
+  ''' Makes sure that we're always rooted in the dart root folder.'''
+  dart_dir = os.path.abspath(os.path.join(
+      os.path.dirname(os.path.realpath(__file__)),
+      os.path.pardir, os.path.pardir))
+  os.chdir(dart_dir)
+
+def test_dart2js(browser, argv):
+  cmd = [
+    './tools/test.py',
+    '-c', 'dart2js',
+    '-r', browser,
+    '--mode=release',
+    '--checked',
+    '--arch=ia32',
+    '-v',
+  ]
+  if argv:
+    cmd.append(self.argv.pop(0))
+  else:
+    print(
+        'Test commands should be followed by tests to run. Defaulting to html')
+    cmd.append('html')
+  call(cmd)
+
+def main(argv):
+  argv.pop(0)
+  Processor().process(argv)
+
+
+if __name__ == '__main__':
+  main(sys.argv)