SDK build bits for dartfmt.

utils/dartfmt/dartfmt.gyp

Index: utils/dartfmt/dartfmt.gyp
===================================================================
--- utils/dartfmt/dartfmt.gyp	(revision 0)
+++ utils/dartfmt/dartfmt.gyp	(revision 0)
@@ -0,0 +1,36 @@
+# Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file

[ricow1, 2014/02/10 07:12:59]

since we are landing this now -> 2014

[pquitslund, 2014/02/10 18:51:11]

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.
+
+{
+  'targets': [
+    {
+      'target_name': 'dartfmt',
+      'type': 'none',
+      'dependencies': [
+        '../../runtime/dart-runtime.gyp:dart',
+        '../../pkg/pkg.gyp:pkg_packages',
+      ],
+      'actions': [
+        {
+          'action_name': 'generate_dartfmt_snapshot',
+          'inputs': [
+            '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)dart<(EXECUTABLE_SUFFIX)',
+            '../../sdk/lib/_internal/libraries.dart',
+            '<(SHARED_INTERMEDIATE_DIR)/packages.stamp',
+            '<!@(["python", "../../tools/list_files.py", "\\.dart$", "../../pkg/analyzer"])',

[ricow1, 2014/02/10 07:12:59]

we probably depend on this implicitly by the packages.stamp above, but I am fine
with having this an explicit dependency as well

[kustermann, 2014/02/10 08:31:35]

The packages.stamp file only needs to change if there are new package
directories (or removed directories in pkg).

I'm not sure what happens if you have a directory as input to an action (which
is what pkg_packages does): does it consider all files (recursively) as an input
or not?

I think we *should* have this here!

[pquitslund, 2014/02/10 18:51:11]

I have to defer to you guys for this bit (and the next).  We're now officially
out of my depth with the build. :)

+          ],
+          'outputs': [
+            '<(SHARED_INTERMEDIATE_DIR)/dartfmt.dart.snapshot',
+          ],
+          'action': [
+            '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)dart<(EXECUTABLE_SUFFIX)',
+            '--snapshot=<(SHARED_INTERMEDIATE_DIR)/dartfmt.dart.snapshot',
+            '--package-root=<(PRODUCT_DIR)/packages/',
+            '../../pkg/analyzer/bin/formatter.dart',

[kustermann, 2014/02/10 08:31:35]

I'm sure I raised this question before:
Why can't we just piggy-back on utils_wrapper.dart.snapshot instead of creating
a new snapshot?

I think this will add another 1+ MB. [Are we concerned about start-up time?]

[pquitslund, 2014/02/10 18:51:11]

I'm not sure if startup is a huge issue.  Certainly not yet.  Feel free to
change this as you see fit.

+          ],
+        },
+      ],
+    },
+  ],
+}