Significant cleanup of create_sdk now that libraries have

tools/create_sdk.py

Index: tools/create_sdk.py
===================================================================
--- tools/create_sdk.py	(revision 11618)
+++ tools/create_sdk.py	(working copy)
@@ -22,46 +22,27 @@
 # ......dart_api.h
 # ......dart_debugger_api.h
 # ....lib/
-# ......builtin/
-# ........builtin_runtime.dart
-# ......io/
-# ........io_runtime.dart
-# ........runtime/
+# ......_internal/
 # ......compiler/
+# ......config/
 # ......core/
-# ........core_runtime.dart
-# ........runtime/
 # ......coreimpl/
-# ........coreimpl_runtime.dart
-# ........runtime/
-# ......dart2js/
-# ......isolate/
-# ........isolate_{dart2js, runtime}.dart
-# ........dart2js/
-# ......dom/
-# ........dom.dart
-# ......html/
-# ........html_dart2js.dart
-# ........html_dartium.dart
 # ......crypto/
-# ........crypto.dart
-# ........(implementation files)
+# ......html/
+# ......io/
+# ......isolate/
 # ......json/
-# ........json_frog.dart
-#.........json.dart
-# ........{frog}/
+# ......math/
 # ......mirrors/
-# ........mirrors.dart
 # ......uri/
-# ........uri.dart
 # ......utf/
 # ......web/
-# ........web.dart
 # ....pkg/
 # ......args/
 # ......dartdoc/
 # ......intl/
 # ......logging/
+# ......unittest/
 # ......(more will come here)
 # ....util/
 # ......analyzer/
@@ -121,30 +102,12 @@
 
 
 def CopyDart2Js(build_dir, sdk_root):
-  '''
-  Install dart2js in SDK/lib/dart2js.
-
-  Currently, we copy too much stuff to this location, but the SDK's
-  layout matches the the layout of the part of the repository we're
-  dealing with here which frees us from rewriting files. The long term
-  plan is to align the layout of the repository and the SDK, at which
-  point we should be able to simplify Main below and share the dart
-  files between the various components to minimize SDK download size.
-  '''
-  copytree('lib', os.path.join(sdk_root, 'lib', 'dart2js', 'lib'),
-           ignore=ignore_patterns('.svn'))
-  copytree(os.path.join('runtime', 'lib'),
-           os.path.join(sdk_root, 'lib', 'dart2js', 'runtime', 'lib'),
-           ignore=ignore_patterns('.svn'))
-  copytree(os.path.join('runtime', 'bin'),
-           os.path.join(sdk_root, 'lib', 'dart2js', 'runtime', 'bin'),
-           ignore=ignore_patterns('.svn'))
   if utils.GuessOS() == 'win32':
     dart2js = os.path.join(sdk_root, 'bin', 'dart2js.bat')
     Copy(os.path.join(build_dir, 'dart2js.bat'), dart2js)
     ReplaceInFiles([dart2js],
                    [(r'%SCRIPTPATH%\.\.\\lib',
-                     r'%SCRIPTPATH%..\lib\dart2js\lib')])
+                     r'%SCRIPTPATH%..\lib')])
     dartdoc = os.path.join(sdk_root, 'bin', 'dartdoc.bat')
     Copy(os.path.join(build_dir, 'dartdoc.bat'), dartdoc)
   else:
@@ -152,7 +115,7 @@
     Copy(os.path.join(build_dir, 'dart2js'), dart2js)
     ReplaceInFiles([dart2js],
                    [(r'\$BIN_DIR/\.\./\.\./lib',
-                     r'$BIN_DIR/../lib/dart2js/lib')])
+                     r'$BIN_DIR/../lib')])
     dartdoc = os.path.join(sdk_root, 'bin', 'dartdoc')
     Copy(os.path.join(build_dir, 'dartdoc'), dartdoc)
 
@@ -249,62 +212,15 @@
   dest_file.close()
 
   #
-  # Create and populate lib/compiler.
-  #
-  compiler_src_dir = join(HOME, 'lib', 'compiler')
-  compiler_dest_dir = join(LIB, 'compiler')
-
-  copytree(compiler_src_dir, compiler_dest_dir, ignore=ignore_patterns('.svn'))
-
-  # Remap imports in lib/compiler/* .
-  for (dirpath, subdirs, filenames) in os.walk(compiler_dest_dir):
-    for filename in filenames:
-      if filename.endswith('.dart'):
-        filename = join(dirpath, filename)
-        file_contents = open(filename).read()
-        file = open(filename, 'w')
-        file_contents = re.sub(r"\.\./lib", "..", file_contents)
-        file.write(file_contents)
-        file.close()
-
-  #
-  # Create and populate lib/html.
-  #
-  html_src_dir = join(HOME, 'lib', 'html')
-  html_dest_dir = join(LIB, 'html')
-  os.makedirs(html_dest_dir)
-
-  copyfile(join(html_src_dir, 'dart2js', 'html_dart2js.dart'),
-           join(html_dest_dir, 'html_dart2js.dart'))
-  copyfile(join(html_src_dir, 'dartium', 'html_dartium.dart'),
-           join(html_dest_dir, 'html_dartium.dart'))
-  copyfile(join(html_src_dir, 'dartium', 'nativewrappers.dart'),
-           join(html_dest_dir, 'nativewrappers.dart'))
-
-  #
-  # Create and populate lib/dom.
-  #
-  dom_src_dir = join(HOME, 'lib', 'dom')
-  dom_dest_dir = join(LIB, 'dom')
-  os.makedirs(dom_dest_dir)
-
-  copyfile(join(dom_src_dir, 'dart2js', 'dom_dart2js.dart'),
-           join(dom_dest_dir, 'dom_dart2js.dart'))
-
-  #
   # Create and populate lib/{core, crypto, isolate, json, uri, utf, ...}.
   #
 
-  for library in ['_internal', 'core', 'coreimpl', 'crypto', 'isolate', 'json',
-                  'math', 'mirrors', 'uri', 'utf', 'web']:
-    src_dir = join(HOME, 'lib', library)
-    dest_dir = join(LIB, library)
-    os.makedirs(dest_dir)
+  for library in ['_internal', 'compiler', 'config', 'html', 'core', 'coreimpl',
+                  'crypto', 'isolate', 'json', 'math', 'mirrors', 'uri', 'utf',
+                  'web']:
+    copytree(join(HOME, 'lib', library), join(LIB, library),
+             ignore=ignore_patterns('*.svn', 'doc', '*.py', '*.gypi', '*.sh'))
 
-    for filename in os.listdir(src_dir):
-      if filename.endswith('.dart'):
-        copyfile(join(src_dir, filename), join(dest_dir, filename))
-
   # Create and copy pkg.
   PKG = join(SDK_tmp, 'pkg')
   os.makedirs(PKG)
@@ -314,17 +230,9 @@
   #
 
   for library in ['args', 'intl', 'logging', 'unittest']:
-    src_dir = join(HOME, 'pkg', library)
-    dest_dir = join(PKG, library)
-    os.makedirs(dest_dir)
+    copytree(join(HOME, 'pkg', library), join(PKG, library),
+             ignore=ignore_patterns('*.svn', 'doc', '*.py', '*.gypi', '*.sh'))

[Bob Nystrom, 2012/08/31 17:52:31]

This means we'll be including each package's tests in the SDK, right? Is that
intended?

[dgrove, 2012/08/31 18:25:42]

For now, that's intended. We can pull them out before M1 if we think it's
important.

 
-    for filename in os.listdir(src_dir):
-      if filename.endswith('.dart'):
-        copyfile(join(src_dir, filename), join(dest_dir, filename))
-    if exists(join(src_dir, 'lib')):
-      copytree(join(src_dir, 'lib'), join(dest_dir, 'lib'),
-           ignore=ignore_patterns('.svn'))
-
   # Create and populate pkg/dartdoc.
   dartdoc_src_dir = join(HOME, 'pkg', 'dartdoc')
   dartdoc_dest_dir = join(PKG, 'dartdoc')
@@ -370,14 +278,6 @@
   copytree(pub_src_dir, pub_dst_dir,
            ignore=ignore_patterns('.svn', 'sdk'))
 
-  # Copy import maps.

[danrubel, 2012/08/31 17:27:51]

We still need this section in there for a little while longer.

[dgrove, 2012/08/31 18:25:42]

config is actually being copied in line 218. Turns out that what's currently
here is overkill.

[danrubel, 2012/08/31 19:24:52]

Good by me!

-  PLATFORMS = ['any', 'vm', 'dartium', 'dart2js' ]
-  os.makedirs(join(LIB, 'config'))
-  for platform in PLATFORMS:
-    import_src = join(HOME, 'lib', 'config', 'import_' + platform + '.config')
-    import_dst = join(LIB, 'config', 'import_' + platform + '.config')
-    copyfile(import_src, import_dst);
-
   # Copy dart2js.
   CopyDart2Js(build_dir, SDK_tmp)