Shared library for embedding VM in Android apps.

tools/gyp/configurations_android.gypi

Index: tools/gyp/configurations_android.gypi
===================================================================
--- tools/gyp/configurations_android.gypi	(revision 15456)
+++ tools/gyp/configurations_android.gypi	(working copy)
@@ -40,7 +40,35 @@
       'ANDROID',
     ],
     'configurations': {
+      'Dart_Debug': {
+        'defines': [
+          'DEBUG',
+        ],
+      },
+      'Dart_Release': {

[vsm, 2012/11/29 23:13:30]

Why do you need Release and Dart_Release?

[gram, 2012/11/29 23:23:23]

We probably don't - but Release was there before, and didn't work.
Dart_Release/Dart_Debug did work so this was an addition, but if you think I
should remove the old Release stuff I will.

[vsm, 2012/11/29 23:41:32]

Sure, I'd get rid of the old code if it doesn't seem to do anything.  Less
confusion on which one to edit.  We can always add it back in if it really meant
something.  Doesn't look like it though.

On 2012/11/29 23:23:23, gram wrote:

+        'defines': [
+          'NDEBUG',
+        ],
+        'cflags!': [
+          '-O2',
+          '-Os',
+        ],
+        'cflags': [
+          '-fdata-sections',
+          '-ffunction-sections',
+          '-fomit-frame-pointer',
+          '-O3',
+        ],
+      },  # Release

[vsm, 2012/11/29 23:13:30]

I think "#Release" meant of end-of-"Release" in the original.  It's Dart_Release
here.  Maybe just delete the comment.

[gram, 2012/11/29 23:23:23]

Done.

+      'Debug': {
+        'defines': [
+          'DEBUG',
+        ],
+      },
       'Release': {
+        'defines': [
+          'NDEBUG',
+        ],
         'cflags!': [
           '-O2',
           '-Os',